run lint and format

Author: Manas-Kenge

backend/src/workflow/manager/actionmanager.py

@@ -1,17 +1,20 @@
-from zope.component import queryUtility
-
-from plone.memoize.instance import memoize
-from plone.contentrules.engine.interfaces import IRuleStorage, IRuleAssignmentManager
 from plone.app.contentrules.conditions.wftransition import WorkflowTransitionCondition
+from plone.app.contentrules.rule import get_assignments
+from plone.app.contentrules.rule import Rule
 from plone.contentrules.engine import utils
-from plone.app.contentrules.rule import Rule, get_assignments
 from plone.contentrules.engine.assignments import RuleAssignment
+from plone.contentrules.engine.interfaces import IRuleAssignmentManager
+from plone.contentrules.engine.interfaces import IRuleStorage
+from plone.memoize.instance import memoize
 from Products.CMFCore.interfaces._events import IActionSucceededEvent
 from Products.CMFCore.utils import getToolByName
-from workflow.manager.utils import generateRuleName, generateRuleNameOld
-
+from workflow.manager.utils import generateRuleName
+from workflow.manager.utils import generateRuleNameOld
+from zope.component import queryUtility
 from zope.i18nmessageid import MessageFactory
-_ = MessageFactory("plone")  
+
+
+_ = MessageFactory("plone")
 
 
 class RuleAdapter:
@@ -24,7 +27,7 @@ def __init__(self, rule, transition):
     @property
     @memoize
     def portal(self):
-        return getToolByName(self.transition, 'portal_url').getPortalObject()
+        return getToolByName(self.transition, "portal_url").getPortalObject()
 
     def activate(self):
         """
@@ -37,14 +40,12 @@ def activate(self):
         self.rule.event = IActionSucceededEvent
 
         assignable = IRuleAssignmentManager(self.portal)
-        path = '/'.join(self.portal.getPhysicalPath())
+        path = "/".join(self.portal.getPhysicalPath())
         assignable[self.rule.__name__] = RuleAssignment(
-            self.rule.id,
-            enabled=True, 
-            bubbles=True
+            self.rule.id, enabled=True, bubbles=True
         )
         assignments = get_assignments(self.rule)
-        if path not in assignments: 
+        if path not in assignments:
             assignments.insert(path)
 
     @property
@@ -58,7 +59,10 @@ def action_index(self, action):
         return self.rule.actions.index(action)
 
     def action_url(self, action):
-        return f'{self.portal.absolute_url()}/{self.rule.id}/++action++{self.action_index(action)}/edit'
+        return (
+            f"{self.portal.absolute_url()}/"
+            f"{self.rule.id}/++action++{self.action_index(action)}/edit"
+        )
 
     def delete_action(self, index):
         self.rule.actions.remove(self.rule.actions[index])
@@ -76,7 +80,7 @@ def get_rule(self, transition):
         rulename_old = generateRuleNameOld(transition)
         if self.storage is not None:
             for rule in self.storage.values():
-                if rule.__name__ in (rulename, rulename_old): 
+                if rule.__name__ in (rulename, rulename_old):
                     return RuleAdapter(rule, transition)
         return None
 
@@ -85,14 +89,14 @@ def create(self, transition):
         if rule is None:
             rule_id = generateRuleName(transition)
             r = Rule()
-            r.title = _("%s transition content rule") % transition.id 
+            r.title = _("%s transition content rule") % transition.id
             r.description = _(
                 "This content rule was automatically created by "
                 "the workflow manager to create actions on "
                 "workflow events. If you want the behavior to "
                 "work as expected, do not modify this out of "
                 "the workflow manager."
-            )  
+            )
             self.storage[rule_id] = r
             rule = RuleAdapter(r, transition)
             rule.activate()
@@ -112,4 +116,4 @@ def available_actions(self):
     def delete_rule_for(self, transition):
         rule = self.get_rule(transition)
         if rule is not None:
-            del self.storage[rule.rule.__name__]
+            del self.storage[rule.rule.__name__]

backend/src/workflow/manager/api/configure.zcml

@@ -2,11 +2,12 @@
     xmlns="http://namespaces.zope.org/zope"
     xmlns:browser="http://namespaces.zope.org/browser"
     xmlns:plone="http://namespaces.plone.org/plone"
-    i18n_domain="workflow.manager">
+    i18n_domain="workflow.manager"
+    >
 
   <!-- -*- extra stuff goes here -*- -->
 
   <include package=".services" />
 
 
-</configure>
+</configure>

backend/src/workflow/manager/api/services/configure.zcml

@@ -2,11 +2,12 @@
     xmlns="http://namespaces.zope.org/zope"
     xmlns:browser="http://namespaces.zope.org/browser"
     xmlns:plone="http://namespaces.plone.org/plone"
-    i18n_domain="workflow.manager">
+    i18n_domain="workflow.manager"
+    >
 
   <!-- -*- extra stuff goes here -*- -->
-<include package=".workflow" />
+  <include package=".workflow" />
 
 
 
-</configure>
+</configure>

backend/src/workflow/manager/api/services/workflow/base.py

@@ -1,23 +1,22 @@
 from AccessControl import Unauthorized
-from Products.CMFCore.utils import getToolByName
 from plone.memoize.view import memoize
 from plone.protect.interfaces import IDisableCSRFProtection
+from Products.CMFCore.utils import getToolByName
 from workflow.manager.actionmanager import ActionManager
-from workflow.manager.permissions import (
-    allowed_guard_permissions,
-    managed_permissions,
-)
+from workflow.manager.permissions import allowed_guard_permissions
+from workflow.manager.permissions import managed_permissions
 from zope.component import getMultiAdapter
+from zope.component import getUtility
 from zope.interface import alsoProvides
 from zope.schema.interfaces import IVocabularyFactory
-from zope.component import getUtility
-import json
 
 
 class Base:
     """A stateless helper for workflow API services."""
 
-    def __init__(self, context, request, workflow_id=None, state_id=None, transition_id=None):
+    def __init__(
+        self, context, request, workflow_id=None, state_id=None, transition_id=None
+    ):
         """
         Initialize with context, request, and specific IDs from the service.
         """
@@ -56,7 +55,11 @@ def selected_workflow(self):
     def selected_state(self):
         """Gets the state object based on the IDs provided at initialization."""
         workflow = self.selected_workflow
-        if workflow and self._state_id and self._state_id in workflow.states.objectIds():
+        if (
+            workflow
+            and self._state_id
+            and self._state_id in workflow.states.objectIds()
+        ):
             return workflow.states.get(self._state_id)
         return None
 
@@ -65,7 +68,11 @@ def selected_state(self):
     def selected_transition(self):
         """Gets the transition object based on the IDs provided at initialization."""
         workflow = self.selected_workflow
-        if workflow and self._transition_id and self._transition_id in workflow.transitions.objectIds():
+        if (
+            workflow
+            and self._transition_id
+            and self._transition_id in workflow.transitions.objectIds()
+        ):
             return workflow.transitions.get(self._transition_id)
         return None
 
@@ -135,30 +142,36 @@ def validate_csrf_token(self, token):
                 (self.context, self.request), name="authenticator"
             )
             # Temporarily set the token in the request
-            original_token = self.request.form.get('_authenticator')
-            self.request.form['_authenticator'] = token
-            
+            original_token = self.request.form.get("_authenticator")
+            self.request.form["_authenticator"] = token
+
             result = authenticator.verify()
-            
+
             # Restore original token
             if original_token is not None:
-                self.request.form['_authenticator'] = original_token
+                self.request.form["_authenticator"] = original_token
             else:
-                self.request.form.pop('_authenticator', None)
-                
+                self.request.form.pop("_authenticator", None)
+
             return result
         except Exception:
             return False
 
     def get_state(self, state_id):
         """Safely gets a state by its ID from the selected workflow."""
-        if self.selected_workflow and state_id in self.selected_workflow.states.objectIds():
+        if (
+            self.selected_workflow
+            and state_id in self.selected_workflow.states.objectIds()
+        ):
             return self.selected_workflow.states[state_id]
         return None
 
     def get_transition(self, transition_id):
         """Safely gets a transition by its ID from the selected workflow."""
-        if self.selected_workflow and transition_id in self.selected_workflow.transitions.objectIds():
+        if (
+            self.selected_workflow
+            and transition_id in self.selected_workflow.transitions.objectIds()
+        ):
             return self.selected_workflow.transitions[transition_id]
         return None
 
@@ -182,11 +195,12 @@ def allowed_guard_permissions(self):
     @memoize
     def assignable_types(self):
         """Returns a list of all user-friendly content types."""
-        vocab_factory = getUtility(IVocabularyFactory,
-            name="plone.app.vocabularies.ReallyUserFriendlyTypes")
+        vocab_factory = getUtility(
+            IVocabularyFactory, name="plone.app.vocabularies.ReallyUserFriendlyTypes"
+        )
         return sorted(
             [{"id": v.value, "title": v.title} for v in vocab_factory(self.context)],
-            key=lambda v: v['title']
+            key=lambda v: v["title"],
         )
 
     def get_assigned_types_for(self, workflow_id):
@@ -199,11 +213,11 @@ def get_assigned_types_for(self, workflow_id):
 
     def getGroups(self):
         """Gets a list of all groups in the portal."""
-        acl_users = getToolByName(self.context, 'acl_users')
+        acl_users = getToolByName(self.context, "acl_users")
         return sorted(
             [
-                {'id': g.getId(), 'title': g.getProperty('title') or g.getId()}
+                {"id": g.getId(), "title": g.getProperty("title") or g.getId()}
                 for g in acl_users.source_groups.getGroups()
             ],
-            key=lambda g: g['title'].lower()
+            key=lambda g: g["title"].lower(),
         )