Cosmetic: Fix Pylint warnings.

bt5/erp5_administration/ExtensionTemplateItem/portal_components/extension.erp5.ERP5Administration.py

@@ -20,15 +20,15 @@ def dumpWorkflowChain(self):
   ti = workflow_tool._listTypeInfo()
   types_info = []
   for t in ti:
-    id = t.getId()
+    id_ = t.getId()
     title = t.Title()
-    if title == id:
+    if title == id_:
       title = None
-    if cbt is not None and cbt.has_key(id):
-      chain = sorted(cbt[id])
+    if cbt is not None and cbt.has_key(id_):
+      chain = sorted(cbt[id_])
     else:
       chain = ['(Default)']
-    types_info.append({'id': id,
+    types_info.append({'id': id_,
                        'title': title,
                        'chain': chain})
   output = []
@@ -94,7 +94,7 @@ def MessageCatalog_deleteNotTranslatedMessageDict(self):
     Delete from  Localizer's MessageCatalog instance messages that are NOT translated.
   """
   not_translated_message_dict = MessageCatalog_getNotTranslatedMessageDict(self)
-  for k,v in not_translated_message_dict.iteritems():
+  for k,_ in not_translated_message_dict.iteritems():
     # delete message from dict
     del(self._messages[k])
   return len(not_translated_message_dict.keys())
@@ -117,7 +117,7 @@ def checkConversionToolAvailability(self):
   try:
     temp_document = newTempOOoDocument(self, document_id, data=document_file.data, source_reference=document_id)
     temp_document.convertToBaseFormat()
-    mimetype, html_result = temp_document.convert(format='html')
+    _, html_result = temp_document.convert(format='html')
   except ConflictError:
     raise
   except: #Which Errors should we catch ?

bt5/erp5_administration/ExtensionTemplateItem/portal_components/extension.erp5.ERP5Administration.xml

@@ -6,10 +6,22 @@
     </pickle>
     <pickle>
       <dictionary>
+        <item>
+            <key> <string>_recorded_property_dict</string> </key>
+            <value>
+              <persistent> <string encoding="base64">AAAAAAAAAAI=</string> </persistent>
+            </value>
+        </item>
         <item>
             <key> <string>default_reference</string> </key>
             <value> <string>ERP5Administration</string> </value>
         </item>
+        <item>
+            <key> <string>description</string> </key>
+            <value>
+              <none/>
+            </value>
+        </item>
         <item>
             <key> <string>id</string> </key>
             <value> <string>extension.erp5.ERP5Administration</string> </value>
@@ -24,6 +36,20 @@
               <none/>
             </value>
         </item>
+        <item>
+            <key> <string>text_content_error_message</string> </key>
+            <value>
+              <tuple/>
+            </value>
+        </item>
+        <item>
+            <key> <string>text_content_warning_message</string> </key>
+            <value>
+              <tuple>
+                <string>W:   88,   29: map/filter on lambda could be replaced by comprehension (deprecated-lambda)</string>
+              </tuple>
+            </value>
+        </item>
         <item>
             <key> <string>version</string> </key>
             <value> <string>erp5</string> </value>
@@ -31,13 +57,28 @@
         <item>
             <key> <string>workflow_history</string> </key>
             <value>
-              <persistent> <string encoding="base64">AAAAAAAAAAI=</string> </persistent>
+              <persistent> <string encoding="base64">AAAAAAAAAAM=</string> </persistent>
             </value>
         </item>
       </dictionary>
     </pickle>
   </record>
   <record id="2" aka="AAAAAAAAAAI=">
+    <pickle>
+      <global name="PersistentMapping" module="Persistence.mapping"/>
+    </pickle>
+    <pickle>
+      <dictionary>
+        <item>
+            <key> <string>data</string> </key>
+            <value>
+              <dictionary/>
+            </value>
+        </item>
+      </dictionary>
+    </pickle>
+  </record>
+  <record id="3" aka="AAAAAAAAAAM=">
     <pickle>
       <global name="PersistentMapping" module="Persistence.mapping"/>
     </pickle>
@@ -50,7 +91,7 @@
                 <item>
                     <key> <string>component_validation_workflow</string> </key>
                     <value>
-                      <persistent> <string encoding="base64">AAAAAAAAAAM=</string> </persistent>
+                      <persistent> <string encoding="base64">AAAAAAAAAAQ=</string> </persistent>
                     </value>
                 </item>
               </dictionary>
@@ -59,7 +100,7 @@
       </dictionary>
     </pickle>
   </record>
-  <record id="3" aka="AAAAAAAAAAM=">
+  <record id="4" aka="AAAAAAAAAAQ=">
     <pickle>
       <global name="WorkflowHistoryList" module="Products.ERP5Type.patches.WorkflowTool"/>
     </pickle>
@@ -72,33 +113,6 @@
                 <key> <string>action</string> </key>
                 <value> <string>validate</string> </value>
             </item>
-            <item>
-                <key> <string>actor</string> </key>
-                <value> <string>ERP5TypeTestCase</string> </value>
-            </item>
-            <item>
-                <key> <string>comment</string> </key>
-                <value> <string></string> </value>
-            </item>
-            <item>
-                <key> <string>time</string> </key>
-                <value>
-                  <object>
-                    <klass>
-                      <global name="DateTime" module="DateTime.DateTime"/>
-                    </klass>
-                    <tuple>
-                      <none/>
-                    </tuple>
-                    <state>
-                      <tuple>
-                        <float>1377844543.44</float>
-                        <string>GMT+9</string>
-                      </tuple>
-                    </state>
-                  </object>
-                </value>
-            </item>
             <item>
                 <key> <string>validation_state</string> </key>
                 <value> <string>validated</string> </value>

bt5/erp5_administration/bt/change_log

+2014-02-24 arnaud.fontaine
+* Cosmetic: Fix Pylint warnings.
+
 2014-02-24 arnaud.fontaine
 * Use Pylint to check source code to unify ZODB Components and Python Script source code checking.
 

bt5/erp5_administration/bt/revision

-159
+160
\ No newline at end of file

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

@@ -53,8 +53,7 @@
             <value> <string encoding="cdata"><![CDATA[
 
 import re\n
-message_re = re.compile(\'[CRWEF]:\\s*(?P<line>\\d+),\\s*(?P<column>\\d+):\\s*.*\')\n
-error_warning_dict = {\'error_list\': [], \'warning_list\': []}\n
+message_re = re.compile(r\'[CRWEF]:\\s*(?P<line>\\d+),\\s*(?P<column>\\d+):\\s*.*\')\n
 \n
 def getParsedMessageList(message_list):\n
   result_list = []\n

product/ERP5/bootstrap/erp5_core/bt/change_log

+2014-02-24 arnaud.fontaine
+* Cosmetic: Fix Pylint warnings.
+
 2014-02-24 arnaud.fontaine
 * Follow API changes to unify ZODB Components and Python Script source code checking.
 

product/ERP5/bootstrap/erp5_core/bt/revision

-41153
+41154
\ No newline at end of file