[18.0][IMP]l10n_it_delivery_note: electronic invoicing DatiDdt by matteotognini · Pull Request #5048 · OCA/l10n-italy

matteotognini · 2026-01-14T11:01:44Z

Il modulo aggiunge i DatiDdt all'xml della fattura elettronica. L'installazione del modulo porta alla disinstallazione automatica del modulo l10n_it_stock_ddt

risolve #5037

TheMule71 · 2026-01-14T13:44:52Z

Immagino tu sia partito da zero e non da #4054 ...
C'era in ballo la guestione dell'attribuzione del numero di riga nel caso di DDT multipli per la stessa fattura...

odooNextev · 2026-01-14T14:03:48Z

Immagino tu sia partito da zero e non da #4054 ... C'era in ballo la guestione dell'attribuzione del numero di riga nel caso di DDT multipli per la stessa fattura...

Ti rispondo io nel frattempo, poi appena riesce continua Matteo.
E' partito da 0 perchè non si poteva fare un porting essendo cambiato tutto lato edi e ci serve in breve tempo un modulo per inserire i DatiDDT che abbiamo in l10n_it_delivery_note perchè attualmente vengono inseriti nel XML solo se sono stati assegnati i campi di l10n_it_stock_ddt.
E' un problema grande perchè non penso che qualcuno di Odoo Italia usa l10n_it_stock_ddt.
Perciò per adempiere almeno ai dati obbligatori gli abbiamo detto di creare una bozza di modulo base che replicasse quello che fa l10n_it_stock_ddt, ma prendendo i dati di l10n_it_delivery_note, poi i dati delle righe arriveranno.

Il problema più grande è quello che ha scritto su Discord e per cui falliscono i test: vorremmo mettere un exclude per evitare che sia installato l10n_it_stock_ddt quando si installa questo, ma è un modulo auto_install che si installa con l10n_it_edi.
Infatti avendo l10n_it_stock_ddt e l10n_it_delivery_note installati in contemporanea creerebbe confusione perchè l'utente non saprebbe quali dati compilare e quali verranno presi in considerazione.

matteotognini · 2026-01-14T15:36:42Z

Ora ho rimosso l'esclusione del modulo l10n_it_ddt_stock, in modo che passino i test e lasciando indicazione che il modulo deve essere disinstallato a mano. Vediamo come procedere in modo da poter integrare anche il RiferimentoLinea

monen17

Grazie della PR!
Penso sarebbe meglio non andare in conflitto con il modulo di Odoo, magari adottando una sincronizzazione tipo quella di cui stiamo parlando con @TheMule71 in #5046, dici che è fattibile?

Perché a occhio qui si sta prendendo la stessa strada (ignorare quello che fa Odoo) che seguimmo per l10n_it_edi_related_document e ora ci stiamo accorgendo che dà problemi.

odooNextev · 2026-01-26T08:06:11Z

Grazie della PR! Penso sarebbe meglio non andare in conflitto con il modulo di Odoo, magari adottando una sincronizzazione tipo quella di cui stiamo parlando con @TheMule71 in #5046, dici che è fattibile?

Perché a occhio qui si sta prendendo la stessa strada (ignorare quello che fa Odoo) che seguimmo per l10n_it_edi_related_document e ora ci stiamo accorgendo che dà problemi.

Come avevo scritto sopra, l10n_it_stock_ddt e l10n_it_delivery_note non dovrebbero essere installati nella stessa istanza perchè non ha senso averli entrambi, è solo fuorviante per l'utente.
Se un utente installa l10n_it_delivery_note è perchè non è soddisfatto del modulo standard di Odoo...
Quello dei documenti collegati è un caso diverso perchè in quel caso estendiamo l10n_it_edi che ovviamente ci serve per altro e non si può disinstallare, mentre l10n_it_stock_ddt è un modulo a parte e non obbligatorio.
L'unico vero problema è che l10n_it_stock_ddt si installa in automatico con qualche dipendenza di l10n_it_delivery_note e bisognerebbe evitarlo.

Poi l'ultima miglioria sarebbe aggiungere uno script per migrare i dati di l10n_it_stock_ddt in l10n_it_delivery_note.

eLBati · 2026-03-26T16:45:41Z

+        for ddt in self.delivery_note_ids:
+            if not ddt.date or ddt.date != self.invoice_date:
+                return False
+        return super()._l10n_it_edi_invoice_is_direct()


chiamando super, se la fattura ha l10n_it_ddt_ids e le date corrispondono, ritorna True ; questo non produce una classificazione TD01/TD24 sbagliata? dovremmo considerare solo le date dei delivery note?

Hai ragione, non bisogna delegare al super, se la fattura ha delivery note è meglio usare quella.

eLBati · 2026-03-26T16:47:58Z

+                <attribute name="invisible">1</attribute>
+            </xpath>
+        </field>
+    </record>


Non dovremmo anche nascondere i campi di l10n_it_stock_ddt?

l10n_it_transport_reason

l10n_it_transport_method

l10n_it_transport_method_details

l10n_it_parcels

O lo fa già?

eLBati · 2026-03-26T16:50:09Z

+        "data/l10n_it_delivery_note_template.xml",
        "report/delivery_data.xml",
        "report/report_delivery_note.xml",
        "views/account_move.xml",


Il campo l10n_it_ddt_id di l10n_it_edi non è anche lui da nascondere, in quanto non più usato per l'XML?

Si, meglio nasconderlo

eLBati · 2026-03-26T16:52:12Z

+        "data/l10n_it_delivery_note_template.xml",
        "report/delivery_data.xml",
        "report/report_delivery_note.xml",
        "views/account_move.xml",


Inoltre: l10n_it_stock_ddt aggiunge un pulsante "DDTs" sulla fattura che mostra i picking. Mi pare che l10n_it_delivery_note aggiunga già il suo pulsante , quindi quello di l10n_it_stock_ddt sarebbe da nascondere

capito, nascondiamo anche quello

eLBati · 2026-03-26T16:55:04Z

+                    </t>
+                </DatiDDT>
+            </t>
+        </xpath>


E in DatiTrasporto attualmente cosa ci finisce?

MezzoTrasporto / CausaleTrasporto

NumeroColli / PesoLordo / PesoNetto

DataOraRitiro / DataInizioTrasporto

ci sono?

Non ho capito bene, ma il template principale è in l10n_it_edi_extension.

Il template in invoice_it_template.xml popola solo il sotto-tag di DatiTrasporto, ovvero IndirizzoResa, con:

indirizzo,

CAP,

comune,

provincia,

nazione.

Le voci che hai indicato non sono al suo interno.

sergiocorato · 2026-06-08T12:34:38Z

@matteotognini puoi fare un rebase?

NextEvTeo · 2026-06-08T13:14:17Z

@matteotognini puoi fare un rebase?

Fatto

NextEvTeo · 2026-06-12T12:39:15Z

+        for picking in self.filtered(lambda p: p.delivery_note_id):
+            picking.write({"l10n_it_ddt_number": picking.delivery_note_id.name})
+        return result
+


Aggiunto per far funzionare i filtri di odoo standard.

@eLBati puoi aggiornare la review?

…dditional states

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch 3 times, most recently from caf9120 to 5af4b5c Compare January 14, 2026 13:08

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch from 5af4b5c to 453ba39 Compare January 14, 2026 15:31

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch from 453ba39 to ed120b9 Compare January 14, 2026 15:39

matteotognini marked this pull request as draft January 14, 2026 15:51

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch 3 times, most recently from 1666b9b to f8377ae Compare January 15, 2026 08:45

monen17 reviewed Jan 23, 2026

View reviewed changes

monen17 mentioned this pull request Jan 23, 2026

l10n_fatturapa_out: output delivery note data in XML #4005

Open

3 tasks

monen17 linked an issue Jan 23, 2026 that may be closed by this pull request

l10n_fatturapa_out: output delivery note data in XML #4005

Open

3 tasks

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch from ba61c51 to 8ed4878 Compare February 25, 2026 15:50

matteotognini changed the title ~~[18.0][ADD]l10n_it_edi_delivery_note: electronic invoicing DatiDdt~~ [18.0][IMP]l10n_it_delivery_note: electronic invoicing DatiDdt Feb 25, 2026

matteotognini marked this pull request as ready for review March 4, 2026 10:15

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch 6 times, most recently from e1fa967 to b4c94cc Compare March 20, 2026 12:44

eLBati requested changes Mar 26, 2026

View reviewed changes

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch 2 times, most recently from 9e9c685 to 97e3916 Compare May 6, 2026 13:54

OCA-git-bot added mod:l10n_it_delivery_note Module l10n_it_delivery_note series:18.0 labels May 6, 2026

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch from 97e3916 to 29af4e4 Compare May 6, 2026 13:58

tafaRU mentioned this pull request Jun 5, 2026

[18.0] [IMP] l10n_it_delivery_note: DN integration with Electronic Invoice #4960

Closed

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch from 29af4e4 to 46373a4 Compare June 8, 2026 13:13

[IMP] l10n_it_delivery_note: electronic invoicing DatiDdt

36450d0

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch from 46373a4 to 36450d0 Compare June 12, 2026 12:27

NextEvTeo reviewed Jun 12, 2026

View reviewed changes

[FIX] l10n_it_delivery_note: Update delivery note filter to include a…

064ceb0

…dditional states

matteotognini force-pushed the 18.0-add-l10n_it_edi_dn branch from 82ec499 to 064ceb0 Compare June 12, 2026 14:29

Uh oh!

Conversation

matteotognini commented Jan 14, 2026 • edited by eLBati Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

TheMule71 commented Jan 14, 2026

Uh oh!

odooNextev commented Jan 14, 2026

Uh oh!

matteotognini commented Jan 14, 2026

Uh oh!

monen17 left a comment

Choose a reason for hiding this comment

Uh oh!

odooNextev commented Jan 26, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sergiocorato commented Jun 8, 2026

Uh oh!

NextEvTeo commented Jun 8, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

matteotognini commented Jan 14, 2026 •

edited by eLBati

Loading