Workflow cleanup: avoid errors when there is no action type

5925759c · Sebastien Robin · 6a8a8fb3 · 5925759c
Commit 5925759c authored Jun 22, 2017 by Sebastien Robin
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

product/ERP5Workflow/Document/Transition.py product/ERP5Workflow/Document/Transition.py +4 -4

No files found.
--- a/product/ERP5Workflow/Document/Transition.py
+++ b/product/ERP5Workflow/Document/Transition.py
@@ -86,10 +86,10 @@ class Transition(IdAsReferenceMixin("transition_", "prefix"), XMLObject,
  security.declareProtected(Permissions.AccessContentsInformation,
                            'getActionType')
  def getActionType(self):
-    prefix_length = len('action_type/')
+    for path in self.getCategoryList():
-    action_type_list = [path[prefix_length:] for path in self.getCategoryList()
+      if path.startswith('action_type/'):
-                        if path.startswith('action_type/')]
+        return path[12:] # 12 is len('action_type/')
-    return action_type_list[0]
+    return None
  security.declareProtected(Permissions.AccessContentsInformation,
                            'getBeforeScriptList')