[Erp5-report] r38251 kazuhiko - /erp5/trunk/products/ERP5Form/Tool/SelectionTool.py

[nobody at svn.erp5.org]

Author: kazuhiko
Date: Fri Sep 10 05:37:24 2010
New Revision: 38251

URL: http://svn.erp5.org?rev=38251&view=rev
Log:
several fixes.
* specify path in setCookie() to avoid sending several duplicated cookies.
* use the normal storage if anonymous storage is not specified.

Modified:
    erp5/trunk/products/ERP5Form/Tool/SelectionTool.py

Modified: erp5/trunk/products/ERP5Form/Tool/SelectionTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Form/Tool/SelectionTool.py?rev=38251&r1=38250&r2=38251&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Form/Tool/SelectionTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5Form/Tool/SelectionTool.py [utf8] Fri Sep 10 05:37:24 2010
@@ -280,9 +280,10 @@ class SelectionTool( BaseTool, SimpleIte
       """
       if not selection_name:
         return
-      if self._isAnonymous():
-        self.REQUEST.response.setCookie('anonymous_uid',
-                                        self.REQUEST.get('anonymous_uid'))
+      anonymous_uid = REQUEST and REQUEST.get('anonymous_uid', None)
+      if anonymous_uid is not None:
+        self.REQUEST.response.setCookie('anonymous_uid', anonymous_uid,
+                                        path='/')
       if selection_object != None:
         # Set the name so that this selection itself can get its own name.
         selection_object.edit(name=selection_name)
@@ -1464,7 +1465,7 @@ class SelectionTool( BaseTool, SimpleIte
     def _getContainer(self):
       if self._isAnonymous():
         container_id = '_v_anonymous_selection_container'
-        storage = self.getAnonymousStorage()
+        storage = self.getAnonymousStorage() or self.getStorage()
       else:
         container_id = '_v_selection_container'
         storage = self.getStorage()