[Checkins] SVN: Products.CMFDefault/branches/charlie_formlib_for_folders/Products/CMFDefault/browser/ Identifying selected items corrected.

Charlie Clark charlie at begeistert.org
Mon Aug 10 13:48:13 EDT 2009


Log message for revision 102632:
  Identifying selected items corrected.

Changed:
  U   Products.CMFDefault/branches/charlie_formlib_for_folders/Products/CMFDefault/browser/new_folder.py
  U   Products.CMFDefault/branches/charlie_formlib_for_folders/Products/CMFDefault/browser/tests/test_new_folder.py

-=-
Modified: Products.CMFDefault/branches/charlie_formlib_for_folders/Products/CMFDefault/browser/new_folder.py
===================================================================
--- Products.CMFDefault/branches/charlie_formlib_for_folders/Products/CMFDefault/browser/new_folder.py	2009-08-10 17:22:44 UTC (rev 102631)
+++ Products.CMFDefault/branches/charlie_formlib_for_folders/Products/CMFDefault/browser/new_folder.py	2009-08-10 17:48:12 UTC (rev 102632)
@@ -122,7 +122,7 @@
     @memoize
     @decode
     def listBatchItems(self):
-        batch_obj = self._get_batch_obj()
+        batch_obj = self._getBatchObj()
         portal_url = self._getPortalURL()
 
         items = []
@@ -405,7 +405,9 @@
                 
     def _get_ids(self, data):
         """Identify objects that have been selected"""
-        ids = [k.split(".")[0] for k, v in data.items() if v == True]
+        ids = [k.split(".")[0] for k, v in data.items() 
+                            if v is True \
+                            and k.split(".")[-1] == 'select']
         return ids
 
     def _get_hidden_vars(self):

Modified: Products.CMFDefault/branches/charlie_formlib_for_folders/Products/CMFDefault/browser/tests/test_new_folder.py
===================================================================
--- Products.CMFDefault/branches/charlie_formlib_for_folders/Products/CMFDefault/browser/tests/test_new_folder.py	2009-08-10 17:22:44 UTC (rev 102631)
+++ Products.CMFDefault/branches/charlie_formlib_for_folders/Products/CMFDefault/browser/tests/test_new_folder.py	2009-08-10 17:48:12 UTC (rev 102632)
@@ -96,6 +96,11 @@
                                        'DummyItem2.select':False,
                                        'DummyItem3.select':True}),
                         ['DummyItem1', 'DummyItem3'])
+        self.assertEquals(
+                        view._get_ids({'delta':True,
+                                       'delta':1}),
+                        []
+                        )
         
     def _make_one(self, name="DummyItem"):
         content = DummyContent(name)



More information about the Checkins mailing list