[12.0] l10n_it_ddt: fix invoice generation

[SimoRubi]

Descrizione del problema o della funzionalità
Fix nella creazione delle fatture da DDT multipli + refactor (commit separato)

Comportamento attuale prima di questa PR
Durante la creazione delle fatture da diversi DDT, alcune righe di un DDT potrebbero risultare ad un'altra fattura creata precedentemente.

Comportamento desiderato dopo questa PR
Tutte le righe dello stesso DDT sono raccolte nella stessa fattura.

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

[eLBati]

@SimoRubi potresti creare la issue con i passi per riprodurre?

[tafaRU]

Per il resto LGTM.
Grazie!

[SimoRubi]

@SimoRubi potresti creare la issue con i passi per riprodurre?

Ho provato a riprodurre l'errore con un test automatico ma non sono riuscito perché non conosco i passi per arrivare alla situazione che poi porta l'errore, quando posso mi ci metto.

[SimoRubi]

@eLBati l'ordine dei commit qui su Github è quel che è, comunque ho 3 commit in questa PR:

1. Aggiungo un test, che fallisce
2. Fix per il test
3. Refactoring

E qui c'è la issue: #1622

[eLBati]

LGTM

[SimoRubi]

@tafaRU puoi aggiornare la tua review? Così se anche per te è ok, potremmo andare avanti con il merge

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[SimoRubi]

@OCA/local-italy-maintainers che dite si può procedere al merge o devo fare altre modifiche? Grazie

[eLBati]

/ocabot merge minor

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-1606-by-eLBati-bump-minor, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 9c99e7d. Thanks a lot for contributing to OCA. ❤️