IMP l10n_it_edi_extension avoiding importing TerzoIntermediarioOSoggettoEmittente

to be moved to dedicated module

Author: eLBati

l10n_it_edi_extension/README.rst

@@ -74,7 +74,6 @@ Le funzionalità principali incluse sono:
       non esistono in anagrafica tra cui:
 
       -  ``<RappresentanteFiscale>``
-      -  ``<TerzoIntermediarioOSoggettoEmittente>``
 
    -  Nome e cognome vengono assegnati ai nuovi contatti in base ai nodi
       ``<Nome>`` e ``<Cognome>`` della fattura elettronica.
@@ -230,7 +229,6 @@ The main features included are:
       they don't exist in the address book, including:
 
       -  ``<RappresentanteFiscale>``
-      -  ``<TerzoIntermediarioOSoggettoEmittente>``
 
    -  Name and surname are set in the new contacts based on the
       ``<Nome>`` and ``<Cognome>`` nodes of the E-Bill.
@@ -366,6 +364,7 @@ Le uniche configurazioni disponibili sono:
 
    -  ``<CessionarioCommittente>``
    -  ``<CedentePrestatore>``
+   -  ``<RappresentanteFiscale>``
 
 Non sono necessarie altre configurazioni specifiche per
 l10n_it_edi_extension: una volta installato, le sue funzionalità sono
@@ -417,7 +416,6 @@ The only available configurations are:
    -  ``<CessionarioCommittente>``
    -  ``<CedentePrestatore>``
    -  ``<RappresentanteFiscale>``
-   -  ``<TerzoIntermediarioOSoggettoEmittente>``
 
 No other specific configurations are required for l10n_it_edi_extension:
 once installed, its features are active and automatically integrate into

l10n_it_edi_extension/__init__.py

@@ -46,16 +46,6 @@ def _l10n_it_fatturapa_pre_migration(env):
                 "l10n_it_edi_tax_representative_id",
             ),
         ],
-        [
-            (
-                "account.move",
-                "intermediary",
-            ),
-            (
-                "account.move",
-                "l10n_it_edi_intermediary_id",
-            ),
-        ],
         [
             (
                 "account.move",
@@ -296,16 +286,6 @@ def _l10n_it_fatturapa_pre_migration(env):
                 "l10n_edi_it_admin_ref",
             ),
         ],
-        [
-            (
-                "res.company",
-                "fatturapa_sender_partner",
-            ),
-            (
-                "res.company",
-                "l10n_edi_it_sender_partner",
-            ),
-        ],
         [
             (
                 "res.company",

l10n_it_edi_extension/models/account_move.py

@@ -17,7 +17,6 @@ class AccountMoveInherit(models.Model):
     l10n_it_edi_tax_representative_id = fields.Many2one(
         "res.partner", string="Tax Representative"
     )
-    l10n_it_edi_intermediary_id = fields.Many2one("res.partner", string="Intermediary")
     l10n_it_edi_sender = fields.Selection(
         [("CC", "Assignee / Partner"), ("TZ", "Third Person")], string="Sender"
     )
@@ -587,8 +586,6 @@ def _l10n_it_edi_extend_partner_info(self, partner_role, partner_info):
             partner_info_xpath = "//CessionarioCommittente"
         elif partner_role == "seller":
             partner_info_xpath = "//CedentePrestatore"
-        elif partner_role == "intermediary":
-            partner_info_xpath = "//TerzoIntermediarioOSoggettoEmittente"
         elif partner_role == "tax_representative":
             partner_info_xpath = "//RappresentanteFiscale"
         else:
@@ -765,9 +762,4 @@ def _l10n_it_edi_import_invoice(self, invoice, data, is_new):
         ):
             invoice.l10n_it_edi_tax_representative_id = tax_representative
 
-        if intermediary := self._l10n_it_edi_extension_create_partner(
-            body_tree,
-            "intermediary",
-        ):
-            invoice.l10n_it_edi_intermediary_id = intermediary
         return invoice

l10n_it_edi_extension/models/res_company.py

@@ -16,12 +16,6 @@ class ResCompanyInherit(models.Model):
         "year several documents with same number)",
     )
     l10n_edi_it_admin_ref = fields.Char(string="Public Administration Reference Code")
-    l10n_edi_it_sender_partner = fields.Many2one(
-        "res.partner",
-        string="Third Party/Sender",
-        help="Data of Third-Party Issuer Intermediary who emits the "
-        "invoice on behalf of the seller/provider",
-    )
     l10n_edi_it_stable_organization = fields.Many2one(
         "res.partner",
         string="Stable Organization",

l10n_it_edi_extension/readme/CONFIGURE.md

@@ -6,6 +6,7 @@ Le uniche configurazioni disponibili sono:
   - Crea il partner se non esiste durante l'importazione: in Fatturazione (o Contabilità per EE) > Configurazione > Impostazioni > Fatturazione Elettronica Italiana, spuntare l'opzione se si vuole attivare la funzionalità per i dati dei nodi:
      - `<CessionarioCommittente>`
      - `<CedentePrestatore>`
+     - `<RappresentanteFiscale>`
 
 Non sono necessarie altre configurazioni specifiche per l10n_it_edi_extension: una volta installato, le sue funzionalità sono attive e si integrano automaticamente nell'interfaccia esistente.
 
@@ -29,7 +30,6 @@ The only available configurations are:
      - `<CessionarioCommittente>`
      - `<CedentePrestatore>`
      - `<RappresentanteFiscale>`
-     - `<TerzoIntermediarioOSoggettoEmittente>`
 
 No other specific configurations are required for l10n_it_edi_extension: once installed, its features are active and automatically integrate into the existing interface.
 

l10n_it_edi_extension/readme/DESCRIPTION.md

@@ -22,7 +22,6 @@ Le funzionalità principali incluse sono:
      - `<CessionarioCommittente>` e `<CedentePrestatore>` invece di scrivere solamente i dettagli nel chatter.
    - Creazione di altri contatti presenti in una fattura elettronica se non esistono in anagrafica tra cui:
      - `<RappresentanteFiscale>`
-     - `<TerzoIntermediarioOSoggettoEmittente>`
    - Nome e cognome vengono assegnati ai nuovi contatti in base ai nodi `<Nome>` e `<Cognome>` della fattura elettronica.
    - Gestione delle Fatture Elettroniche Multiple:
      - Supporto per la suddivisione automatica di file XML contenenti più fatture
@@ -104,7 +103,6 @@ The main features included are:
      - `<CessionarioCommittente>` and `<CedentePrestatore>` instead of just writing the details in the chatter.
    - Creation of other contacts present in an electronic invoice if they don't exist in the address book, including:
      - `<RappresentanteFiscale>`
-     - `<TerzoIntermediarioOSoggettoEmittente>`
    - Name and surname are set in the new contacts based on the `<Nome>` and `<Cognome>` nodes of the E-Bill.
    - Multiple Electronic Invoice Management:
      - Support for automatic splitting of XML files containing multiple invoices

l10n_it_edi_extension/static/description/index.html

@@ -412,7 +412,6 @@ <h1 class="title">Italy - E-invoicing - Base Feature</h1>
 <li>Creazione di altri contatti presenti in una fattura elettronica se
 non esistono in anagrafica tra cui:<ul>
 <li><tt class="docutils literal">&lt;RappresentanteFiscale&gt;</tt></li>
-<li><tt class="docutils literal">&lt;TerzoIntermediarioOSoggettoEmittente&gt;</tt></li>
 </ul>
 </li>
 <li>Nome e cognome vengono assegnati ai nuovi contatti in base ai nodi
@@ -563,7 +562,6 @@ <h1 class="title">Italy - E-invoicing - Base Feature</h1>
 <li>Creation of other contacts present in an electronic invoice if
 they don’t exist in the address book, including:<ul>
 <li><tt class="docutils literal">&lt;RappresentanteFiscale&gt;</tt></li>
-<li><tt class="docutils literal">&lt;TerzoIntermediarioOSoggettoEmittente&gt;</tt></li>
 </ul>
 </li>
 <li>Name and surname are set in the new contacts based on the
@@ -703,6 +701,7 @@ <h1><a class="toc-backref" href="#toc-entry-1">Configuration</a></h1>
 funzionalità per i dati dei nodi:<ul>
 <li><tt class="docutils literal">&lt;CessionarioCommittente&gt;</tt></li>
 <li><tt class="docutils literal">&lt;CedentePrestatore&gt;</tt></li>
+<li><tt class="docutils literal">&lt;RappresentanteFiscale&gt;</tt></li>
 </ul>
 </li>
 </ul>
@@ -753,7 +752,6 @@ <h1><a class="toc-backref" href="#toc-entry-1">Configuration</a></h1>
 <li><tt class="docutils literal">&lt;CessionarioCommittente&gt;</tt></li>
 <li><tt class="docutils literal">&lt;CedentePrestatore&gt;</tt></li>
 <li><tt class="docutils literal">&lt;RappresentanteFiscale&gt;</tt></li>
-<li><tt class="docutils literal">&lt;TerzoIntermediarioOSoggettoEmittente&gt;</tt></li>
 </ul>
 </li>
 </ul>

l10n_it_edi_extension/tests/import_xmls/IT05979361218_011.xml

@@ -52,11 +52,6 @@ def _edi_import_invoice(self, filename):
 
         return moves
 
-    def test_01_xml_import(self):
-        move = self._edi_import_invoice("IT05979361218_011.xml")
-        move._extend_with_attachments(move.l10n_it_edi_attachment_id, new=True)
-        self.assertEqual(move.l10n_it_edi_intermediary_id.vat, "03339130126")
-
     def test_02_xml_import(self):
         move = self._edi_import_invoice("IT02780790107_11005.xml")
         move._extend_with_attachments(move.l10n_it_edi_attachment_id, new=True)
@@ -81,7 +76,6 @@ def test_03_xml_import(self):
         move._extend_with_attachments(move.l10n_it_edi_attachment_id, new=True)
         self.assertEqual(move.ref, "FT/2015/0008")
         self.assertEqual(move.l10n_it_edi_sender, "TZ")
-        self.assertEqual(move.l10n_it_edi_intermediary_id.name, "MARIO ROSSI")
         self.assertEqual(
             move.l10n_it_edi_line_ids[0].l10n_it_edi_discount_rise_price_ids[0].name,
             "SC",

l10n_it_edi_extension/tests/test_import_edi_extension_xml.py

@@ -154,10 +154,6 @@
             <xpath expr="//field[@name='l10n_it_tax_system']" position="after">
                 <field name="l10n_edi_it_art73" invisible="country_code != 'IT'" />
                 <field name="l10n_edi_it_admin_ref" invisible="country_code != 'IT'" />
-                <field
-                    name="l10n_edi_it_sender_partner"
-                    invisible="country_code != 'IT'"
-                />
                 <field
                     name="l10n_edi_it_stable_organization"
                     invisible="country_code != 'IT'"