[16.0][IMP] l10n_it_delivery_note: create invoice from delivery notes with 1:1 lines correspondence

[odooNextev]

Al momento non è possibile creare una fattura direttamente da ddt come nelle precedenti versioni. (#3552)
Ho cercato di riportare il comportamento del modulo l10n_it_ddt della v10 in l10n_it_delivery_note alla v16 e permettere di fatturare i DN con associazione 1:1 tra le loro righe e quelle della fattura creata: #3979
Questa funzionalità è disponibile selezionando alcuni DN dalla tree view e creando la fattura dal wizard che appare nelle azioni, mentre le altre opzioni di fatturazione rimangono invariate.
Impostando una fatturazione con servizi nel wizard, verranno inclusi anche tutti i servizi da fatturare presenti nell'ordine di vendita a cui appartengono i DN.
Si possono selezionare solo DN dello stesso partner.

[OCA-git-bot]

Hi @MarcoCalcagni, @aleuffre, @renda-dev,
some modules you are maintaining are being modified, check this out!

[odooNextev]

@francesco-ooops c'è la issue, è la #3979 che ho messo nella descrizione
Se tu, @aleuffre, @Borruso, @MarcoCalcagni, @MaurizioPellegrinet e chiunque voglia valutarlo funzionalmente e nel codice mi dite che può essere un buon approccio, sostituisco questa PR nel task della v16 della issue con quella che avevo fatto precedentemente #4184 che aveva qualche problema tecnico (anche se funzionava).

[Borruso]

Grazie per la PR

ho notato che i test sono rossi
poiché non viene valorizzata la riga di sale line nelle invoice,
anche se nella _prepare_invoice_line la valorizzi.
Per il resto mi sembra ottima!

[odooNextev]

Grazie per la PR

ho notato che i test sono rossi poiché non viene valorizzata la riga di sale line nelle invoice, anche se nella _prepare_invoice_line la valorizzi. Per il resto mi sembra ottima!

Grazie della segnalazione, ma test fallivano perchè, modificando l'ordine della creazione delle righe delle fatture, dovevo cambiare anche gli indici delle liste.
Ora aggiungo anche un test ad hoc per la fatturazione da più DN dello stesso ordine con righe fattura corrispondenti a quelle dei DN.

Non ho capito bene se intendi che il campo non viene effettivamente valorizzato anche se ho messo questa impostazione in _prepare_invoice_line:
https://github.com/OCA/l10n-italy/pull/4385/files#diff-2ab35465a8828143ebe51cb5a35b6d98131aa057b1638efc13e04f8cb22916c0R219

[Borruso]

Grazie della segnalazione, ma test fallivano perchè, modificando l'ordine della creazione delle righe delle fatture, dovevo cambiare anche gli indici delle liste.
Ora aggiungo anche un test ad hoc per la fatturazione da più DN dello stesso ordine con righe fattura corrispondenti a quelle dei DN.

intendevo proprio questo ottimo attendo il test 👍

[Borruso]

LGTM

[odooNextev]

LGTM

Grazie 👍

[odooNextev]

@francesco-ooops potresti fare una review funzionale quando hai tempo?

[francesco-ooops]

Ciao, vedo che sia questa che #4375 hanno lo stesso obiettivo per le due diverse versioni, direi di concentrarci sull'altra che ho già testato estensivamente e poi portare il tutto, che ne dici?

[odooNextev]

@francesco-ooops va bene chiudere prima quella, però avevo provato a portare le modifiche così come sono in quella PR alla 16, ma non funzionava ed allora ho provato un approccio diverso.
Non ricordo più se tra la 14 e la 16 c'è differenza nella creazione delle fatture da SO o non si comportava come avrei voluto ed ho preferito tenere la modifica solo per il wizard.

[francesco-ooops]

@odooNextev cosa intendi con "ho preferito tenere la modifica solo per il wizard"?

[francesco-ooops]

@odooNextev non credo abbia senso avere due logiche diverse per la fatturazione da SO e da DDT, da qualsiasi parte fatturi (ddt singolo / più ddt / SO singolo / più SO) il risultato deve essere lo stesso

Questo il test più semplice possibile

SO: 1 riga, qty 2

Consegno con DDT 1 qty 1
Consegno con DDT 2 qty 1

Fatturo SO o entrambi i DDT insieme, risultato atteso:

DDT1
qty 1
DDT2
qty 1

Al momento è così invece:

[MaurizioPellegrinet]

Test funzionale: OK

[odooNextev]

cosa intendi con "ho preferito tenere la modifica solo per il wizard"?

@francesco-ooops il cliente per cui ho principalmente sviluppato questa funzionalità non fattura mai i DDT da SO, ma direttamente dalla lista come qui sotto:

Ho preferito mantenere inalterato invece il comportamento attuale per le altre modalità di fatturazione, come quella da SO, perchè qualcuno potrebbe preferirlo.
Dobbiamo capire se questa modalità non interessi più a nessuno, non è detto che qualcuno che fatturi un SO che ha prodotto dei DDT voglia collegarli 1:1 con la fattura.
Per me è uguale, ditemi voi.

Questo invece è come funziona la fatturazione da wizard:

Registrazione.schermo.2024-10-08.alle.14.37.09.webm