Changeset 6471 for main/waeup.sirp/trunk/src/waeup/sirp/accesscodes

Timestamp:

23 Jun 2011, 15:29:31 (14 years ago)

Author:

Henrik Bettermann

Message:

Produce more user friendly and shorter object history messages.

Location:

main/waeup.sirp/trunk/src/waeup/sirp/accesscodes

Files:

• browser.py (modified) (2 diffs)
• tests/test_accesscodes.py (modified) (2 diffs)
• workflow.py (modified) (1 diff)

main/waeup.sirp/trunk/src/waeup/sirp/accesscodes/browser.py

@@ -139 +139 @@
             if 'disable' in form:
                 try:
-                    comment = u"Disabled by %s" % self.request.principal.id
+                    comment = u"AC disabled"
                     self.context.disable(entry, comment)
                     self.flash('%s disabled.' % entry)
@@ -146 +146 @@
             elif 'enable' in form:
                 try:
-                    comment = u"Re-enabled by %s" % self.request.principal.id
+                    comment = u"AC re-enabled"
                     self.context.enable(entry, comment)
                     self.flash('%s (re-)enabled.' % entry)

main/waeup.sirp/trunk/src/waeup/sirp/accesscodes/tests/test_accesscodes.py

@@ -192 +192 @@
         history = IObjectHistory(self.ac1)
         msgs = history.messages
-        assert msgs[-1].endswith('Used by system')
+        assert msgs[-1].endswith('AC used by system')
 
 class AccessCodeTests(FunctionalTestCase):
@@ -236 +236 @@
         # Access codes have a history.
         match = re.match(
-            '^../../.... ..:..:.. - Initialized by system',
+            '^....-..-.. ..:..:.. - AC initialized by system',
             self.ac1.history)
         assert match is not None

main/waeup.sirp/trunk/src/waeup/sirp/accesscodes/workflow.py

@@ -120 +120 @@
 def handle_accesscode_transition_event(obj, event):
     # append message to history
-    timestamp = datetime.now().strftime("%d/%m/%Y %H:%M:%S")
-    msg = '%s -' % timestamp
     if event.comment is not None:
-        msg = '%s %s' % (msg, event.comment)
+        msg = '%s' % event.comment
     else:
-        msg = '%s %s by system' % (msg,event.destination.title())
+        msg = 'AC %s' % event.destination
 
     history = IObjectHistory(obj)