[Patch] Ability to have a default status on triage operations

[norberto.bezi@…]

Added ability to have a default status on triage operations.

The following example will set the status to reopen by default but to bug type tickets it will set development.

Example:

reopen = closed -> *
reopen.operations = del_resolution,triage,del_owner
reopen.permissions = TICKET_CREATE
reopen.triage_field = type
reopen.triage_split = *->reopened,bug->development

SVN Diff:

Index: controller.py
===================================================================
--- controller.py       (revision 8373)
+++ controller.py       (working copy)
@@ -354,13 +354,16 @@
                                 action + '.triage_field').strip()
         transitions = self.config.get('ticket-workflow',
                                       action + '.triage_split').strip()
+        default_status = 'new'
         for transition in [x.strip() for x in transitions.split(',')]:
             value, status = [y.strip() for y in transition.split('->')]
             if value == ticket[field].strip():
                 break
+            else:
+                if value == '*':
+                    default_status = status
         else:
-            self.env.log.error("Bad configuration for 'triage' operation in action '%s'" % action)
-            status = 'new'
+            status = default_status
         return status

[Ryan J Ollos]

Updated description - Wrapped patch code in description with a code block. Btw, patches as attachments are always preferred to inline pastes.

[mklerk@…]

Should be very handy

[Eli Carter]

While I see the usefulness of this syntax extension (especially for text fields where a user might enter an unrecognized value), the patch eliminates the error-checking case.