Check attribute without translation parameter (OCA#104)

pylint_odoo/checkers/modules_odoo.py

@@ -118,6 +118,11 @@
         'missing-manifest-dependency',
         settings.DESC_DFLT
     ),
+    'W%d37' % settings.BASE_OMODULE_ID: (
+        '%s The xml attribute is missing the translation="off" tag %s',
+        'xml-attribute-translatable',
+        settings.DESC_DFLT
+    ),
 }
 
 
@@ -811,3 +816,18 @@ def _check_file_not_used(self):
         if self.msg_args:
             return False
         return True
+
+    def _check_xml_attribute_translatable(self):
+        """The xml attribute is missing the translation="off" tag
+            Example  <attribute name="groups">sale.group</attribute>
+        """
+        self.msg_args = []
+        for xml_file in self.filter_files_ext('xml', relpath=True):
+            for record in self.get_xml_records(
+                    os.path.join(self.module_path, xml_file), None,
+                    '//attribute[not(@translation)]'):
+                self.msg_args.append(
+                    ("%s:%d" % (xml_file, record.sourceline), 'xml_id'))
+        if self.msg_args:
+            return False
+        return True

pylint_odoo/misc.py

@@ -385,7 +385,7 @@ def parse_xml(self, xml_file):
             return xmlsyntax_error_exception.message
         return doc
 
-    def get_xml_records(self, xml_file, model=None):
+    def get_xml_records(self, xml_file, model=None, more=None):
         """Get tag `record` of a openerp xml file.
         :param xml_file: Path of file xml
         :param model: String with record model to filter.
@@ -398,9 +398,13 @@ def get_xml_records(self, xml_file, model=None):
             model_filter = ''
         else:
             model_filter = "[@model='{model}']".format(model=model)
+        if more is None:
+            more_filter = ''
+        else:
+            more_filter = more
         doc = self.parse_xml(xml_file)
-        return doc.xpath("/openerp//record" + model_filter) + \
-            doc.xpath("/odoo//record" + model_filter) \
+        return doc.xpath("/openerp//record" + model_filter + more_filter) + \
+            doc.xpath("/odoo//record" + model_filter + more_filter) \
             if not isinstance(doc, basestring) else []
 
     def get_field_csv(self, csv_file, field='id'):

pylint_odoo/test/main.py

@@ -58,6 +58,7 @@
     'xml-syntax-error': 2,
     'renamed-field-parameter': 2,
     'attribute-string-redundant': 33,
+    'xml-attribute-translatable': 1,
 }
 
 

pylint_odoo/test_repo/broken_module/model_view_odoo2.xml

@@ -25,6 +25,10 @@
                     <field name="partner_id"/>
                     <field name="user_id"/>
                 </xpath>
+                <xpath expr="//field[@name='description']" position="attributes">
+                    <attribute name="colors">red</attribute>
+                    <attribute name="colors" translation="off">red</attribute>
+                </xpath>
             </field>
         </record>