[FIX] l10n_it_reverse_charge: force amount_currency set in line values

[odooNextev]

Ho provato a correggere in maniera un po' empirica il problema segnalato nella issue: #4111

Dopo un po' di test sembra reggere, ma di sicuro necessita di reviews.

[odooNextev]

@Borruso @eLBati @francesco-ooops

[eLBati]

Si sovrappone mica a #3879 ?

[MaurizioPellegrinet]

Functional test: OK

[odooNextev]

@SirAionTech ho aggiunto un test giusto per cominciare, poi bisognerà aggiungere dei check sugli importi e non solo sulla creazione e riconciliazione come adesso.
Mi sono fermato perchè non riesco vorrei capire meglio lo scopo di with_supplier_self_invoice nelle posizioni fiscali: mi sembra di capire che nei test è usato per le fatture extra EU, ma secondo i miei colleghi non serve un autofattura addizionale neanche in questo caso.

[Borruso]

LGTM

[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). 🤖

[SirAionTech]

@SirAionTech ho aggiunto un test giusto per cominciare, poi bisognerà aggiungere dei check sugli importi e non solo sulla creazione e riconciliazione come adesso. Mi sono fermato perchè non riesco vorrei capire meglio lo scopo di with_supplier_self_invoice nelle posizioni fiscali: mi sembra di capire che nei test è usato per le fatture extra EU, ma secondo i miei colleghi non serve un autofattura addizionale neanche in questo caso.

Se secondo te non è pronta per essere mergiata, meglio che la metti in bozza (vedi https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/changing-the-stage-of-a-pull-request#converting-a-pull-request-to-a-draft).

Per capire meglio with_supplier_self_invoice, magari @eLBati che lo aggiunse in 9ad9b4f può chiarirti le idee.

[odooNextev]

@SirAionTech hai ragione e la metto in bozza, ma se non ho risposte a breve la rivalido pronta per il merge perchè comunque funziona ed il test l'ho fatto.
Quello che vorrei è un chiarimento sull'interpretazione di quel campo per sviluppare più approfonditamente il test visto che i miei colleghi che si occupano della contabilità lo usano in maniera differente dai test ed il modulo funziona lo stesso.
Ma immagino che le tempistiche si allungheranno parecchio ed al momento senza questa fix il modulo non funziona correttamente.

[eLBati]

Mi sono fermato perchè non riesco vorrei capire meglio lo scopo di with_supplier_self_invoice

@odooNextev sono riuscito a risponderti in chiamata?

[odooNextev]

Mi sono fermato perchè non riesco vorrei capire meglio lo scopo di with_supplier_self_invoice

@odooNextev sono riuscito a risponderti in chiamata?

Sì e no, nel senso che con ho capito perchè si usa questo campo, ma con questa configurazione faccio fatica a fare un test che abbia un senso per giustificare questa modifica

[eLBati]

@odooNextev quindi non riesci a replicare questo #4111 nei test?

[odooNextev]

@odooNextev quindi non riesci a replicare questo #4111 nei test?

Non riesco con la posizione fiscale nel file common dei test fiscal_position_extra perchè ha with_supplier_self_invoice a True e non mi genera le stesse registrazioni a cui sono "abituato".
Dovrei creare delle altre configurazioni come metto di test nell'ultimo commit

[odooNextev]

@eLBati @SirAionTech ho replicato una registrazione di fattura extra EU in valuta estera con la configurazione che usiamo internamento nel test che ho aggiunto facendo verificare gli importi sia in valuta della fattura che in quelli dell'azienda (che è poi quello che doveva risolvere la PR)

[eLBati]

Resterebbe solo da verificare , per sicurezza, che anche in caso di with_supplier_self_invoice il problema non si presenti. Ma per me si può procedere

[odooNextev]

Si può mergiare?

[SirAionTech]

Grazie della PR!

[SirAionTech]

/ocabot merge patch

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-4238-by-SirAionTech-bump-patch, awaiting test results.

[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). 🤖

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-4238-by-SirAionTech-bump-patch, awaiting test results.

[OCA-git-bot]

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