That script was responsible for setting [None] as site list on ingested...

That script was responsible for setting [None] as site list on ingested documents. Refactor it a little to do costly things inside the cached method

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

bt5/erp5_base/SkinTemplateItem/portal_skins/erp5_base/ERP5Site_getPersonCareerDict.xml

@@ -56,29 +56,27 @@
 \n
 from Products.ERP5Type.Cache import CachingMethod\n
 \n
-portal = context.getPortalObject()\n
-result = {\'group_list\': [],\n
-          \'site_list\': [],\n
-          }\n
-person = portal.ERP5Site_getAuthenticatedMemberPersonValue(user_name=user_name)\n
-if person is None:\n
-  return result\n
+def getPersonCareerDict(user_name):\n
+  portal = context.getPortalObject()\n
+  result = {\'group_list\': [],\n
+            \'site_list\': [],\n
+            }\n
+  person = portal.ERP5Site_getAuthenticatedMemberPersonValue(user_name=user_name)\n
+  if person is None:\n
+    return result\n
 \n
-def getPersonCareerDict(username):\n
-  person = portal.restrictedTraverse(person_relative_url)\n
   group = person.getGroup()\n
   organisation = person.getSubordinationValue()\n
   if group is not None:\n
-    result[\'group_list\']=[group]\n
+    result[\'group_list\'] = [group]\n
   if organisation is not None:\n
-    result[\'site_list\']=[organisation.getSite()]\n
+    result[\'site_list\'] = organisation.getSiteList()\n
   return result\n
 \n
-person_relative_url = person.getRelativeUrl()\n
 getPersonCareerDict = CachingMethod(getPersonCareerDict,\n
                                     ("ERP5Site_getPersonCareerDict",),\n
-                                    cache_factory=\'erp5_ui_short\')\n
-return getPersonCareerDict(person_relative_url)\n
+                                    cache_factory=\'erp5_content_short\')\n
+return getPersonCareerDict(user_name)\n
 </string> </value>
         </item>
         <item>

bt5/erp5_base/bt/revision

-965
+966
\ No newline at end of file