ERP5Workflow: PERF: remove categories usage on Workflow and Transition

by redefining getSourceValue (workflow) and getDestinationValue (transition)

product/ERP5Workflow/Document/Transition.py

@@ -139,3 +139,16 @@ class Transition(IdAsReferenceMixin("transition_", "prefix"), XMLObject,
     parent = self.getParentValue()
     return [parent._getOb(transition_id) for transition_id
             in self.getAfterScriptIdList()]
+
+  def getDestinationValue(self):
+    """
+    returns the destination object
+    """
+
+    destination_path_list = [path for path in self.getCategoryList()
+                             if path.startswith('destination/')]
+    if destination_path_list:
+      destination_id = destination_path_list[0].split('/')[-1]
+      parent = self.getParentValue()
+      return parent._getOb(destination_id)
+    return None

product/ERP5Workflow/Document/Workflow.py

@@ -61,6 +61,7 @@ from tempfile import mktemp
 from types import StringTypes
 from zLOG import LOG, INFO, WARNING
 
+
 ACTIVITY_GROUPING_COUNT = 100
 
 class Workflow(IdAsReferenceMixin("", "prefix"), XMLObject):
@@ -1210,3 +1211,16 @@ class Workflow(IdAsReferenceMixin("", "prefix"), XMLObject):
           permission_to_delete.append(permission)
       for permission in permission_to_delete:
         del state.state_permission_roles_dict[permission]
+
+  def getSourceValue(self):
+    """
+    returns the source object
+    """
+    # this function is redefined here for performance reasons:
+    # avoiding the usage of categories speeds up workflow *a lot*
+    source_path_list = [path for path in self.getCategoryList()
+                        if path.startswith('source/')]
+    if source_path_list:
+      source_id = source_path_list[0].split('/')[-1]
+      return self._getOb(source_id)
+    return None