continue_recursivity variable was just unneeded, difficult to read trickery

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@35972 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/ERP5/Document/SupplyChain.py

@@ -188,9 +188,12 @@ class SupplyChain(Path, XMLObject):
       previous_link_list = self.getPreviousSupplyLinkList(current_supply_link)
       # Test each link
       for previous_link in previous_link_list:
-        continue_recursivity = False
-        # Great, we find a valid one
-        if previous_link.isProductionSupplyLink():
+        if not previous_link.isProductionSupplyLink():
+          # current is invalid
+          if not recursive:
+            continue
+        else:
+          # Great, we found a valid one
           transformation_link_list.append(previous_link)
           # Prevent infinite loop when 2 production_link have the same
           # destination
@@ -200,13 +203,9 @@ class SupplyChain(Path, XMLObject):
                   "Those SupplyLinks are in conflict: %r and %r" %\
                   (current_supply_link.getRelativeUrl(),\
                    previous_link.getRelativeUrl())
-          if all:
-            continue_recursivity = True
-        # Reject the current
-        elif recursive:
-          continue_recursivity = True
-        # Continue to browse the chain ?
-        if continue_recursivity:
+          if not recursive and not all:
+            continue
+
         # Browse the previous link
         transformation_link_list.extend(
           self.getPreviousProductionSupplyLinkList(