[Checkins] SVN: Products.DCWorkflow/trunk/Products/DCWorkflow/ Zap string exceptions.

[Tres Seaver]

Log message for revision 124571:
  Zap string exceptions.

Changed:
  U   Products.DCWorkflow/trunk/Products/DCWorkflow/CHANGES.txt
  U   Products.DCWorkflow/trunk/Products/DCWorkflow/ContainerTab.py
  U   Products.DCWorkflow/trunk/Products/DCWorkflow/Variables.py

-=-
Modified: Products.DCWorkflow/trunk/Products/DCWorkflow/CHANGES.txt
===================================================================
--- Products.DCWorkflow/trunk/Products/DCWorkflow/CHANGES.txt	2012-03-11 16:46:09 UTC (rev 124570)
+++ Products.DCWorkflow/trunk/Products/DCWorkflow/CHANGES.txt	2012-03-11 16:55:35 UTC (rev 124571)
@@ -4,6 +4,9 @@
 2.3.0-alpha (unreleased)
 ------------------------
 
+- Removed string exceptions.
+  (https://bugs.launchpad.net/zope-cmf/+bug/952301)
+
 - Made sure converted tools are used as utilities.
 
 - Don't crash worklist's ``manage_main`` if variables are Expression objects.

Modified: Products.DCWorkflow/trunk/Products/DCWorkflow/ContainerTab.py
===================================================================
--- Products.DCWorkflow/trunk/Products/DCWorkflow/ContainerTab.py	2012-03-11 16:46:09 UTC (rev 124570)
+++ Products.DCWorkflow/trunk/Products/DCWorkflow/ContainerTab.py	2012-03-11 16:55:35 UTC (rev 124571)
@@ -19,6 +19,7 @@
 from Acquisition import aq_inner
 from Acquisition import aq_parent
 from OFS.Folder import Folder
+from zExceptions import BadRequest
 
 _marker = []  # Create a new marker object.
 
@@ -53,7 +54,7 @@
     def _checkId(self, id, allow_dup=0):
         if not allow_dup:
             if self._mapping.has_key(id):
-                raise 'Bad Request', 'The id "%s" is already in use.' % id
+                raise BadRequest('The id "%s" is already in use.' % id)
         return Folder._checkId(self, id, allow_dup)
 
     def _getOb(self, name, default=_marker):
@@ -111,7 +112,7 @@
     def manage_renameObjects(self, ids=[], new_ids=[], REQUEST=None):
         """Rename several sub-objects"""
         if len(ids) != len(new_ids):
-            raise 'Bad Request', 'Please rename each listed object.'
+            raise BadRequest('Please rename each listed object.')
         for i in range(len(ids)):
             if ids[i] != new_ids[i]:
                 self.manage_renameObject(ids[i], new_ids[i])

Modified: Products.DCWorkflow/trunk/Products/DCWorkflow/Variables.py
===================================================================
--- Products.DCWorkflow/trunk/Products/DCWorkflow/Variables.py	2012-03-11 16:46:09 UTC (rev 124570)
+++ Products.DCWorkflow/trunk/Products/DCWorkflow/Variables.py	2012-03-11 16:55:35 UTC (rev 124571)
@@ -21,6 +21,7 @@
 from App.class_init import InitializeClass
 from App.special_dtml import DTMLFile
 from OFS.SimpleItem import SimpleItem
+from zExceptions import BadRequest
 
 from Products.CMFCore.Expression import Expression
 from Products.DCWorkflow.ContainerTab import ContainerTab
@@ -148,7 +149,7 @@
     def _checkId(self, id, allow_dup=0):
         wf_def = aq_parent(aq_inner(self))
         if id == wf_def.state_var:
-            raise 'Bad Request', '"%s" is used for keeping state.' % id
+            raise BadRequest('"%s" is used for keeping state.' % id)
         return ContainerTab._checkId(self, id, allow_dup)
 
     def getStateVar(self):