From b4ad8c9427f72bbe8541267499f49ab97321e485 Mon Sep 17 00:00:00 2001
From: Vincent Pelletier <vincent@nexedi.com>
Date: Wed, 16 Mar 2011 13:06:33 +0000
Subject: [PATCH] Cleanup script.

Get rid of dead (commented-out) code.
Reindent & factorise remaining code.
Add an XXX about bad script naming.


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@44340 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 ...obab_getPossibleUserAssignmentSiteList.xml | 42 ++++++++-----------
 bt5/erp5_banking_core/bt/revision             |  2 +-
 2 files changed, 19 insertions(+), 25 deletions(-)

diff --git a/bt5/erp5_banking_core/SkinTemplateItem/portal_skins/erp5_banking_core/Baobab_getPossibleUserAssignmentSiteList.xml b/bt5/erp5_banking_core/SkinTemplateItem/portal_skins/erp5_banking_core/Baobab_getPossibleUserAssignmentSiteList.xml
index e8e231017a..1c22b8ec73 100644
--- a/bt5/erp5_banking_core/SkinTemplateItem/portal_skins/erp5_banking_core/Baobab_getPossibleUserAssignmentSiteList.xml
+++ b/bt5/erp5_banking_core/SkinTemplateItem/portal_skins/erp5_banking_core/Baobab_getPossibleUserAssignmentSiteList.xml
@@ -54,33 +54,26 @@
         <item>
             <key> <string>_body</string> </key>
             <value> <string># return site of the user and possible counter on wich user can be assigned\n
-if context.getDestination() is None:\n
+# XXX: this script should be named "Assignment_getPossibleSiteList", as it is not used anywhere else\n
+destination_value = context.getDestinationValue()\n
+if destination_value is None:\n
   # must have an organisation defined to limit site diplayed\n
   return [[\'\', \'\']]\n
 \n
-#dest_site_title = context.getDestinationValue().getSiteTitle()\n
-\n
-site_list = context.Delivery_getVaultItemList(user_site=0, vault_type=("site", "site/surface/banque_interne/guichet", "site/surface/gros_paiement/guichet", "site/surface/gros_versement/guichet", "site/surface/operations_diverses/guichet"), first_level=1, strict_membership=1, leaf_node=0, base_site=context.getDestinationValue().getSite())\n
-# now must filter again this list to remove useless categories\n
-#site_dict = {}\n
-#for site in site_list:\n
-#  site_dict[site[0]] = site[1]\n
-#  site_split = site[0].split(\'/\')\n
-#  if dest_site_title in site_split:\n
-#    if len(site_split) == 4:\n
-#      # case of counter\n
-#      counter = "/".join(site_split[:-1])\n
-#      if not site_dict.has_key(counter):\n
-#        if not \'siege\' in site[1]:\n
-#          site_dict[counter] = \'/\'.join(site[1].split("/")[:7])\n
-#        else:\n
-#          site_dict[counter] = \'/\'.join(site[1].split("/")[:6])\n
-#    else:\n
-#      site_dict[site[0]] = site[1]\n
-\n
-#list = site_dict.items()\n
-#list.sort()\n
-#return list\n
+site_list = context.Delivery_getVaultItemList(\n
+  user_site=0,\n
+  vault_type=(\n
+    \'site\',\n
+    \'site/surface/banque_interne/guichet\',\n
+    \'site/surface/gros_paiement/guichet\',\n
+    \'site/surface/gros_versement/guichet\',\n
+    \'site/surface/operations_diverses/guichet\',\n
+  ),\n
+  first_level=1,\n
+  strict_membership=1,\n
+  leaf_node=0,\n
+  base_site=destination_value.getSite(),\n
+)\n
 return site_list\n
 </string> </value>
         </item>
@@ -120,6 +113,7 @@ return site_list\n
                           <tuple>
                             <string>_getattr_</string>
                             <string>context</string>
+                            <string>destination_value</string>
                             <string>None</string>
                             <string>site_list</string>
                           </tuple>
diff --git a/bt5/erp5_banking_core/bt/revision b/bt5/erp5_banking_core/bt/revision
index 55a318f19d..f5e47edc5f 100644
--- a/bt5/erp5_banking_core/bt/revision
+++ b/bt5/erp5_banking_core/bt/revision
@@ -1 +1 @@
-551
\ No newline at end of file
+552
-- 
2.30.9