[Erp5-report] r24089 - /erp5/trunk/products/CMFActivity/ActivityTool.py

[nobody at svn.erp5.org]

Author: vincent
Date: Wed Oct  8 10:22:54 2008
New Revision: 24089

URL: http://svn.erp5.org?rev=24089&view=rev
Log:
Use try..except..else instead of treating non-raising case in "try" block.

Modified:
    erp5/trunk/products/CMFActivity/ActivityTool.py

Modified: erp5/trunk/products/CMFActivity/ActivityTool.py
URL: http://svn.erp5.org/erp5/trunk/products/CMFActivity/ActivityTool.py?rev=24089&r1=24088&r2=24089&view=diff
==============================================================================
--- erp5/trunk/products/CMFActivity/ActivityTool.py (original)
+++ erp5/trunk/products/CMFActivity/ActivityTool.py Wed Oct  8 10:22:54 2008
@@ -1062,13 +1062,14 @@
           else:
             try:
               m.activateResult(self, result, object)
-              m.is_executed = MESSAGE_EXECUTED
             except:
               m.is_executed = MESSAGE_NOT_EXECUTED
               m.exc_type = sys.exc_info()[0]
               LOG('ActivityTool', WARNING,
                   'Could not call method %s on object %s' % (
                   m.method_id, m.object_path), error=sys.exc_info())
+            else:
+              m.is_executed = MESSAGE_EXECUTED
       if logging:
         LOG('Activity Tracking', INFO, 'invoked group messages')