[REF] po-duplicate-message-definition: Ignore i18n_extra folder

Odoo exports translations grouped by "msgid" and delete duplicated

if you will not use odoo exports feature so you will need to manage manually the po file

It could be in i18n_extra in order to ignore the Odoo's normal flow

More info about OCA#122

src/oca_pre_commit_hooks/checks_odoo_module_po.py

@@ -330,8 +330,8 @@ def check_po(self):
             if entry.obsolete:
                 continue
 
-            # po_duplicate_message_definition
-            if self.is_message_enabled("po-duplicate-message-definition"):
+            # Ignore i18n_extra https://github.com/OCA/odoo-pre-commit-hooks/issues/122
+            if self.is_message_enabled("po-duplicate-message-definition") and self.data_section != "i18n_extra":
                 duplicated[hash(entry.msgid)].append(entry)
 
             if self.is_message_enabled("po-duplicate-model-definition"):
@@ -351,7 +351,9 @@ def check_po(self):
                     msg_id_short = f"{msg_id_short}..."
                 self.register_error(
                     code="po-duplicate-message-definition",
-                    message=f"Duplicate PO message definition `{msg_id_short}` in lines {duplicated_str}",
+                    message=f"Duplicate PO message definition `{msg_id_short}` in lines {duplicated_str}. "
+                    "Odoo exports these items by msgid and delete one of them. "
+                    "Use the `i18n_extra` folder instead of `i18n` to ignore this message.",
                     filepath=self.filename_short,
                     line=self._get_po_line_number(entries[0]),
                 )

test_repo/broken_module/i18n_extra/es.po

@@ -0,0 +1,25 @@
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: Odoo Server\n"
+"Report-Msgid-Bugs-To: \n"
+"POT-Creation-Date: 1985-04-14 17:12+0000\n"
+"PO-Revision-Date: 1985-04-14 02:03+0000\n"
+"Last-Translator: Moisés López <moylop260@vauxoo.com>\n"
+"Language-Team: \n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=UTF-8\n"
+"Content-Transfer-Encoding: \n"
+"Plural-Forms: \n"
+
+#. module: broken_module
+#: model:ir.model.fields,field_description5:broken_module.field_wizard_description5
+#, python-format
+msgid "'Duplicate PO message definition' ignored for i18n_extra"
+msgstr "'translation' 1"
+
+#. module: broken_module
+#: model:ir.model.fields,field_description10:broken_module.field_wizard_description10
+#, python-format
+msgid "'Duplicate PO message definition' ignored for i18n_extra"
+msgstr "'translation' 2"