* revise some docstrings.

* use list comprehension instead of map().
* remove needress import of DomainSelection from PlanningBox.py.


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

product/ERP5Form/PlanningBox.py

@@ -54,7 +54,7 @@ from DateTime import DateTime
 from Products.Formulator import Widget, Validator
 from Products.Formulator.Errors import FormValidationError, ValidationError
 from SelectionTool import makeTreeList,TreeListLine
-from Selection import Selection, DomainSelection
+from Selection import Selection
 import OFS
 from AccessControl import ClassSecurityInfo
 from zLOG import LOG

product/ERP5Form/SelectionTool.py

@@ -251,7 +251,8 @@ class SelectionTool( UniqueObject, SimpleItem ):
 
     def updateSelectionCheckedUidList(self, selection_name, listbox_uid, uids, REQUEST=None):
       """
-        Updates the checked uids for a given selection_name
+        Updates the unchecked uids(listbox_uids) and checked uids (uids)
+        for a given selection_name
       """
       if listbox_uid is None:
         listbox_uid = []
@@ -425,7 +426,8 @@ class SelectionTool( UniqueObject, SimpleItem ):
     security.declareProtected(ERP5Permissions.View, 'getSelectionColumns')
     def getSelectionColumns(self, selection_name, columns=None, REQUEST=None):
       """
-        Returns the columns in the selection
+        Returns the columns in the selection if not empty, otherwise
+        returns the value of columns argument
       """
       if columns is None: columns = []
       selection = self.getSelectionFor(selection_name, REQUEST=REQUEST)
@@ -906,7 +908,7 @@ class SelectionTool( UniqueObject, SimpleItem ):
       """
         Get the list of values checked or selected for 'selection_name'
       """
-      return map(lambda x:x.getObject().getUid(), self.getSelectionValueList(selection_name, REQUEST=REQUEST, selection_method=selection_method, context=context))
+      return [x.getObject().getUid() for x in self.getSelectionValueList(selection_name, REQUEST=REQUEST, selection_method=selection_method, context=context)]
 
     security.declareProtected(ERP5Permissions.View, 'selectionHasChanged')
     def selectionHasChanged(self, md5_string, object_uid_list):