[16.0][FIX] l10n_it_split_payment: multi-taxes

[TennyMkt]

risolve #4945
(e sostituisce #4783)

La principale novità consiste nel modificare la funzione _build_writeoff_line in modo che non calcoli lo storno di una sola linea tax ma costruisca lo storno di tutte le imposte dell'account.move. Per questo viene spostata da account.move.line a account.move (e si modificano le traduzioni delle stringhe della funzione).
Inoltre per gestire la casistica di una fattura Split Payment di cui viene modificato il regime fiscale, sarebbe necessario eliminare la riga di "Split Payment Write Off" ma, avendo display_type = tax, non è possibile quindi viene azzerata in modo da sistemare i conteggi delle linee.

and line.move_id.is_sale_document(include_receipts=True)
è stato tolto in modo che vengano gestite anche le fatture fornitore.

[francesco-ooops]

@micheledic che ne pensi?

[TennyMkt]

Ho aggiornato la PR aggiungendo anche il test che ha proposto @micheledic in #4783

[micheledic]

Fatto un giro su runbot, funziona!

[micheledic]

@micheledic che ne pensi?

continuiamo con questa, mi sembra più completa!

[TennyMkt]

Ho aggiornato la PR per aggiungere nella write della account.move :

        if "fiscal_position_id" in vals:
            self._compute_amount()

perchè altrimenti nella casistica in cui una fattura in un regime fiscale "non split payment" viene modificato il regime fiscale impostandolo come "Split Payment" non verrebbe aggiunta la riga di storno se amount_sp non è aggiornato

[eLBati]

Grazie @TennyMkt
.po files non dovrebbero essere modificati in PR, se ne occupa poi weblate.

Per il resto 👍

[TennyMkt]

Grazie @TennyMkt .po files non dovrebbero essere modificati in PR, se ne occupa poi weblate.

Per il resto 👍

Aggiorno la PR togliendo le modifiche ai file .po o posso lasciarla così?

[francesco-ooops]

@TennyMkt la cartella non dovrebbe essere inclusa nella PR

[TennyMkt]

PR Aggiornata togliendo modifiche in i18n

[francesco-ooops]

@eLBati merge?

[TennyMkt]

Facendo ulteriori test è emerso che utilizzando l'Azione Trasforma in fattura/nota di credito da una fattura Split Payment, la riga di storno si duplica.

La problematica sembra causata dalla funzione write aggiunta in account.move.line.py:

def write(self, vals):
    res = super().write(vals)
    move_to_compute_sp = []
    for line in self:
        if (
            (line.move_id not in move_to_compute_sp)
            and line.move_id.split_payment
            and line.display_type == "tax"
            and not line.is_split_payment
        ):
            move_to_compute_sp.append(line.move_id)
    for move in move_to_compute_sp:
        move.compute_split_payment()
    return res

Facendo test anche senza questa fare compute_split_payment() ogni volta che viene modificata una riga della fattura, lo storno viene aggiornato ugualmente.

Stiamo facendo ulteriori test prima di aggiornare la PR

[TennyMkt]

Facendo ulteriori test è emerso che utilizzando l'Azione Trasforma in fattura/nota di credito da una fattura Split Payment, la riga di storno si duplica.

La problematica sembra causata dalla funzione write aggiunta in account.move.line.py:

def write(self, vals):
    res = super().write(vals)
    move_to_compute_sp = []
    for line in self:
        if (
            (line.move_id not in move_to_compute_sp)
            and line.move_id.split_payment
            and line.display_type == "tax"
            and not line.is_split_payment
        ):
            move_to_compute_sp.append(line.move_id)
    for move in move_to_compute_sp:
        move.compute_split_payment()
    return res

Facendo test anche senza questa fare compute_split_payment() ogni volta che viene modificata una riga della fattura, lo storno viene aggiornato ugualmente.

Stiamo facendo ulteriori test prima di aggiornare la PR

Aggiornata la PR togliendo l'estensione della write di account.move.line.py perchè da ulteriori test risulta inutile anzi dannosa nel caso di utilizzo di "Trasforma in fattura/nota di credito"

[danall59]

Il modulo è funzionante ed è un grosso passo aventi rispetto alla situzione attuale che non gestisce la fatture con più imposte, non gestisce correttamente la trasformazione in note credito, la generazione fattura da ordine e altro.
Abbiamo fatto numerosi test e Il modulo è ora in produzione su alcuni nostri clienti con situazioni complesse.

[MaurizioPellegrinet]

Test funzionale: OK

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

[TheMule71]

_sync_dynamic_line è un @contextmanager credo sarebbe meglio usarlo tipo così:

https://github.com/odoo/odoo/blob/bb60d831a7cdb0606c1baae9a8985b2fdb7dc15b/addons/account/models/account_move_line.py#L1615-L1617

[TennyMkt]

PR aggiornata inserendo gli aggiornamenti in righe
with move._sync_dynamic_lines(container={"records": move}):
però ho dovuto anche aggiungere dei
with_context(skip_split_payment_computation=True)
Perchè la _sync_dynamic_lines fa delle write e quindi farebbe dei loop infiniti

[TennyMkt]

Ho aggiornato la PR per risolvere un altro problema dovuto da questo modulo:
siccome anche la funzione _check_balanced è un @contextmanager e viene chiamata senza with, non viene controllato lo sbilanciamento delle registrazioni (nemmeno quelle senza split payment)

[TheMule71]

LGTM

[tafaRU]

/ocabot merge patch

[OCA-git-bot]

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

[OCA-git-bot]

It looks like something changed on 16.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 16.0-ocabot-merge-pr-4947-by-tafaRU-bump-patch, awaiting test results.

[OCA-git-bot]

It looks like something changed on 16.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 16.0-ocabot-merge-pr-4947-by-tafaRU-bump-patch, awaiting test results.

[OCA-git-bot]

It looks like something changed on 16.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 16.0-ocabot-merge-pr-4947-by-tafaRU-bump-patch, awaiting test results.

[OCA-git-bot]

It looks like something changed on 16.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 16.0-ocabot-merge-pr-4947-by-tafaRU-bump-patch, awaiting test results.

[OCA-git-bot]

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