Changes between Version 32 and Version 33 of TracWorkflow

Timestamp:

Apr 12, 2010, 3:03:34 PM (14 years ago)

Author:

Marc Wäckerlin

Comment:

How-To Combine SVN trac-post-commit-hook With Test Workflow

TracWorkflow

@@ -116 +116 @@
 }}}
 
+=== How-To Combine SVN trac-post-commit-hook With Test Workflow ===
+
+If you installed a {{{post-commit-hook}}} in your subversion repository that calls {{{trac-post-commit-hook}}} as provided by trac, then you might want to change the behaviour: Now a developer should set the tickets to {{{testing}}}, not {{{closed}}} and assign them to the reporter.
+
+If you change the following method in file {{{trac-post-commit-hook}}}, then the subversion comments to close a ticket, sucht as {{{"closes #64"}}} won't close the ticket any more, but they will put the ticket to status {{{testing}}} and switch the owner to the reporter:
+
+{{{
+    def _cmdClose(self, ticket):
+        ticket['status'] = 'testing'
+        ticket['owner'] = ticket['reporter']
+        #ticket['status'] = 'closed'
+        #ticket['resolution'] = 'fixed'
+}}}
+
+Commented out are the original two lines.
+
+Of course you could also add new keywords, such as {{{test}}} or {{{testing}}} and a new function {{{_cmdClose}}}. Then a developer can choose whether he wants to put the ticket to testing state or to close it.
+
+In that case, extend the {{{_supported_cmds}}}, e.g. to:
+
+{{{
+    _supported_cmds = {'close':      '_cmdClose',
+                       'closed':     '_cmdClose',
+                       'closes':     '_cmdClose',
+                       'fix':        '_cmdClose',
+                       'fixed':      '_cmdClose',
+                       'fixes':      '_cmdClose',
+                       'test':       '_cmdTest',
+                       'testing':    '_cmdTest',
+                       'addresses':  '_cmdRefs',
+                       're':         '_cmdRefs',
+                       'references': '_cmdRefs',
+                       'refs':       '_cmdRefs',
+                       'see':        '_cmdRefs'}
+}}}
+
+... and add a new method {{{_cmdTest}}} just below  {{{_cmdClose}}} (and don't modify {{{_cmdClose}}}):
+{{{
+    def _cmdTest(self, ticket):
+        ticket['status'] = 'testing'
+        ticket['owner'] = ticket['reporter']
+}}}
+
+In this case, don't forget to add the new keywords {{{test}}} and {{{testing}}} to file {{{trac-pre-commit-hook}}}, in case you also installed the subversion {{{pre-commit-hook}}}:
+{{{
+for tmp in re.findall('(?:close|closed|closes|fix|fixed|fixes|test|testing|addresses|references|refs|re|see)'
+}}}
 == Example: Add simple optional generic review state ==