When doing the logical AND on all workflow defined permissions, "do not...

When doing the logical AND on all workflow defined permissions, "do not acquire" should be the default, even if the current workflow state does not define roles.


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

product/ERP5Type/patches/DCWorkflow.py

@@ -272,10 +272,10 @@ def updateRoleMappingsFor(self, ob):
             # in each DCWorkflow
             for other_workflow,other_sdef in other_data_list:
               if p in other_workflow.permissions:
-                for role in roles:
                 other_roles = other_sdef.permission_roles.get(p, [])
                 if type(other_roles) is type(()) :
                   role_type = 'tuple'
+                for role in roles:
                   if role not in other_roles :
                     refused_roles.append(role)
             for role in refused_roles :