ERP5Workflow: PERF: redefine getDestinationIdList not to use the objects directly

product/ERP5Workflow/Document/State.py

@@ -89,6 +89,16 @@ class State(IdAsReferenceMixin("state_", "prefix"), XMLObject, CustomStorageMatr
       possible_transition_list.append(tr_path)
       self.setCategoryList(possible_transition_list)
 
+  security.declareProtected(Permissions.AccessContentsInformation,
+                            'getDestinationIdList')
+  def getDestinationIdList(self):
+    """
+    this getter is redefined to improve performance:
+    instead of getting all the transition objects from the destination list
+    to then use their ids, extract the information from the string
+    """
+    return [path.split('/')[-1] for path in self.getDestinationList()]
+
   def getTransitions(self):
     # return possible transition id list:
     return self.getDestinationIdList()

product/ERP5Workflow/Document/Transition.py

@@ -76,4 +76,4 @@ class Transition(IdAsReferenceMixin("transition_", "prefix"), XMLObject,
              PropertySheet.Transition,
              PropertySheet.Guard,
              PropertySheet.ActionInformation,
-  )
\ No newline at end of file
+  )