l10n-italy icon indicating copy to clipboard operation
l10n-italy copied to clipboard

l10n_it_fatturapa_out_rc: riferimenti duplicati in "Documenti correlati"

Open vincenzoterzulli opened this issue 2 years ago • 1 comments

Modulo

l10n_it_fatturapa_out_rc

Descrizione del problema

Se si annulla e rivalida una fattura fornitore EXTRA UE precedentemente validata, nella sezione "Documenti correlati" dell'autofattura di vendita vengono duplicati i riferimenti alla fattura fornitore di partenza.

image

I dati duplicati vengono così riportati nella fattura elettronica dell'autofattura di vendita.

image

Riproduzione del problema

Versioni coinvolte:

  • [ ] v8 (nell'attuale versione della PR #2855)
  • [ ] v12
  • [ ] v14

Passi per riprodurre il problema

  1. Creare una fattura fornitore EXTRA UE e validarla
  2. Verificare nell'autofattura di vendita la presenza dei riferimenti alla fattura fornitore (fin qui tutto ok)
  3. Annullare, reimpostare a bozza e validare nuovamente la fattura fornitore di partenza
  4. Verificare nell'autofattura di vendita la presenza dei riferimenti duplicati alla fattura fornitore

Comportamento atteso Riferimenti non duplicati

Informazioni aggiuntive Il problema è stato osservato durante i test funzionali della PR 2855 che porta dalla v12 alla v8 le ultime correzioni riguardanti il reverse charge. Al momento in cui scrivo il problema risulta essere di sicuro presente anche nella v12 (versione di origine del backport) ed è causato dal modo con cui vengono creati i riferimenti dei documenti correlati nell'autofattura di vendita:

https://github.com/OCA/l10n-italy/blob/05138da0b5ae75c72c7bacd4ae05a3014d3e02a3/l10n_it_fatturapa_out_rc/models/account_invoice.py#L8-L26

Una possibile soluzione (attualmente in prova) prevede di eliminare tutti i riferimenti a documenti correlati prima della creazione dei nuovi, tramite la seguente istruzione: self.rc_self_invoice_id.related_documents = [(5, False, False)]

image

vincenzoterzulli avatar Aug 11 '22 15:08 vincenzoterzulli

per v8 la soluzione proposta è questa: https://github.com/OCA/l10n-italy/commit/4da29f188b39843103840dc116d569079baa3a89

vincenzoterzulli avatar Aug 12 '22 15:08 vincenzoterzulli