Opened 5 years ago

Last modified 17 months ago

#9554 new defect

triage problem with action leave BEFORE triaged

Reported by: anonymous Owned by: Eli Carter
Priority: high Component: AdvancedTicketWorkflowPlugin
Severity: normal Keywords:
Cc: angel Trac Release: 0.12


Trac 0.12.2 Ubuntu 10.04 AdvancedTicketWorkflowPlugin 0.11dev-r9962

My workflow is configured to triage the ticket according to ticket type from new to working_(task) or working_(cr). Having a ticket in state new, doing some changes (comments, milestones, doesn't matter what) and using action leave (configured as in default workflow) causes an error in class TicketWorkflowOpTriage method _new_status, because variables "transistions" and "field" are empty. So the line "value, status = [y.strip() for y in transition.split('->')]" failes because there is nothing to unpack.

   def _new_status(self, ticket, action):
        """Determines the new status"""
        field = self.config.get('ticket-workflow',
                                action + '.triage_field').strip()
        transitions = self.config.get('ticket-workflow',
                                      action + '.triage_split').strip()
        if not field and not transitions:
            return ticket['status']
            for transition in [x.strip() for x in transitions.split(',')]:
                value, status = [y.strip() for y in transition.split('->')]
                if value == ticket[field].strip():
                self.env.log.error("Bad configuration for 'triage' operation in action '%s'" % action)
                status = 'new'
            return status

After adding the "if not field and not transitions:" everthing works fine!

Attachments (0)

Change History (2)

comment:1 Changed 5 years ago by anonymous

Cc: angel added; anonymous removed

comment:2 Changed 17 months ago by ahowell82@…

Thanks this works for the 1.2 version

Modify Ticket

as new The owner will remain Eli Carter.

Add Comment

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

Note: See TracTickets for help on using tickets.