[18.0] [IMP] l10n_it_edi_doi_extension: add migration scripts by odooNextev · Pull Request #4830 · OCA/l10n-italy

odooNextev · 2025-07-09T14:41:46Z

Migrazione da l10n_it_declaration_of_intent e fix di sincronizzazione

Obiettivo

Questa PR aggiunge al modulo l10n_it_edi_doi_extension la gestione automatica
della migrazione dei dati dal vecchio modulo OCA l10n_it_declaration_of_intent
(Odoo 16) al nuovo modulo l10n_it_edi_doi di Odoo 18.

La migrazione viene eseguita completamente in automatico tramite hook pre/post-init,
senza richiedere interventi manuali da parte dell'utente (a parte il backup del
database).

Le fatture con più dichiarazioni in v16 vengono migrate con amount = 0 nel bridge:
è un comportamento atteso perché la v16 non memorizzava l'importo per dichiarazione

Commit inclusi

1. `[IMP] add migration scripts` (`5afe7901d`)

Introduce gli hook di migrazione e aggiorna la documentazione.

File principali:

__init__.py — cuore della migrazione (hook pre e post init)
__manifest__.py — registrazione degli hook pre_init_hook e post_init_hook
models/account_move.py — fallback per le fatture migrate senza imposta DOI
models/declaration_of_intent.py — aggiunto campo number (già presente in v16)
views/l10n_it_edi_doi_declaration_of_intent_views.xml — aggiunto campo number
nel form e nella list view
readme/USAGE.md — sezione di migrazione per gli utenti finali

Cosa fa il pre-init hook:

Aggiunge le repartition lines mancanti sulle imposte gruppo split payment
(necessarie per Odoo 18, causavano ValidationError altrimenti)
Elimina le view del vecchio modulo incompatibili con Odoo 18
(usavano il tag <tree> rinominato in <list>)
Marca il vecchio modulo come to remove
Se il database aveva solo il vecchio modulo (tabella nuova vuota):
rinomina la tabella e il modello, rinomina i campi, mappa i valori degli stati
Se il database aveva già dati in l10n_it_edi_doi (installazione parziale):
inserisce i vecchi record con offset ID per evitare conflitti di chiave primaria

Cosa fa il post-init hook:

Popola la tabella bridge account.move.doi dalle vecchie relazioni many2many,
preservando tutti i collegamenti (non solo il primo)
Per le fatture migrate senza imposta DOI, imposta l10n_it_edi_doi_amount
da amount_untaxed
Aggiorna i riferimenti ir.model.data al nuovo modulo
Pulisce i residui del vecchio modulo (view, menu, access rules, ir.model, ecc.)

Come testare

Scenario 1 — Migrazione da v16 con dati

Migrare un database di Odoo 16 con l10n_it_declaration_of_intent installato ed
alcune dichiarazioni di intento e fatture collegate a Odoo 18
Installare l10n_it_edi_doi_extension
Verificare dopo l'installazione:
- Le dichiarazioni compaiono nel menu con i numeri di protocollo corretti
  (split in protocol_number_part1 / protocol_number_part2)
- Gli stati sono mappati correttamente (valid → active, expired → terminated,
  close → revoked)
- Le fatture hanno l10n_it_edi_doi_id popolato
- La tabella bridge account.move.doi contiene i record migrati
- Gli importi calcolati (fatturato, residuo) sono corretti
Disinstallare l10n_it_declaration_of_intent e verificare che non ci siano errori

2. `[FIX] sync l10n_it_edi_doi_id and fix available amount compute` (`d823abe29`)

Correzioni al modello bridge account.move.doi.

File: models/account_move_doi.py

Modifiche:

_compute_declaration_available: ora usa declaration_id.remaining invece di
ricalcolare threshold - invoiced inline (allineato ai campi calcolati del modello)
Aggiunto metodo _sync_l10n_it_edi_doi_id: mantiene sincronizzato il campo
l10n_it_edi_doi_id della fattura con il primo record bridge associato
Override di create, write, unlink per chiamare _sync_l10n_it_edi_doi_id
automaticamente ad ogni modifica dei record bridge

Come testare

Scenario 1 — Installazione pulita (senza migrazione)

Verificare che l10n_it_edi_doi_id si aggiorni automaticamente alla creazione/
modifica/eliminazione di un record bridge

Scenario 2 — Fix bridge model

Con il modulo installato, aprire una fattura
Nel tab "Dichiarazioni di Intento", aggiungere un record bridge
Verificare che l10n_it_edi_doi_id sulla fattura punti alla dichiarazione del
primo record bridge
Eliminare il record bridge e verificare che l10n_it_edi_doi_id venga svuotato

Note per i revisori

In caso di errore di chiave duplicata durante l'installazione di l10n_it_edi_doi
(per posizioni fiscali o imposte già presenti dal v16), l'utente deve rimuovere
manualmente i duplicati: questo non è gestito dalla nostra migrazione perché
avviene nel post_init di l10n_it_edi_doi, eseguito prima dei nostri hook
Le fatture con più dichiarazioni in v16 vengono migrate con amount = 0 nel bridge:
è un comportamento atteso perché la v16 non memorizzava l'importo per dichiarazione
Il file USAGE.md contiene la documentazione tecnica completa per chi volesse
approfondire i dettagli

mmircoli-nexapp · 2026-02-16T11:16:46Z

Scusa la pr è pronta per la review? hai già usato questo modulo su Odoo.sh?

odooNextev · 2026-02-16T11:23:52Z

Scusa la pr è pronta per la review? hai già usato questo modulo su Odoo.sh?

No, prima deve essere approvata e mergiata questa: #5029
Senza il supporto per le doi multiple infatti non si ha una corrispondenza esatta degli oggetti dalla 16 alla 18

mmircoli-nexapp · 2026-02-16T11:27:02Z

Scusa la pr è pronta per la review? hai già usato questo modulo su Odoo.sh?

No, prima deve essere approvata e mergiata questa: #5029 Senza il supporto per le doi multiple infatti non si ha una corrispondenza esatta degli oggetti dalla 16 alla 18

Grazie del chiarimento

mmircoli-nexapp · 2026-02-17T15:35:25Z

@sergiocorato potresti lanciare un rebase qui cosi da arrivare al merge anche di questa?

francesco-ooops · 2026-02-17T15:43:58Z

/ocabot rebase

OCA-git-bot · 2026-02-17T15:44:10Z

@francesco-ooops The rebase process failed, because command git rebase origin/18.0 failed with output:

Rebasing (1/1)
Auto-merging l10n_it_edi_doi_extension/README.rst
CONFLICT (content): Merge conflict in l10n_it_edi_doi_extension/README.rst
Auto-merging l10n_it_edi_doi_extension/__manifest__.py
Auto-merging l10n_it_edi_doi_extension/static/description/index.html
error: could not apply 9d3bdda31... [IMP] l10n_it_edi_doi_extension: add migration scripts
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply 9d3bdda31... [IMP] l10n_it_edi_doi_extension: add migration scripts

sergiocorato · 2026-02-17T17:40:12Z

@sergiocorato potresti lanciare un rebase qui cosi da arrivare al merge anche di questa?

Il rebase fa prima a farlo l' autore in genere

mmircoli-nexapp · 2026-02-17T17:42:06Z

Ok Dott. Matteo Mircoli Il giorno 17 feb 2026, alle ore 18:40, Sergio Corato ***@***.***> ha scritto: [https://avatars.githubusercontent.com/u/7657311?s=20&v=4]sergiocorato left a comment (OCA/l10n-italy#4830)<https://eu-central-1.protection.sophos.com?d=github.com&u=aHR0cHM6Ly9naXRodWIuY29tL09DQS9sMTBuLWl0YWx5L3B1bGwvNDgzMCNpc3N1ZWNvbW1lbnQtMzkxNjExNjIxOQ==&p=m&i=NjkyZGQwNzBiMjkxNTYwMzVkMjczNGEx&t=Yzd1NHhDZ1lUYXZwa2NxSUpzWWxhUmlBeGp4Q2N4TmJLZ28yaHRaTWNucz0=&h=4205fa3fc498444a8494b24db24032e0&s=AVNPUEhUT0NFTkNSWVBUSVY6K0T94M0wPyDMzKN9BocAHrXlbSFJ3THbbw-AzQ3TEw> @sergiocorato<https://eu-central-1.protection.sophos.com?d=github.com&u=aHR0cHM6Ly9naXRodWIuY29tL3Nlcmdpb2NvcmF0bw==&p=m&i=NjkyZGQwNzBiMjkxNTYwMzVkMjczNGEx&t=c1NDZkRBY0I5Y1hnd0hOYnZsb3AwRkhvdm1DK2RVL2hXZWc0NHIreFJ6dz0=&h=4205fa3fc498444a8494b24db24032e0&s=AVNPUEhUT0NFTkNSWVBUSVY6K0T94M0wPyDMzKN9BocAHrXlbSFJ3THbbw-AzQ3TEw> potresti lanciare un rebase qui cosi da arrivare al merge anche di questa? Il rebase fa prima a farlo l' autore in genere — Reply to this email directly, view it on GitHub<https://eu-central-1.protection.sophos.com?d=github.com&u=aHR0cHM6Ly9naXRodWIuY29tL09DQS9sMTBuLWl0YWx5L3B1bGwvNDgzMCNpc3N1ZWNvbW1lbnQtMzkxNjExNjIxOQ==&p=m&i=NjkyZGQwNzBiMjkxNTYwMzVkMjczNGEx&t=Yzd1NHhDZ1lUYXZwa2NxSUpzWWxhUmlBeGp4Q2N4TmJLZ28yaHRaTWNucz0=&h=4205fa3fc498444a8494b24db24032e0&s=AVNPUEhUT0NFTkNSWVBUSVY6K0T94M0wPyDMzKN9BocAHrXlbSFJ3THbbw-AzQ3TEw>, or unsubscribe<https://eu-central-1.protection.sophos.com?d=github.com&u=aHR0cHM6Ly9naXRodWIuY29tL25vdGlmaWNhdGlvbnMvdW5zdWJzY3JpYmUtYXV0aC9CWTNDRjNWSUlTVVRKUE40RFRQTzZSRDRNTkhCSEFWQ05GU002QUFBQUFDQkVLN0VCMlZISTJEU01WUVdJWDNMTVY0M09TTFRPTjJXS1EzUE5WV1dLM1RVSE1aVFNNSldHRVlUTU1SUkhF&p=m&i=NjkyZGQwNzBiMjkxNTYwMzVkMjczNGEx&t=bFg4TEI2KzZRS1V3U2RuSTE3bXhucWVPMWxpaEFBUk1aaGxrM0JNRDIwMD0=&h=4205fa3fc498444a8494b24db24032e0&s=AVNPUEhUT0NFTkNSWVBUSVY6K0T94M0wPyDMzKN9BocAHrXlbSFJ3THbbw-AzQ3TEw>. You are receiving this because you commented.Message ID: ***@***.***>

odooNextev · 2026-02-18T08:03:49Z

@mmircoli-nexapp purtroppo non basta il solo rebase, devo aggiungere la parte che riguarda le doi multiple siccome non avevo la certezza di come sarebbero stati definiti i modelli effettivamente finché l'altra PR fosse stata approvata.
Lo farò a breve.

mmircoli-nexapp · 2026-02-18T08:06:12Z

@mmircoli-nexapp purtroppo non basta il solo rebase, devo aggiungere la parte che riguarda le doi multiple siccome non avevo la certezza di come sarebbero stati definiti i modelli effettivamente finché l'altra PR fosse stata approvata. Lo farò a breve.

Grazie Mille

odooNextev · 2026-03-03T14:57:36Z

Ciao @mmircoli-nexapp,
ora la PR è pronta da testare.
Per ora l'ho fatto in locale su alcuni DB.

mmircoli-nexapp · 2026-03-03T15:03:47Z

Ciao @mmircoli-nexapp, ora la PR è pronta da testare. Per ora l'ho fatto in locale su alcuni DB.

Grazie Mille come torno sul Cliente che devo migrare faccio qualche prova, che almeno non rompa nulla

monen17

Grazie della PR!
Si può aggiungere il supporto per la migrazione con OpenUpgrade? Così migrando con OpenUpgrade il modulo viene migrato in automatico senza doverlo installare a manina, vedi ad esempio https://github.com/OCA/l10n-italy/wiki/Migrazione-con-rinomina-modulo,-compatibile-con-OpenUpgrade-e-Odoo-SA.

l10n_it_edi_doi_extension/__init__.py

l10n_it_edi_doi_extension/tests/test_doi_issued_from_company.py

l10n_it_edi_doi_extension/views/l10n_it_edi_doi_declaration_of_intent_views.xml

l10n_it_edi_doi_extension/__init__.py

l10n_it_edi_doi_extension/models/account_move_doi.py

eLBati

LGTM

eLBati · 2026-03-17T16:55:58Z

/ocabot rebase

OCA-git-bot · 2026-03-17T16:56:06Z

Congratulations, PR rebased to 18.0.

monen17

Grazie delle modifiche!
C'è ancora qualcosa ma decisamente poca roba rispetto all'inizio.

Puoi aggiornare la descrizione della PR? Mi pare descriva ancora le vecchie modifiche.

Cosa ne pensi invece della proposta nella vecchia revisione:

Si può aggiungere il supporto per la migrazione con OpenUpgrade? Così migrando con OpenUpgrade il modulo viene migrato in automatico senza doverlo installare a manina, vedi ad esempio https://github.com/OCA/l10n-italy/wiki/Migrazione-con-rinomina-modulo,-compatibile-con-OpenUpgrade-e-Odoo-SA.

?

l10n_it_edi_doi_extension/__init__.py

monen17 · 2026-03-20T20:16:34Z

l10n_it_edi_doi_extension/readme/INSTALL.md

+- *Invoices with multiple DOIs show amount = 0*: expected behaviour. The old many2many
+  relation did not store per-declaration amounts. Open each affected invoice and assign
+  the correct amount in the "Declarations of Intent" tab.


Ora che viene popolato account_move_doi, questo non dovrebbe succedere giusto?

monen17 · 2026-03-20T20:18:28Z

l10n_it_edi_doi_extension/tests/test_doi_issued_from_company.py

        Test that declaration_available uses remaining (which includes not_yet_invoiced)
        """
-        invoice = self._create_invoice("9e", self.partner, tax=self.tax, in_type=True)
+        invoice = self._create_invoice("9e", self.partner, taxes=self.tax, in_type=True)


Questa modifica va nell'altro commit

monen17 · 2026-03-20T20:27:14Z

l10n_it_edi_doi_extension/hooks.py

+    # Populate l10n_it_edi_doi_amount for all invoices linked to DOI
+    # In v18, this field is a stored computed field that calculates the
+    # total amount of invoice lines that have the special DOI tax applied.
+    # However, in v16, invoices used different taxes
+    # (like "Non imponibile iva ART. 8...") and the DOI amount wasn't
+    # explicitly tracked.
+    #
+    # Since we can't recompute using the v18 logic (invoices don't have
+    # the v18 DOI tax), we manually populate it with the invoice's
+    # untaxed amount. This matches the v16 behavior where the full invoice
+    # amount was considered.
+    #
+    # In v18, l10n_it_edi_doi_amount is always positive regardless of move_type.
+    # The v18 compute is: sum(price_total) * -direction_sign
+    # Since price_total already has the accounting sign and direction_sign
+    # flips it back, the result is always positive.
+    # We use ABS(amount_untaxed) as a proxy for migrated invoices.
+    openupgrade.logged_query(
+        env.cr,
+        """
+        UPDATE account_move am
+        SET l10n_it_edi_doi_amount = ABS(am.amount_untaxed)
+        WHERE am.l10n_it_edi_doi_id IS NOT NULL
+        AND am.l10n_it_edi_doi_amount = 0
+        """,
+    )


In 16.0 si sapeva l'importo per ogni abbinamento dichiarazione/fattura grazie alle righe della dichiarazione (l10n_it_declaration_of_intent.declaration_line).

Ora che account_move_doi viene popolato correttamente, non si potrebbe mettere l'importo giusto qui?

monen17 · 2026-03-20T20:31:27Z

l10n_it_edi_doi_extension/hooks.py

+    # Populate account_move_doi bridge table for multiple DOI support.
+    # Source priority:
+    # 1. l10n_it_declaration_of_intent_declaration_line: has the real per-invoice
+    #    amount (the authoritative source from v16).
+    # 2. _REL_TABLE (m2m): no amount, used only if lines were already dropped.
+    # 3. l10n_it_edi_doi_id (many2one): last resort if both old tables are gone.


Perché non si sa se esistono queste tabelle? La tabella delle righe dovrebbe esserci: prendiamo semplicemente i dati da lì no?

l10n_it_edi_doi_extension/hooks.py

eLBati · 2026-04-07T13:32:57Z

/ocabot rebase

…lable amount compute

OCA-git-bot · 2026-04-07T13:33:06Z

Congratulations, PR rebased to 18.0.

odooNextev marked this pull request as draft July 9, 2025 14:42

monen17 added the no issue needed label Jan 9, 2026

francesco-ooops added Migration scripts 18.0 labels Feb 16, 2026

stenext force-pushed the 18.0-imp-l10n_it_edi_doi_extension-migration_scripts branch 3 times, most recently from fde17d0 to d823abe Compare March 3, 2026 14:12

odooNextev marked this pull request as ready for review March 3, 2026 14:27

stenext force-pushed the 18.0-imp-l10n_it_edi_doi_extension-migration_scripts branch from d823abe to 0c5a2d9 Compare March 3, 2026 14:56

monen17 requested changes Mar 6, 2026

View reviewed changes

stenext force-pushed the 18.0-imp-l10n_it_edi_doi_extension-migration_scripts branch 3 times, most recently from c995965 to 5737200 Compare March 12, 2026 16:49

odooNextev requested a review from monen17 March 12, 2026 16:52

stenext force-pushed the 18.0-imp-l10n_it_edi_doi_extension-migration_scripts branch from 5737200 to 136358b Compare March 12, 2026 17:07

eLBati approved these changes Mar 14, 2026

View reviewed changes

monen17 requested changes Mar 20, 2026

View reviewed changes

stenext and others added 2 commits April 7, 2026 13:33

[FIX] l10n_it_edi_doi_extension: sync l10n_it_edi_doi_id and fix avai…

bd8ec3a

…lable amount compute

[IMP] l10n_it_edi_doi_extension: add migration scripts

069a2dc

OCA-git-bot force-pushed the 18.0-imp-l10n_it_edi_doi_extension-migration_scripts branch from 136358b to 069a2dc Compare April 7, 2026 13:33

Uh oh!

Conversation

odooNextev commented Jul 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Migrazione da l10n_it_declaration_of_intent e fix di sincronizzazione

Obiettivo

Commit inclusi

1. [IMP] add migration scripts (5afe7901d)

Come testare

2. [FIX] sync l10n_it_edi_doi_id and fix available amount compute (d823abe29)

Come testare

Note per i revisori

Uh oh!

mmircoli-nexapp commented Feb 16, 2026

Uh oh!

odooNextev commented Feb 16, 2026

Uh oh!

mmircoli-nexapp commented Feb 16, 2026

Uh oh!

mmircoli-nexapp commented Feb 17, 2026

Uh oh!

francesco-ooops commented Feb 17, 2026

Uh oh!

OCA-git-bot commented Feb 17, 2026

Uh oh!

sergiocorato commented Feb 17, 2026

Uh oh!

mmircoli-nexapp commented Feb 17, 2026 via email

Uh oh!

odooNextev commented Feb 18, 2026

Uh oh!

mmircoli-nexapp commented Feb 18, 2026

Uh oh!

odooNextev commented Mar 3, 2026

Uh oh!

mmircoli-nexapp commented Mar 3, 2026

Uh oh!

monen17 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

eLBati left a comment

Choose a reason for hiding this comment

Uh oh!

eLBati commented Mar 17, 2026

Uh oh!

OCA-git-bot commented Mar 17, 2026

Uh oh!

monen17 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

monen17 Mar 20, 2026

Choose a reason for hiding this comment

Uh oh!

monen17 Mar 20, 2026

Choose a reason for hiding this comment

Uh oh!

monen17 Mar 20, 2026

Choose a reason for hiding this comment

Uh oh!

monen17 Mar 20, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

eLBati commented Apr 7, 2026

Uh oh!

odooNextev commented Jul 9, 2025 •

edited

Loading

1. `[IMP] add migration scripts` (`5afe7901d`)

2. `[FIX] sync l10n_it_edi_doi_id and fix available amount compute` (`d823abe29`)