Make erp5_core pass coding style test

/reviewed-on nexedi/erp5!996

bt5/erp5_core_proxy_field_legacy/SkinTemplateItem/portal_skins/erp5_core_proxy_field_legacy/Base_viewFieldLibrary/planning_box.xml

@@ -540,7 +540,7 @@
                 </item>
                 <item>
                     <key> <string>round_script</string> </key>
-                    <value> <string>Planning_roundBoundToDay</string> </value>
+                    <value> <string>Base_roundPlanningBoxBoundToDay</string> </value>
                 </item>
                 <item>
                     <key> <string>sec_axis_script</string> </key>

bt5/erp5_mrp/SkinTemplateItem/portal_skins/erp5_mrp/ProductionOrderModule_viewPlanning/planning_box.xml

@@ -127,7 +127,7 @@
                 </item>
                 <item>
                     <key> <string>round_script</string> </key>
-                    <value> <string>Planning_roundBoundToMinute</string> </value>
+                    <value> <string>Base_roundPlanningBoxBoundToMinute</string> </value>
                 </item>
                 <item>
                     <key> <string>selection_name</string> </key>

bt5/erp5_mrp/SkinTemplateItem/portal_skins/erp5_mrp/ProductionOrder_viewPlanning/planning_box.xml

@@ -115,7 +115,7 @@
                 </item>
                 <item>
                     <key> <string>round_script</string> </key>
-                    <value> <string>Planning_roundBoundToMinute</string> </value>
+                    <value> <string>Base_roundPlanningBoxBoundToMinute</string> </value>
                 </item>
                 <item>
                     <key> <string>selection_name</string> </key>

bt5/erp5_project/SkinTemplateItem/portal_skins/erp5_project/TaskModule_viewPlanningReport/planning_box.xml

@@ -134,7 +134,7 @@
                 </item>
                 <item>
                     <key> <string>round_script</string> </key>
-                    <value> <string>Planning_roundBoundToMinute</string> </value>
+                    <value> <string>Base_roundPlanningBoxBoundToMinute</string> </value>
                 </item>
                 <item>
                     <key> <string>selection_name</string> </key>

bt5/erp5_project/SkinTemplateItem/portal_skins/erp5_project/TaskReportModule_viewPlanningReport/planning_box.xml

@@ -131,7 +131,7 @@
                 </item>
                 <item>
                     <key> <string>round_script</string> </key>
-                    <value> <string>Planning_roundBoundToMinute</string> </value>
+                    <value> <string>Base_roundPlanningBoxBoundToMinute</string> </value>
                 </item>
                 <item>
                     <key> <string>selection_name</string> </key>

bt5/erp5_simulation/SkinTemplateItem/portal_skins/erp5_simulation/Delivery_viewSolveDivergenceDialog/your_dialog_updated.xml

@@ -279,7 +279,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_simulation/SkinTemplateItem/portal_skins/erp5_simulation/GenericSolver_viewConfigurationFormBox/my_tested_property_list.xml

@@ -189,7 +189,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_simulation/SkinTemplateItem/portal_skins/erp5_simulation/ItemListSplitSolver_viewConfigurationFormBox/my_start_date.xml

@@ -146,7 +146,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_simulation/SkinTemplateItem/portal_skins/erp5_simulation/ItemListSplitSolver_viewConfigurationFormBox/my_stop_date.xml

@@ -146,7 +146,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_simulation/SkinTemplateItem/portal_skins/erp5_simulation/QuantitySplitMoveSolver_viewConfigurationFormBox/my_delivery_solver.xml

@@ -174,7 +174,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_simulation/SkinTemplateItem/portal_skins/erp5_simulation/QuantitySplitMoveSolver_viewConfigurationFormBox/my_delivery_url.xml

@@ -158,7 +158,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_simulation/SkinTemplateItem/portal_skins/erp5_simulation/QuantitySplitSolver_viewConfigurationFormBox/my_delivery_solver.xml

@@ -174,7 +174,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_simulation/SkinTemplateItem/portal_skins/erp5_simulation/QuantitySplitSolver_viewConfigurationFormBox/my_start_date.xml

@@ -146,7 +146,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_simulation/SkinTemplateItem/portal_skins/erp5_simulation/QuantitySplitSolver_viewConfigurationFormBox/my_stop_date.xml

@@ -146,7 +146,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_ui_test/SkinTemplateItem/portal_skins/erp5_ui_test/BarModule_viewMultiForm2Dialog/your_email_1.xml

@@ -106,7 +106,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

bt5/erp5_ui_test/SkinTemplateItem/portal_skins/erp5_ui_test/BarModule_viewMultiForm2Dialog/your_text_1.xml

@@ -106,7 +106,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

product/ERP5/bootstrap/erp5_core/DocumentTemplateItem/portal_components/document.erp5.ScriptConstraint.py

@@ -37,7 +37,7 @@ class ScriptConstraint(ConstraintMixin):
   portal_type = 'Script Constraint'
 
   def _createConsistencyMessage(self, object_relative_url, message, mapping):
-    # XXX If I put in the right place I have TypeError: 'NoneType' object is not callable
+    # XXX If I put in the right place I have TypeError: 'NoneType' object is not callable
     from Products.ERP5Type.ConsistencyMessage import ConsistencyMessage
     return ConsistencyMessage(self,
       object_relative_url=object_relative_url,

product/ERP5/bootstrap/erp5_core/ExtensionTemplateItem/portal_components/extension.erp5.DocumentExtraction.py

@@ -107,23 +107,8 @@ def getExcerptText(context, txt, sw, tags, trail, maxlines):
   txt = txt.replace('-',' - ') # to find hyphenated occurrences
   txt = txt.replace(',',', ')
   txt = txt.replace(';','; ')
-  r = re.compile('\s+')
+  r = re.compile(r'\s+')
   txt = re.sub(r,' ',txt)
   text = ' '.join(txt.split('\n')).split(' ') # very rough tokenization
   return [p for p in generateParts(context,text,sw,tags,trail,maxlines)]
 
-
-if __name__=='__main__':
-  sw='pricing priority right acting proportion'
-  txt=' '.join([l.strip() for l in open('offer.txt').readlines()])
-
-  # configuration
-
-  tags=('<b>','</b>')
-  trail=5
-  maxlines=5
-  for p in getExcerptText(None,txt,sw,tags,trail,maxlines):
-    print p
-
-
-# vim: filetype=python syntax=python shiftwidth=2

product/ERP5/bootstrap/erp5_core/ExtensionTemplateItem/portal_components/extension.erp5.FolderWorkflowActionUtils.py

@@ -52,7 +52,6 @@ def getDocumentGroupByWorkflowStateList(self, form_id='', **kw):
             doc.getPortalTypeName())
     return get_url
 
-  request = self.REQUEST
   portal = self.getPortalObject()
   Base_translateString = portal.Base_translateString
   wf_tool = portal.portal_workflow

product/ERP5/bootstrap/erp5_core/ExtensionTemplateItem/portal_components/extension.erp5.FolderWorkflowActionUtils.xml

@@ -45,9 +45,7 @@
         <item>
             <key> <string>text_content_warning_message</string> </key>
             <value>
-              <tuple>
-                <string>W: 55,  2: Unused variable \'request\' (unused-variable)</string>
-              </tuple>
+              <tuple/>
             </value>
         </item>
         <item>

product/ERP5/bootstrap/erp5_core/ExtensionTemplateItem/portal_components/extension.erp5.InventoryBrain.py

@@ -14,7 +14,6 @@
 from Products.ZSQLCatalog.zsqlbrain import ZSQLBrain
 from Products.ERP5Type.TransactionalVariable import getTransactionalVariable
 from ZTUtils import make_query
-from Products.CMFCore.utils import getToolByName
 from Products.ERP5Type.Message import translateString
 from ComputedAttribute import ComputedAttribute
 

product/ERP5/bootstrap/erp5_core/ExtensionTemplateItem/portal_components/extension.erp5.InventoryBrain.xml

@@ -45,9 +45,7 @@
         <item>
             <key> <string>text_content_warning_message</string> </key>
             <value>
-              <tuple>
-                <string>W: 17,  0: Unused getToolByName imported from Products.CMFCore.utils (unused-import)</string>
-              </tuple>
+              <tuple/>
             </value>
         </item>
         <item>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_auto_logout/Base_getAutoLogoutSessionKey.py

-from AccessControl import getSecurityManager
 from zExceptions import Unauthorized
 if REQUEST is not None: # Cheap "do not call from URL" protection - not that the session key is secret
   raise Unauthorized

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Alarm_getReportURL.py

 from ZTUtils import make_query
 
-object = brain.getObject()
-if object is None:
+obj = brain.getObject()
+if obj is None:
   return None
 
 url = context.absolute_url()
 method = context.getReportMethodId('Alarm_viewReport')
-kw = { 'active_process' : 'portal_activities/%s' % object.id,
+kw = { 'active_process' : 'portal_activities/%s' % obj.id,
        'reset' : '1', 
      }
 

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_callDialogMethod.py

@@ -104,7 +104,6 @@ except FormValidationError, validation_errors:
 can_redirect = 1
 MARKER = [] # A recognisable default value. Use with 'is', not '=='.
 listbox_id_list = [] # There should not be more than one listbox - but this give us a way to check.
-file_id_list = [] # For uploaded files.
 for field in form.get_fields():
   k = field.id
   v = request.get(k, MARKER)
@@ -151,7 +150,7 @@ if len(listbox_id_list):
 if hasattr(kw, 'previous_md5_object_uid_list'):
   selection_list = context.portal_selections.callSelectionFor(kw['list_selection_name'], context=context)
   if selection_list is not None:
-    object_uid_list = map(lambda x:x.getObject().getUid(), selection_list)
+    object_uid_list = [x.getObject().getUid() for x in selection_list]
     error = context.portal_selections.selectionHasChanged(kw['previous_md5_object_uid_list'], object_uid_list)
     if error:
       error_message = context.Base_translateString("Sorry, your selection has changed.")
@@ -167,7 +166,7 @@ listbox_uid = kw.get('listbox_uid', None)
 # In such cases, we must not try to update a non-existing selection.
 if listbox_uid is not None and kw.has_key('list_selection_name'):
   uids = kw.get('uids')
-  selected_uids = context.portal_selections.updateSelectionCheckedUidList(
+  context.portal_selections.updateSelectionCheckedUidList(
     kw['list_selection_name'],
     listbox_uid, uids)
 # Remove values which doesn't work with make_query.

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_configureSortOn.py

-# Updates attributes of an Zope document
-# which is in a class inheriting from ERP5 Base
-
-
-from Products.Formulator.Errors import ValidationError, FormValidationError
+from Products.Formulator.Errors import FormValidationError
 
 request = context.REQUEST
 field_sort_type = request.form.get('field_sort_type', None)

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_configureUI.py

-# Updates attributes of an Zope document
-# which is in a class inheriting from ERP5 Base
-
-
-from Products.Formulator.Errors import ValidationError, FormValidationError
-
+from Products.Formulator.Errors import FormValidationError
 
 request=context.REQUEST
 

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_convertDateListToChartList.py

-# This scripts allows to update a list so that it
-# can be displayed correctly in a graph
-# The list given have to be of the forme:
-#  list = [[Datetime(),value (,value)*],([Datetime(),value (,value)*])*]
-
-list.sort()
-
-# Check if there is any none value, and replace it by 0
-
-formated_list = []
-if len(list) >= 1:
-  for i in range(len(list)):
-    for index_value in range(1,len(list[0])):
-      if list[i][index_value]==None:
-        list[i][index_value]=0
-  formated_list.append(list[0])
-  for i in range(1,len(list)):
-    nb_days = int(list[i][0]-list[i-1][0])
-    for day in range(1,nb_days):
-      formated_list.append([list[i-1][0]+day])
-      for nb_value in range(1,len(list[i-1])):
-        formated_list[len(formated_list)-1].append(list[i-1][nb_value])
-    formated_list.append(list[i])
-
-return formated_list

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_convertDateListToChartList.xml

-<?xml version="1.0"?>
-<ZopeData>
-  <record id="1" aka="AAAAAAAAAAE=">
-    <pickle>
-      <global name="PythonScript" module="Products.PythonScripts.PythonScript"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>Script_magic</string> </key>
-            <value> <int>3</int> </value>
-        </item>
-        <item>
-            <key> <string>_bind_names</string> </key>
-            <value>
-              <object>
-                <klass>
-                  <global name="NameAssignments" module="Shared.DC.Scripts.Bindings"/>
-                </klass>
-                <tuple/>
-                <state>
-                  <dictionary>
-                    <item>
-                        <key> <string>_asgns</string> </key>
-                        <value>
-                          <dictionary>
-                            <item>
-                                <key> <string>name_container</string> </key>
-                                <value> <string>container</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_context</string> </key>
-                                <value> <string>context</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_m_self</string> </key>
-                                <value> <string>script</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_subpath</string> </key>
-                                <value> <string>traverse_subpath</string> </value>
-                            </item>
-                          </dictionary>
-                        </value>
-                    </item>
-                  </dictionary>
-                </state>
-              </object>
-            </value>
-        </item>
-        <item>
-            <key> <string>_params</string> </key>
-            <value> <string>list=[]</string> </value>
-        </item>
-        <item>
-            <key> <string>id</string> </key>
-            <value> <string>Base_convertDateListToChartList</string> </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
-</ZopeData>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_convertVariationListToQuantityList.py

-# This scripts allows to update a list so that it
-# converts variation on a quantity to the quantity itself
-# ie, if list=[[DateTime('09/10/2003'),+4],[DateTime('09/19/2003'),-8]], 
-# and initial_quantity = [3]
-# result: [[DateTime('2003/09/10'), 7], [DateTime('2003/09/19'), -1]]
-# The list given have to be of the forme:
-#  list = [[Datetime(),value (,value)*],([Datetime(),value (,value)*])*]
-# The initial_quantity have to be like this :
-#  initial_quantity = [value (,value)*]
-
-
-list.sort()
-
-quantity_list = []
-#if type(initial_quantity) is type(1):
-#  initial_quantity = [initial_quantity]
-
-if len(list) >= 1 and (len(list[0])-1)==len(initial_quantity):
-  quantity_list.append([list[0][0]])
-  for i in range(1,len(list[0])):
-    if list[0][i]==None:
-      list[0][i]=0
-    quantity_list[0].append(initial_quantity[i-1] + list[0][i])
-  for value in range(1,len(list)):
-    quantity_list.append([list[value][0]])
-    for i in range(1,len(list[0])):
-      if list[value][i]==None:
-        list[value][i]=0
-      quantity_list[value].append(quantity_list[value-1][i] + list[value][i])
-
-return quantity_list

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_convertVariationListToQuantityList.xml

-<?xml version="1.0"?>
-<ZopeData>
-  <record id="1" aka="AAAAAAAAAAE=">
-    <pickle>
-      <global name="PythonScript" module="Products.PythonScripts.PythonScript"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>Script_magic</string> </key>
-            <value> <int>3</int> </value>
-        </item>
-        <item>
-            <key> <string>_bind_names</string> </key>
-            <value>
-              <object>
-                <klass>
-                  <global name="NameAssignments" module="Shared.DC.Scripts.Bindings"/>
-                </klass>
-                <tuple/>
-                <state>
-                  <dictionary>
-                    <item>
-                        <key> <string>_asgns</string> </key>
-                        <value>
-                          <dictionary>
-                            <item>
-                                <key> <string>name_container</string> </key>
-                                <value> <string>container</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_context</string> </key>
-                                <value> <string>context</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_m_self</string> </key>
-                                <value> <string>script</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_subpath</string> </key>
-                                <value> <string>traverse_subpath</string> </value>
-                            </item>
-                          </dictionary>
-                        </value>
-                    </item>
-                  </dictionary>
-                </state>
-              </object>
-            </value>
-        </item>
-        <item>
-            <key> <string>_params</string> </key>
-            <value> <string>list=[],initial_quantity=[]</string> </value>
-        </item>
-        <item>
-            <key> <string>id</string> </key>
-            <value> <string>Base_convertVariationListToQuantityList</string> </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
-</ZopeData>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_createNewDocument.py

@@ -2,7 +2,6 @@
 this type cannot be added in the container.
 """
 Base_translateString = context.Base_translateString
-REQUEST=context.REQUEST
 parent = context.getParentValue()
 allowed_type_list = parent.getVisibleAllowedContentTypeList()
 

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_createRelation.py

@@ -19,11 +19,11 @@ for v in values:
     # We should not edit the id field
     # because the object which is created is not attached to a
     # module yet
-    id = v
+    id_ = v
   else:
-    id=str(module.generateNewId())
-  module.invokeFactory(type_name=portal_type,id=id)
-  new_ob = module.get(id)
+    id_ = str(module.generateNewId())
+  module.invokeFactory(type_name=portal_type, id=id_)
+  new_ob = module.get(id_)
   kw = {}
   if catalog_index != 'id':
     kw[catalog_index] = v
@@ -34,59 +34,3 @@ for v in values:
 o.setValue(base_category, ref_list, portal_type=[portal_type])
 
 return request[ 'RESPONSE' ].redirect( return_url )
-
-
-try:
-  # Validate the form
-  form = getattr(context,form_id)
-  form.validate_all_to_request(request)
-  my_field = None
-  # Find out which field defines the relation
-  for f in form.get_fields():
-    if f.has_value( 'base_category'):
-      if f.get_value('base_category') == base_category:
-        k = f.id
-        v = getattr(request,k,None)
-        if v != context.getProperty(k[3:]):
-          my_field = f
-  if my_field:
-    kw ={}
-    kw[my_field.get_value('catalog_index')] = request.get( my_field.id, None)
-    context.portal_selections.setSelectionParamsFor('Base_viewRelatedObjectList', kw.copy())
-    kw['base_category'] = base_category
-    kw['portal_type'] = my_field.get_value('portal_type')
-    request.set('base_category', base_category)
-    request.set('portal_type', my_field.get_value('portal_type'))
-    request.set('form_id', 'Base_viewRelatedObjectList')
-    request.set(my_field.get_value('catalog_index'), request.get( my_field.id, None))
-    relation_list = context.portal_catalog(**kw)
-    if len(relation_list) > 0:
-      return context.Base_viewRelatedObjectList( REQUEST=request )
-    else:
-      request.set('catalog_index', my_field.get_value('catalog_index'))
-      request.set('relation_values', request.get( my_field.id, None))
-      return context.Base_viewCreateRelationDialog( REQUEST=request )
-      pass
-      # context.newRelation(base_category, my_field.get_value('portal_type'))
-except FormValidationError, validation_errors:
-  # Pack errors into the request
-  field_errors = form.ErrorFields(validation_errors)
-  request.set('field_errors', field_errors)
-  return form(request)
-else:
-  message = 'Relation+Unchanged.'
-
-if not selection_index:
-  redirect_url = '%s/%s?%s' % ( o.absolute_url()
-                            , form_id
-                            , 'portal_status_message=%s' % message
-                            )
-else:
-  redirect_url = '%s/%s?selection_index=%s&selection_name=%s&%s' % ( o.absolute_url()
-                            , form_id
-                            , selection_index
-                            , selection_name
-                            , 'portal_status_message=%s' % message
-                            )
-
-request[ 'RESPONSE' ].redirect( redirect_url )

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_edit.py

@@ -57,7 +57,7 @@ def editListBox(listbox_field, listbox):
   if listbox is not None:
     global_property_dict = {}
     if listbox_field.has_value('global_attributes'):
-      hidden_attribute_list = map(lambda x:x[0], listbox_field.get_value('global_attributes'))
+      hidden_attribute_list = [x[0] for x in listbox_field.get_value('global_attributes')]
       for hidden_attribute in hidden_attribute_list:
         global_property_dict[hidden_attribute] = getattr(request, hidden_attribute, None)
     for item_url, listbox_item_dict in listbox.items():
@@ -129,10 +129,10 @@ def editMatrixBox(matrixbox_field, matrixbox):
           columns = matrixbox_field.get_value('columns')
           tabs = matrixbox_field.get_value('tabs')
 
-        column_ids = map(lambda x: x[0], columns)
-        line_ids = map(lambda x: x[0], lines)
-        tab_ids = map(lambda x: x[0], tabs)
-        extra_dimension_category_list_list = [[category for category, label in dimension_list] for dimension_list in extra_dimension_list_list]
+        column_ids = [x[0] for x in columns]
+        line_ids = [x[0] for x in lines]
+        tab_ids = [x[0] for x in tabs]
+        extra_dimension_category_list_list = [[category for category, _ in dimension_list] for dimension_list in extra_dimension_list_list]
 
         # There are 3 cases
         # Case 1: we do 1 dimensional matrix
@@ -250,8 +250,6 @@ server_physical_path = context.getPhysicalPath()
 server_physical_path = list(server_physical_path)
 server_url = request["SERVER_URL"]
 server_physical_path.insert(0, server_url)
-#calculate direct the url instead of using absolute_url
-new_url = '/'.join(server_physical_path)
 
 # for web mode, we should use 'view' instead of passed form_id
 # after 'Save & View'.

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_editRelation.py

 # Updates relation of an ERP5 document
-from Products.ERP5Type.Message import Message
-from Products.Formulator.Errors import ValidationError
 from Products.ERP5Form.MultiRelationField import SUB_FIELD_ID
 
 if listbox_uid is not None:
   selection_tool = context.getPortalObject().portal_selections
-  selected_uids = selection_tool.updateSelectionCheckedUidList(
+  selection_tool.updateSelectionCheckedUidList(
               selection_name, listbox_uid, uids)
   uids = selection_tool.getSelectionCheckedUidsFor(selection_name)
 

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_generateParentDomain.py

-from Products.ERP5Type.Document import newTempDomain
 domain_list=[]
 request = context.REQUEST
-filter = {}
+filter_ = {}
 
 try:
   # first, try to get form_id from the request
@@ -12,7 +11,7 @@ except KeyError:
 
 if listbox is not None:
   portal_type_list = [x[0] for x in listbox.get_value('portal_types')]
-  filter['portal_type'] = portal_type_list
+  filter_['portal_type'] = portal_type_list
 
 if depth == 0:
   parent_obj = request['here']
@@ -22,7 +21,7 @@ else:
   parent_url = parent.getProperty('parent_url')
   parent_obj = context.restrictedTraverse(parent_url)
 
-for obj in parent_obj.contentValues(filter=filter):
+for obj in parent_obj.contentValues(filter=filter_):
   domain = parent.generateTempDomain(id='%s_%s' % (depth, obj.getId()))
   domain.edit(title=obj.getTitle(),
               parent_url=obj.getRelativeUrl(),

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_generateRandomString.py

@@ -10,4 +10,4 @@ if include_letters:
   character_set = '%s%s' %(character_set, letters)
 if include_digits:
   character_set = '%s%s' %(character_set, digits)
-return ''.join([choice(character_set) for i in range(int(string_length))])
+return ''.join([choice(character_set) for _ in range(int(string_length))])

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_generateWeekDomain.py

@@ -20,7 +20,7 @@ bound_start = DateTime(bound_start.year() , bound_start.month() , bound_start.da
 
 # Normalize Week. XXX this should be in preferences as well
 while bound_start.Day() is not 'Sunday':
-   bound_start =  bound_start - 1
+  bound_start =  bound_start - 1
 current_date =  bound_start + 7 * bound_variation
 bound_stop  = current_date + 7
 current_date =  DateTime(current_date.year() , current_date.month() , current_date.day())

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_generateYearDomain.py

@@ -21,12 +21,6 @@ current_date = DateTime(year, 1, 1)
 default_link_url ='setLanePath?form_id=%s&list_selection_name=%s' %(
                                  form_id, selection_name)
 
-# Define date format using user Preferences
-date_order = portal.portal_preferences.getPreferredDateOrder()
-date_format = dict(ymd='%m/%d',
-                   dmy='%d/%m',
-                   mdy='%m/%d').get(date_order, '%m/%d')
-
 category_list = []
 if depth == 0:
   # getting list of months
@@ -45,7 +39,7 @@ if depth == 0:
     if current_date.month() != 12:
       stop_date = DateTime(current_date.year(),current_date.month() +1,1)
     else:
-       stop_date = DateTime(year+1, 1, 1)
+      stop_date = DateTime(year+1, 1, 1)
     o.setProperty('stop', stop_date)
     
     o.setProperty('relative_position', int(current_date))

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getCategoriesSpreadSheetMapping.py

@@ -88,13 +88,12 @@ if not (content_type.startswith('application/vnd.sun.xml')
   tmp_ooo.edit(data=import_file.read(),
                content_type=content_type)
   tmp_ooo.convertToBaseFormat()
-  ignored, import_file_content = tmp_ooo.convert('ods')
+  _, import_file_content = tmp_ooo.convert('ods')
   parser.openFromString(str(import_file_content))
 else:
   parser.openFile(import_file)
 
 # Extract tables from the speadsheet file
-filename = parser.getFilename()
 spreadsheet_list = parser.getSpreadsheetsMapping(no_empty_lines=True)
 
 
@@ -128,8 +127,6 @@ for table_name in spreadsheet_list.keys():
     else:
       # If there is a new column with a header and the path definition has
       # started, that seems the path definition has ended
-      if 'path_0' in property_map.values():
-        path_index == None # FIXME: useless statement, but what was the original intention ??
       property_map[column_index] = column_id.encode('utf8')
     column_index += 1
 

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getHelpUrl.py

 if context.REQUEST.has_key('workflow_action'): # We are on a workflow transition
-  help = '%s#%s' % (getattr(getattr(context, form_id), 'form_action'),context.REQUEST['workflow_action'])
+  help_relative_url = '%s#%s' % (getattr(getattr(context, form_id), 'form_action'),context.REQUEST['workflow_action'])
 elif action is not None:
-  help = '%s#%s' % (context.getPortalTypeName(), action)
+  help_relative_url = '%s#%s' % (context.getPortalTypeName(), action)
 elif form_id is not None:
-  help = '%s_%s' % (context.getPortalTypeName(), form_id)
+  help_relative_url = '%s_%s' % (context.getPortalTypeName(), form_id)
 else:
-  help = context.getPortalTypeName()
-return '%s/%s' % (context.portal_preferences.getPreferredHtmlStyleDocumentationBaseUrl(), help)
+  help_relative_url = context.getPortalTypeName()
+return '%s/%s' % (context.portal_preferences.getPreferredHtmlStyleDocumentationBaseUrl(), help_relative_url)

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getLineListFieldPropertyDictList.py

+if default_sub_field_property_dict is None:
+  default_sub_field_property_dict = {}
+
 # Define a dictionary where we store the subfields to display.
 sub_field_dict = {}
 split_depth = 1

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getLineListFieldPropertyDictList.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>item_list, value_list, default_sub_field_property_dict={}, is_right_display=0</string> </value>
+            <value> <string>item_list, value_list, default_sub_field_property_dict=None, is_right_display=0</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getListbox.py

@@ -45,7 +45,7 @@ if listbox:
 # the listbox is there.
 for group in ('bottom', 'center', 'left', 'right'):
   for field in form.get_fields_in_group(group):
-     if (isListBox(field) and
-         not field.get_value('hidden') and
-         field.get_value('enabled')):
-        return field
+    if (isListBox(field) and
+        not field.get_value('hidden') and
+        field.get_value('enabled')):
+      return field

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getMultiListFieldPropertyDictList.py

+if default_sub_field_property_dict is None:
+  default_sub_field_property_dict = {}
+
 # Define a dictionary where we store the subfields to display.
 sub_field_dict = {}
 split_depth = 1

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getMultiListFieldPropertyDictList.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>item_list, value_list, default_sub_field_property_dict={}, is_right_display=0</string> </value>
+            <value> <string>item_list, value_list, default_sub_field_property_dict=None, is_right_display=0</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getPreferredSectionItemList.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>portal_type=[\'Organisation\'], validation_state=(\'validated\', \'draft\'), base_category=None</string> </value>
+            <value> <string>portal_type=(\'Organisation\', ), validation_state=(\'validated\', \'draft\'), base_category=None</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getRelatedDocumentList.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>portal_type= [], limit=None, **kw</string> </value>
+            <value> <string>portal_type=(), limit=None, **kw</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getSafeIdFromString.py

@@ -8,14 +8,14 @@ translation_map = { "a": ['\xe0']
                   }
 
 clean_id = ''
-if string == None:
+if s is None:
   return None
-string = string.lower()
-string = string.strip()
+s = s.lower()
+s = s.strip()
 # oocalc inserts some strange chars when you press - key in a text cell.
 # Following line is a workaround for this, because \u2013 does not exist in latin1
-string = string.replace(u'\u2013', '-')
-for char in string.encode('iso8859_1'):
+s = s.replace(u'\u2013', '-')
+for char in s.encode('iso8859_1'):
   if char == '_' or char.isalnum():
     clean_id += char
   elif char.isspace() or char in ('+', '-'):

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getSafeIdFromString.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>string=None</string> </value>
+            <value> <string>s=None</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getSearchText.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>selection=None, argument_name_list=[]</string> </value>
+            <value> <string>selection=None, argument_name_list=()</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getSourceVisibility.py

 # controls the visibility of "developper mode" links in ERP5 web interface
 # (edit form, edit field, edit portal type, ...) for object
 
-if object is None:
- object = context
+if obj is None:
+  obj = context
 
-return context.portal_preferences.getPreferredHtmlStyleDevelopperMode() and context.portal_membership.checkPermission('View management screens', object)
+return context.portal_preferences.getPreferredHtmlStyleDevelopperMode() and context.portal_membership.checkPermission('View management screens', obj)

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getSourceVisibility.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>object=None</string> </value>
+            <value> <string>obj=None</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getWorkflowHistory.py

@@ -5,7 +5,7 @@ history_name_list = ['building_history', ' installation_history', 'history']
 history = {}
 
 portal_workflow = context.getPortalObject().portal_workflow
-workflow_id_list = [workflow_id for workflow_id, workflow_state in context.getWorkflowStateItemList()]
+workflow_id_list = [x[0] for x in context.getWorkflowStateItemList()]
 
 for history_name in history_name_list:
   
@@ -16,7 +16,7 @@ for history_name in history_name_list:
       list_history_item = portal_workflow.getInfoFor(ob=context, name=history_name, wf_id=wf_id)
     except ConflictError:
       raise
-    except:
+    except Exception:
       pass
 
     if list_history_item not in ((), None):

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_getWorkflowHistoryItemList.py

-request = context.REQUEST
 from Products.ERP5Type.Document import newTempBase
-from Products.ERP5Type.Document import newTempMappedValue
 from Products.ERP5Type.Utils import getTranslationStringWithContext
 
 from AccessControl import getSecurityManager
@@ -12,7 +10,7 @@ result = []
 i = 1
 portal_object = context.getPortalObject()
 portal_workflow = portal_object.portal_workflow
-workflow_id_list = [x for x, y in context.getWorkflowStateItemList()]
+workflow_id_list = [x[0] for x in context.getWorkflowStateItemList()]
 if not workflow_id in workflow_id_list:
   return []
 
@@ -31,12 +29,8 @@ def getActorName(actor):
 
 
 # Get history
-# XXX Compatibility
-for history_name in ['history', 'building_history', 'installation_history']:
-  workflow_item_list = portal_workflow.getInfoFor(ob=context, 
-                                          name='history', wf_id=workflow_id)
-  if workflow_item_list != []:
-    break
+workflow_item_list = portal_workflow.getInfoFor(
+    ob=context, name='history', wf_id=workflow_id)
 
 wf_state_var = portal_workflow[workflow_id].variables.getStateVar()
 wf_states = portal_workflow[workflow_id].states

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_hashCategoryList.py

@@ -5,6 +5,8 @@
   in such way that it is possible to select multiple
   categories for the same document )
 """
+if default_sub_field_property_dict is None:
+  default_sub_field_property_dict = {}
 # Initialise result
 sub_field_list = []
 

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_hashCategoryList.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>item_list, value_list, default_sub_field_property_dict={}, is_right_display=0</string> </value>
+            <value> <string>item_list, value_list, default_sub_field_property_dict=None, is_right_display=0</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_jumpToRelatedDocument.py

-from ZTUtils import make_query
-
 request=context.REQUEST
 
 form = getattr(context,form_id)
 field = form.get_field(field_id)
 base_category = field.get_value('base_category')
-portal_type = map(lambda x:x[0],field.get_value('portal_type'))
+portal_type = [x[0] for x in field.get_value('portal_type')]
 kw = {}
 for k, v in field.get_value('parameter_list') :
   kw[k] = v
@@ -16,7 +14,7 @@ if len(jump_reference_list)==1:
   jump_reference = jump_reference_list[0]
   return jump_reference.Base_redirect()
 else:
-  selection_uid_list = map(lambda x:x.getUid(),jump_reference_list) or None
+  selection_uid_list = [x.getUid() for x in jump_reference_list] or None
   kw = {'uid': selection_uid_list}
   # We need to reset the selection. Indeed, some sort columns done in another 
   # jump could be meaningless for this particular jump. The consequence could 

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Planning_roundBoundToDay.xml → product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_roundPlanningBoxBoundToDay.xml

@@ -54,7 +54,7 @@
         </item>
         <item>
             <key> <string>id</string> </key>
-            <value> <string>Planning_roundBoundToDay</string> </value>
+            <value> <string>Base_roundPlanningBoxBoundToDay</string> </value>
         </item>
       </dictionary>
     </pickle>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Planning_roundBoundToInt.xml → product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_roundPlanningBoxBoundToInt.xml

@@ -54,7 +54,7 @@
         </item>
         <item>
             <key> <string>id</string> </key>
-            <value> <string>Planning_roundBoundToInt</string> </value>
+            <value> <string>Base_roundPlanningBoxBoundToInt</string> </value>
         </item>
       </dictionary>
     </pickle>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Planning_roundBoundToMinute.xml → product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_roundPlanningBoxBoundToMinute.xml

@@ -54,7 +54,7 @@
         </item>
         <item>
             <key> <string>id</string> </key>
-            <value> <string>Planning_roundBoundToMinute</string> </value>
+            <value> <string>Base_roundPlanningBoxBoundToMinute</string> </value>
         </item>
       </dictionary>
     </pickle>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_searchHandler.py

-from Products.Formulator.Errors import ValidationError, FormValidationError
-from ZTUtils import make_query
-
-request=context.REQUEST
-
-if isinstance(list_form_id,tuple):
-  list_form_id = list_form_id[0]
-# The type list is not working with isinstance, I have do do this bad hack
-if hasattr(list_form_id,'sort'):
-  list_form_id = list_form_id[0]
-
-module_name = context.getId()
-
-try:
-  # Validate the form
-  form = getattr(context,dialog_id)
-  form.validate_all_to_request(request)
-  kw = {}
-  for f in form.get_fields():
-    k = f.id
-    # XXX Remove your_ parameters...
-    k = k[5:]
-    v = getattr(request,k,None)
-    if v is not None and k != 'list_form_id' :
-      kw[k] = v
-    if list_method_id is not None and list_method_id  != '' :
-      kw['list_method_id'] = list_method_id
-  url_params_string = make_query(kw)
-#  raise str(kw), url_params_string
-
-except FormValidationError, validation_errors:
-  # Pack errors into the request
-  field_errors = form.ErrorFields(validation_errors)
-  request.set('field_errors', field_errors)
-  return form(request)
-
-
-if url_params_string != '':
-  redirect_url = '%s/%s?%s' % ( context.absolute_url()
-                            , list_form_id
-                            , url_params_string
-                            )
-else:
-  redirect_url = '%s/%s' % ( context.absolute_url()
-                            , list_form_id
-                            )
-
-
-return request.RESPONSE.redirect( redirect_url )

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_searchHandler.xml

-<?xml version="1.0"?>
-<ZopeData>
-  <record id="1" aka="AAAAAAAAAAE=">
-    <pickle>
-      <global name="PythonScript" module="Products.PythonScripts.PythonScript"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>Script_magic</string> </key>
-            <value> <int>3</int> </value>
-        </item>
-        <item>
-            <key> <string>_bind_names</string> </key>
-            <value>
-              <object>
-                <klass>
-                  <global name="NameAssignments" module="Shared.DC.Scripts.Bindings"/>
-                </klass>
-                <tuple/>
-                <state>
-                  <dictionary>
-                    <item>
-                        <key> <string>_asgns</string> </key>
-                        <value>
-                          <dictionary>
-                            <item>
-                                <key> <string>name_container</string> </key>
-                                <value> <string>container</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_context</string> </key>
-                                <value> <string>context</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_m_self</string> </key>
-                                <value> <string>script</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_subpath</string> </key>
-                                <value> <string>traverse_subpath</string> </value>
-                            </item>
-                          </dictionary>
-                        </value>
-                    </item>
-                  </dictionary>
-                </state>
-              </object>
-            </value>
-        </item>
-        <item>
-            <key> <string>_params</string> </key>
-            <value> <string>form_id,dialog_id,list_form_id,list_method_id=\'\'</string> </value>
-        </item>
-        <item>
-            <key> <string>id</string> </key>
-            <value> <string>Base_searchHandler</string> </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
-</ZopeData>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_sortObjectListLikeSQL.py

@@ -11,6 +11,6 @@ def generic_sort(a,b):
         return result
   return result 
 
-unordered_list = map(lambda x: x.getObject(), unordered_list)
+unordered_list = [x.getObject() for x in unordered_list]
 unordered_list.sort(generic_sort)
 return unordered_list

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_sortObjectListLikeSQL.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>unordered_list=[], sort_order=()</string> </value>
+            <value> <string>unordered_list, sort_order=()</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_updateDialogForm.py

 from string import zfill
 request = context.REQUEST
 
-from string import zfill
-
 if kw.get('update', False):
   # Ensure ERP5JS correctly refresh the page
   request.RESPONSE.setStatus(400)

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/FieldValidator_requireIfDialogConfirmed.xml → product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_validateDialogConfirmation.xml

@@ -54,7 +54,7 @@
         </item>
         <item>
             <key> <string>id</string> </key>
-            <value> <string>FieldValidator_requireIfDialogConfirmed</string> </value>
+            <value> <string>Base_validateDialogConfirmation</string> </value>
         </item>
       </dictionary>
     </pickle>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/PropertySheetChild_validateIDField.py → product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_validateIDAsReferenceSuffix.py

-from Products.ERP5Type.Log import log
 portal = container.getPortalObject()
 if not(id_value.endswith(portal.restrictedTraverse(REQUEST.object_path)\
                          .getIdAsReferenceSuffix())):

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/PropertySheetChild_validateIDField.xml → product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_validateIDAsReferenceSuffix.xml

@@ -54,7 +54,7 @@
         </item>
         <item>
             <key> <string>id</string> </key>
-            <value> <string>PropertySheetChild_validateIDField</string> </value>
+            <value> <string>Base_validateIDAsReferenceSuffix</string> </value>
         </item>
       </dictionary>
     </pickle>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_validateRelation.py

-from Products.Formulator.Errors import ValidationError, FormValidationError
+from Products.Formulator.Errors import FormValidationError
 from ZTUtils import make_query
 portal = context.getPortalObject()
 Base_translateString = portal.Base_translateString
@@ -46,8 +46,6 @@ def getOrderedUids(uids, values, catalog_index):
     uids.append(value_to_uid[value])
   return uids
 
-  field.get_value('base_category')
-
 try:
   # Validate the form
   form = getattr(context,form_id)
@@ -58,20 +56,19 @@ try:
   # Find out which field defines the relation
   for f in form.get_fields():
     if f.has_value( 'base_category'):
-        #if f.get_value('base_category') == base_category:
-        k = f.id
-        v = getattr(request,k,None)
-        if v in (None, '', 'None', [], ()) and context.getProperty(k[3:]) in (None, '', 'None', [], ()):
-          # The old value is None and the new value is not significant
-          # This bug fix is probably temporary since '' means None
-          pass
-        elif v != context.getProperty(k[3:]):
-          old_value = context.getProperty(k[3:])
-          my_field = f
-          new_value = v
-          base_category = f.get_value( 'base_category')
+      #if f.get_value('base_category') == base_category:
+      k = f.id
+      v = getattr(request,k,None)
+      if v in (None, '', 'None', [], ()) and context.getProperty(k[3:]) in (None, '', 'None', [], ()):
+        # The old value is None and the new value is not significant
+        # This bug fix is probably temporary since '' means None
+        pass
+      elif v != context.getProperty(k[3:]):
+        old_value = context.getProperty(k[3:])
+        my_field = f
+        new_value = v
+        base_category = f.get_value( 'base_category')
   if my_field and base_category is not None:
-    empty_list = 0
     if new_value == '':
       new_value = []
     if same_type(new_value,'a'):
@@ -90,7 +87,7 @@ try:
       if checkSameKeys(new_value, old_value):
         # Reorder keys
         same_keys = 1
-    portal_type = map(lambda x:x[0],my_field.get_value('portal_type'))
+    portal_type = [x[0] for x in my_field.get_value('portal_type')]
     # We work with strings - ie. single values
     kw ={}
     kw[my_field.get_value('catalog_index')] = new_value
@@ -103,7 +100,7 @@ try:
     request.set('field_id', my_field.id)
     previous_uids = o.getValueUidList(base_category, portal_type=portal_type)
     relation_list = context.portal_catalog(**kw)
-    relation_uid_list = map(lambda x: x.uid, relation_list)
+    relation_uid_list = [x.uid for x in relation_list]
     uids = []
     for uid in previous_uids:
       if uid in relation_uid_list:
@@ -125,9 +122,9 @@ try:
       # If we have only one in the list, we don't want to lose our time by
       # selecting it. So we directly do the update
       if len(relation_list) == 1:
-          selection_index=None
-          uids = [relation_list[0].uid]
-          return o.Base_editRelation( form_id = form_id,
+        selection_index=None
+        uids = [relation_list[0].uid]
+        return o.Base_editRelation( form_id = form_id,
                                     field_id = my_field.id,
                                     selection_index = selection_index,
                                     selection_name = selection_name,

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_viewDict.py

@@ -13,10 +13,10 @@ from pprint import pformat
 
 ret = '<html><body><table width=100%>\n'
 
-dict = context.showDict().items()
-dict.sort()
+property_dict = context.showDict().items()
+property_dict.sort()
 i = 0
-for k,v in dict:
+for k,v in property_dict:
   if (i % 2) == 0:
     c = '#88dddd'
   else:

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_viewFieldLibrary/my_planning_box.xml

@@ -540,7 +540,7 @@
                 </item>
                 <item>
                     <key> <string>round_script</string> </key>
-                    <value> <string>Planning_roundBoundToDay</string> </value>
+                    <value> <string>Base_roundPlanningBoxBoundToDay</string> </value>
                 </item>
                 <item>
                     <key> <string>sec_axis_script</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Base_viewFieldLibrary/my_view_mode_id_as_reference.xml

@@ -116,7 +116,7 @@
       <dictionary>
         <item>
             <key> <string>method_name</string> </key>
-            <value> <string>PropertySheetChild_validateIDField</string> </value>
+            <value> <string>Base_validateIDAsReferenceSuffix</string> </value>
         </item>
       </dictionary>
     </pickle>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/BusinessTemplate_save.py

 REQUEST=context.REQUEST
-RESPONSE=REQUEST.RESPONSE
 
 return context.portal_templates.save(context, REQUEST=REQUEST)

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/CategoryTool_importCategoryFile.py

@@ -72,12 +72,10 @@ for base_category, category_list in category_list_spreadsheet_dict.iteritems():
     except ValueError:
       category_id = category_path
       container = portal_categories
-      is_base_category = True
       category_type = 'Base Category'
       category_type_property_id_set = base_category_property_id_set
     else:
       container = resolveCategory(container_path)
-      is_base_category = False
       category_type = 'Category'
       category_type_property_id_set = category_property_id_set
     try:

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/CategoryTool_importCategoryFile.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>import_file, update_existing_property=False, keep_existing_category=True, detailed_report=False, simulation_mode=False, displayed_report=[], effective_date=None, expiration_date=None, existing_category_list=\'keep\', create_local_property=False, **kw</string> </value>
+            <value> <string>import_file, update_existing_property=False, keep_existing_category=True, detailed_report=False, simulation_mode=False, displayed_report=(), effective_date=None, expiration_date=None, existing_category_list=\'keep\', create_local_property=False, **kw</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Category_getSortedCategoryChildValueList.py

@@ -2,7 +2,7 @@ from Products.ERP5Type.Log import log
 # this script is no longer needed.
 log('Category_getSortedCategoryChildValueList', 'use getCategoryChildValueList method')
 
-value_list = filter(lambda o: o.getRelativeUrl() != context.getRelativeUrl(),context.getCategoryChildValueList())
+value_list = [o for o in context.getCategoryChildValueList() if o.getRelativeUrl() != context.getRelativeUrl()]
 sort_id='int_index'
 value_list.sort(key=lambda x: x.getProperty(sort_id))
 return value_list

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_checkDataWithScript.py

@@ -42,8 +42,8 @@ if depth == maximum_depth and id_list:
     if recursive:
       for subdocument in document.objectValues():
         recurse(subdocument)
-  for id in id_list:
-    recurse(document[id])
+  for id_ in id_list:
+    recurse(document[id_])
   if active_process is None:
     return error_list
   if error_list:
@@ -84,8 +84,8 @@ else:
   active_script = getattr(activate(priority=4), script.id)
   if depth < maximum_depth:
     relative_url = relative_url and relative_url + '/' or ''
-    for id in id_list:
-      active_script(relative_url=relative_url + id, **kw)
+    for id_ in id_list:
+      active_script(relative_url=relative_url + id_, **kw)
   else:
     kw['relative_url'] = relative_url
     for i in xrange(0, len(id_list), packet):

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_checkNamingConventions.py

@@ -85,13 +85,13 @@ def checkField(folder, form, field):
       a = template_field.getListMethodName()
       path += '/listbox'
       for x in 'columns', 'all_columns':
-        for id, title in field.get_value(x):
+        for id_, title in field.get_value(x):
           error_message += checkTitle(path, x, title, field, form)
       if a not in (None, "portal_catalog", "searchFolder", "objectValues",
                    "contentValues", "ListBox_initializeFastInput"):
         if not a.endswith('List'):
           if 0:
-            error_message += "%s : %s : %r Bad Naming Convention\n" % (path, id, a)
+            error_message += "%s : %s : %r Bad Naming Convention\n" % (path, id_, a)
   return error_message
 
 def isListBox(field):
@@ -107,7 +107,7 @@ titlecase_sub = re.compile(r"[A-Za-z]+('[A-Za-z]+)?").sub
 titlecase_repl = lambda mo: mo.group(0)[0].upper() + mo.group(0)[1:].lower()
 titlecase = lambda s: titlecase_sub(titlecase_repl, s)
 
-def checkTitle(path, id, title, field=None, form=None):
+def checkTitle(path, id_, title, field=None, form=None):
   """
     Generic function that test the validity of a title.
   """
@@ -115,11 +115,11 @@ def checkTitle(path, id, title, field=None, form=None):
   if (form is not None and form.pt not in ('form_dialog', 'folder_workflow_action_dialog')) or form is None:
     if (field is not None and not field.get_value('hidden') and \
      (title is None or len(title.strip()) == 0)) or (field is None and (title is None or len(title.strip()) == 0)):
-      return "%s : %s : can't be empty\n" % (path, id)
+      return "%s : %s : can't be empty\n" % (path, id_)
 
   for c in title:
     if c.lower() not in ALLOWED_CHARS:
-      return "%s : %s : %r character not allowed\n" % (path, id, c)
+      return "%s : %s : %r character not allowed\n" % (path, id_, c)
 
   title = re.sub(re.compile(r"\b(" + "|".join(re.escape(x) for x in SENTENCE_PART_SET) + r")\b"), "", title)
 
@@ -132,22 +132,22 @@ def checkTitle(path, id, title, field=None, form=None):
 
     if word.upper() in ABBREVIATION_WORD_SET:
       if not word.isupper():
-        error_message += '%s : %s : %r is not upper case even though it is an abbriviation\n' % (path, id, word)
+        error_message += '%s : %s : %r is not upper case even though it is an abbriviation\n' % (path, id_, word)
     elif word.endswith('s') and word[:-1].upper() in ABBREVIATION_WORD_SET:
       if not word[:-1].isupper():
-        error_message += '%s : %s : %r is not upper case even though it is an abbriviation\n' % (path, id, word)
+        error_message += '%s : %s : %r is not upper case even though it is an abbriviation\n' % (path, id_, word)
     elif "-" in word and word.split("-")[0].upper() in ABBREVIATION_WORD_SET:
       if not word.split("-")[0].isupper():
-        error_message += '%s : %s : %r is not upper case even though it is an abbriviation\n' % (path, id, word)
+        error_message += '%s : %s : %r is not upper case even though it is an abbriviation\n' % (path, id_, word)
     else:
       if word.lower() in CLOSED_CLASS_WORD_SET and word != word_list[0] :
         if (word.capitalize()== word or titlecase(word)== word):
-          error_message += '%s : %s : %r is a closed-class word and should not be titlecased\n' % (path, id, word)
+          error_message += '%s : %s : %r is a closed-class word and should not be titlecased\n' % (path, id_, word)
       elif (word.capitalize()!= word and titlecase(word)!= word) and \
          word not in LOWERCASE_WORD_SET and word not in SPECIALCASE_WORD_SET and word not in CLOSED_CLASS_WORD_SET :
-          error_message += '%s : %s : %r is not titlecased\n' % (path, id, word)
+        error_message += '%s : %s : %r is not titlecased\n' % (path, id_, word)
   if len(word_list) > 1 and word_list[-1].upper() == 'LIST' and word_list[-2].upper() != 'PACKING':
-    error_message += '%s : %s : %r is a jargon\n' % (path, id, title)
+    error_message += '%s : %s : %r is a jargon\n' % (path, id_, title)
   return error_message
 
 

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_getHelpUrl.py

-help = context.getPortalTypeName()
+help_topic = context.getPortalTypeName()
 if workflow_action is not None: # First, the workflow transition case.
-  help = '%s_%s' % (help, workflow_action)
+  help_topic = '%s_%s' % (help_topic, workflow_action)
 elif current_action is not None: # Then, we are able to get the action.
-  help = '%s_%s' % (help, current_action['id'])
+  help_topic = '%s_%s' % (help_topic, current_action['id'])
 elif current_form_id is not None: # Otherwise, get the form we are in.
-  help = '%s_%s' % (help, current_form_id)
-return '%s/%s' % (context.portal_preferences.getPreferredHtmlStyleDocumentationBaseUrl(), help)
+  help_topic = '%s_%s' % (help_topic, current_form_id)
+return '%s/%s' % (context.portal_preferences.getPreferredHtmlStyleDocumentationBaseUrl(), help_topic)

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_getPlanningBox.py

@@ -12,5 +12,5 @@ if form.meta_type != 'ERP5 Form':
 # the listbox is there.
 for group in ('bottom', 'center', 'left', 'right'):
   for field in form.get_fields_in_group(group):
-     if field.meta_type == 'PlanningBox' and not(field['hidden']) and field['enabled']:
-       return field
+    if field.meta_type == 'PlanningBox' and not(field['hidden']) and field['enabled']:
+      return field

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_newContent.py

@@ -4,7 +4,6 @@ Content will be created in the appropriate module. It is
 intended to be called from the user interface only.
 """
 translateString = context.Base_translateString
-request = context.REQUEST
 
 # Create the new content in appropriate module
 portal_object = context.getPortalObject()

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_reindexCategory.py

@@ -10,23 +10,23 @@ nb_types = {}
 for error in error_list:
   # We count the number of each portal type
   if error[1]=='portal_type':
-    type = error[3]
-    if nb_types.has_key(type):
-      nb_types[type] = nb_types[type] + 1
+    portal_type = error[3]
+    if nb_types.has_key(portal_type):
+      nb_types[portal_type] = nb_types[portal_type] + 1
     else:
-      nb_types[type] = 1
+      nb_types[portal_type] = 1
   else: 
     #print error
     return_list.append(error)
 
-for type in nb_types.keys():
+for portal_type in nb_types.keys():
   # Find the number of each portal type in the catalog
-  count_result = context.portal_catalog.countResults(portal_type=type)
+  count_result = context.portal_catalog.countResults(portal_type=portal_type)
   nb_catalog = count_result[0][0]
-  if nb_types[type] != nb_catalog:
+  if nb_types[portal_type] != nb_catalog:
     message = "XXX Warning for %s: there is %i lines in the catalog instead of %i" % \
-      (type,nb_catalog,nb_types[type])
+      (portal_type, nb_catalog, nb_types[portal_type])
     return_list.append(('Count Error', 'PortalRoot_reindexAll',1,message))
-  #else: print "%s: %i" % (type,nb_types[type])
+  #else: print "%s: %i" % (portal_type,nb_types[portal_type])
 
 return return_list

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_reindexLatestIndexedObjects.py

@@ -13,7 +13,7 @@ unindex_count = 0
 for candidate in candidate_list:
   path = candidate['path']
   try:
-    object = portal.restrictedTraverse(path)
+    obj = portal.restrictedTraverse(path)
   except KeyError:
     # Object is unreachable, remove it from catalog
     # Use SQLQueue because all activities are triggered on the same object,
@@ -21,7 +21,7 @@ for candidate in candidate_list:
     catalog.activate(activity="SQLQueue").unindexObject(uid=candidate['uid'])
     unindex_count += 1
   else:
-    object.reindexObject()
+    obj.reindexObject()
     reindex_count += 1
 
 print '%s object reindexed, %s object unindexed' % (reindex_count, unindex_count)

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_setNewIdPerBundle.py

 activate_kw = {"tag" : tag}
 folder = context.restrictedTraverse(folder_path)
 method = getattr(context, method)
-for id in id_list:
-  ob = folder.get(id)
+for id_ in id_list:
+  ob = folder.get(id_)
   new_id = method(ob)
   ob.setDefaultActivateParameterDict(activate_kw)
   ob.setId(new_id)

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_updateTranslationTable.py

@@ -11,9 +11,9 @@ z_catalog_translation_list = sql_catalog.z_catalog_translation_list
 def catalog_translation_list(object_list):
   parameter_dict = {}
   for i in object_list:
-    for property in ('language', 'message_context', 'portal_type',
+    for property_ in ('language', 'message_context', 'portal_type',
                      'original_message', 'translated_message'):
-      parameter_dict.setdefault(property, []).append(i[property])
+      parameter_dict.setdefault(property_, []).append(i[property_])
   z_catalog_translation_list(**parameter_dict)
 
 # Translate every workflow state in the context of the state variable

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityAcquiredCategoryFromContent.py

@@ -10,10 +10,10 @@ If unsure, you should use ERP5Type_getSecurityCategoryFromContent.
 """
 category_list = []
 
-if object is None:
+if ob is None:
   return []
 
 for base_category in base_category_list:
-  category_list.append({base_category: object.getAcquiredCategoryMembershipList(base_category)})
+  category_list.append({base_category: ob.getAcquiredCategoryMembershipList(base_category)})
 
 return category_list

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityAcquiredCategoryFromContent.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>base_category_list, user_name, object, portal_type</string> </value>
+            <value> <string>base_category_list, user_name, ob, portal_type</string> </value>
         </item>
         <item>
             <key> <string>_proxy_roles</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityCategoryFromArrow.py

@@ -3,4 +3,4 @@ log("ERP5Type_getSecurityCategoryFromArrow is deprecated, "
     "use ERP5Type_getSecurityCategoryFromContent instead")
 
 return context.ERP5Type_getSecurityCategoryFromContent(
-    base_category_list, user_name, object, portal_type)
+    base_category_list, user_name, ob, portal_type)

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityCategoryFromArrow.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>base_category_list, user_name, object, portal_type</string> </value>
+            <value> <string>base_category_list, user_name, ob, portal_type</string> </value>
         </item>
         <item>
             <key> <string>_proxy_roles</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityCategoryFromContent.py

@@ -19,7 +19,7 @@ The parameters are
 
   base_category_list -- list of category values we need to retrieve
   user_name          -- string obtained from getSecurityManager().getUser().getId()
-  object             -- object which we want to assign roles to
+  ob                 -- object which we want to assign roles to
   portal_type        -- portal type of object
 
 NOTE: for now, this script requires proxy manager
@@ -27,10 +27,10 @@ NOTE: for now, this script requires proxy manager
 
 category_list = []
 
-if object is None:
+if ob is None:
   return []
 
 for base_category in base_category_list:
-  category_list.append({base_category: object.getCategoryMembershipList(base_category)})
+  category_list.append({base_category: ob.getCategoryMembershipList(base_category)})
 
 return category_list

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityCategoryFromContent.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>base_category_list, user_name, object, portal_type</string> </value>
+            <value> <string>base_category_list, user_name, ob, portal_type</string> </value>
         </item>
         <item>
             <key> <string>_proxy_roles</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityCategoryFromSelf.py

@@ -14,7 +14,7 @@ The parameters are
 
   base_category_list -- list of category values we need to retrieve
   user_name          -- string obtained from getSecurityManager().getUser().getId()
-  object             -- object which we want to assign roles to
+  ob                 -- object which we want to assign roles to
   portal_type        -- portal type of object
 
 NOTE: for now, this script requires proxy manager
@@ -22,10 +22,10 @@ NOTE: for now, this script requires proxy manager
 
 category_list = []
 
-if object is None:
+if ob is None:
   return []
 
 for base_category in base_category_list:
-  category_list.append({base_category: object.getRelativeUrl()})
+  category_list.append({base_category: ob.getRelativeUrl()})
 
 return category_list

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityCategoryFromSelf.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>base_category_list, user_name, object, portal_type</string> </value>
+            <value> <string>base_category_list, user_name, ob, portal_type</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityChildCategoryFromAssignment.py

@@ -14,7 +14,7 @@ The parameters are
 
   base_category_list -- list of category values we need to retrieve
   user_name          -- string obtained from getSecurityManager().getUser().getId()
-  object             -- object which we want to assign roles to
+  ob                 -- object which we want to assign roles to
   portal_type        -- portal type of object
 
 NOTE: for now, this script requires proxy manager

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Type_getSecurityChildCategoryFromAssignment.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>base_category_list, user_name, object, portal_type</string> </value>
+            <value> <string>base_category_list, user_name, ob, portal_type</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>