it makes no sense to calculate portal_url outside the cached method and use it...

it makes no sense to calculate portal_url outside the cached method and use it inside the cached method, by moving this calculation the performance might comes back to a better result

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

product/ERP5Type/patches/DCWorkflow.py

@@ -115,8 +115,6 @@ def DCWorkflowDefinition_listGlobalActions(self, info):
       return None  # Optimization
     workflow_tool = getToolByName(self, 'portal_workflow')
     workflow = getattr(workflow_tool, self.id)
-    portal_url = getToolByName(self, 'portal_url')
-    portal_url = portal_url()
     _getPortalTypeListForWorkflow = CachingMethod(workflow.getPortalTypeListForWorkflow,
                                                   id=('_getPortalTypeListForWorkflow', self.id), 
                                                   cache_factory = 'erp5_ui_long')
@@ -125,6 +123,8 @@ def DCWorkflowDefinition_listGlobalActions(self, info):
       return None
 
     def _listGlobalActions(user=None, id=None, portal_path=None):
+      portal_url = getToolByName(self, 'portal_url')
+      portal_url = portal_url()
       sm = getSecurityManager()
       portal = self._getPortalRoot()
       res = []