Changes between Version 4 and Version 6 of CookBook/Configuration/SignedTickets

Timestamp:

(multiple changes)

Author:

(multiple changes)

Comment:

(multiple changes)

CookBook/Configuration/SignedTickets

@@ -22 +22 @@
 }}}
 
+=== Signed Resolution
+
+Add `signed` as a //Resolution// through the web admin page //Admin// -> //Ticket System// -> //Resolutions// (`/admin/ticket/resolution`), or using TracAdmin:
+{{{#!sh
+trac-admin $env resolution add signed
+}}}
+
 === Sign workflow step
 
@@ -27 +34 @@
 {{{#!ini
 [ticket-workflow]
-sign = new -> closed
+sign = new,assigned,accepted,reopened -> closed
+sign.name = resolve
 sign.permissions = TICKET_SIGN
 sign.operations = set_resolution
 sign.set_resolution = signed
+}}}
+
+You probably also want to configure the list of //Resolutions//
+for other actions with a `set_resolution` operation, so
+that the //Signed// resolution does not appear in the list.
+
+{{{#!ini
+resolve.set_resolution = fixed, invalid, wontfix, duplicate, worksforme
 }}}
 
@@ -38 +54 @@
 
 {{{#!python
+# -*- coding: utf-8 -*-
+
 from trac.core import *
 from trac.perm import IPermissionPolicy
 from trac.ticket.model import Ticket
 
+
 class ReadonlySignedTickets(Component):
-      implements(IPermissionPolicy)
 
-      def check_permission(self, action, username, resource, perm):
-          if resource is None or resource.realm != 'ticket' or \
-             resource.id is None or action == 'TICKET_VIEW' or \
-             action == 'TICKET_ADMIN' or 'TICKET_ADMIN' in perm:
-              return None
+    implements(IPermissionPolicy)
 
-          t = Ticket(self.env, resource.id)
-          if t['status'] == 'closed' and t['resolution'] == 'signed':
-             return False
+    allowed_actions = ('TICKET_VIEW',)
+
+    def check_permission(self, action, username, resource, perm):
+        if resource is None or resource.realm != 'ticket' or \
+                resource.id is None or \
+                action in self.allowed_actions or \
+                action == 'TICKET_ADMIN' or 'TICKET_ADMIN' in perm:
+            return None
+
+        t = Ticket(self.env, resource.id)
+        if t['status'] == 'closed' and t['resolution'] == 'signed':
+            return False
 }}}
 
@@ -61 +84 @@
 [trac]
 permission_policies = ReadonlySignedTickets, ...
+}}}
+
+Additional permissions can be specified in `allowed_actions`.
+If you wish to allow commenting on signed tickets, add
+`TICKET_APPEND` to the `allowed_actions`:
+{{{#!python
+    allowed_actions = ('TICKET_VIEW', 'TICKET_APPEND')
 }}}
 
@@ -91 +121 @@
 If you want to use a `signed` ''status'' instead of a resolution, change the last two lines of code to:
 {{{#!python
-          # ...
-          if t['status'] == 'signed':
-             return False
+         # ...
+        if t['status'] == 'signed':
+            return False
 }}}
 And change the workflow step to: