Opened 4 years ago

Closed 4 years ago

#10523 closed defect (fixed)

leaving blank display names gives malformed labels

Reported by: gary.martin@… Owned by: jun66j5
Priority: normal Component: TracWorkflowAdminPlugin
Severity: normal Keywords:
Cc: Trac Release: 1.0


This is more of a problem for pasted workflows in text mode as the gui encourages a display name to be edited from an initial "Action name" but you are also allowed to remove that name. On saving this currently results in entries of the form =

in the [ticket-workflow] section of the trac.ini and this results in malformed labels on tickets.

This could be considered Trac's fault for not noticing that the entries are blank but something like the following would be one way of solving the problem here (just skips output of {{{}} entries without values).

  • tracworkflowadmin/

    627627                for (name, value) in self.config.options('ticket-workflow'):
    628628                    self.config.remove('ticket-workflow', name)
    629629                out['result'] = 0
    630                 for key in newOptions.keys():
    631                     self.config.set('ticket-workflow', key, newOptions[key])
     630                [self.config.set('ticket-workflow', key, opt)
     631                    for key, opt in newOptions.iteritems()
     632                    if not(key.endswith('.name') and not opt)]
    633634            except Exception, e:
    634635                self.config.parse_if_needed(force=True)

Attachments (0)

Change History (1)

comment:1 Changed 4 years ago by jun66j5

  • Resolution set to fixed
  • Status changed from new to closed

(In [12237]) tracworkflowadmin: fixed blank display for the action in ticket page if is blank

Report and initial patch by Gary Martin, Thanks. Closes #10523.

Add Comment

Modify Ticket

as closed The owner will remain jun66j5.
The resolution will be deleted. Next status will be 'reopened'.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.