Martijn Pieters wrote:
Log message for revision 38178:
  Massive style cleanup: Move to new raise exception style; for motivation, see:


Modified: Zope3/trunk/src/zope/app/workflow/stateful/definition.py
===================================================================
--- Zope3/trunk/src/zope/app/workflow/stateful/definition.py    2005-08-30 
21:36:02 UTC (rev 38177)
+++ Zope3/trunk/src/zope/app/workflow/stateful/definition.py    2005-08-30 
21:50:19 UTC (rev 38178)
@@ -46,6 +46,10 @@
     implements(IStatefulStatesContainer)
+class NoLocalProcessDefinition(Exception):
+    """No local process definition found"""
+ +
 class StateNamesVocabulary(SimpleVocabulary):
     """Vocabulary providing the names of states in a local process definition.
     """
@@ -61,7 +65,7 @@
             for obj in zapi.getParents(context):
                 if IStatefulProcessDefinition.providedBy(obj):
                     return obj.getStateNames()
-        raise 'NoLocalProcessDefinition', 'No local process definition found.'
+        raise NoLocalProcessDefinition('No local process definition found.')

I'm not sure it's worth worrying about (and I definitely prefer the new version better), but these type conversions are not backward compatible. Of course it's only a problem if anyone is trying to catch the string version of the exception.
--
Benji York
Senior Software Engineer
Zope Corporation
_______________________________________________
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/archive%40mail-archive.com

Reply via email to