[16.0] [FIX] l10n_it_account_stamp: messaggio errato dopo applicazione bollo + reimposta a bozza modifica l'addebito del bollo #4529
Conversation
primes2h
force-pushed
the
16.0-FIX-account_tax_stamp_behaviour
branch
5 times, most recently
from
e75bc23 to
5a78dff
Compare
| @@ -97,7 +96,6 @@ def add_tax_stamp_line(self): | |||
| inv.write({"invoice_line_ids": [(0, 0, invoice_line_vals)]}) | |||
|
|
|||
| def is_tax_stamp_line_present(self): | |||
| self.ensure_one() | |||
There was a problem hiding this comment.
Perché rimuovere ensure_one()?
Il test ha senso per più fatture? Ossia serve sapere che tra N fatture almeno una abbia una riga che sia un bollo?
Se è una cosa che ha senso sapere fattura per fattura (a senso mi pare così), allora l'ensure_one ci va e se scatena un errore il problema è nel chiamante...
There was a problem hiding this comment.
Perché rimuovere
ensure_one()?
In realtà era solo un revert alla situazione precedente, era stato aggiunto dalla PR #3802.
Se è una cosa che ha senso sapere fattura per fattura (a senso mi pare così), allora l'ensure_one ci va e se scatena un errore il problema è nel chiamante...
Attualmente nel codice viene sempre passata una singola fattura a is_tax_stamp_line_present(), però ha senso che ci sia il controllo. Riaggiunto.
primes2h
force-pushed
the
16.0-FIX-account_tax_stamp_behaviour
branch
from
5a78dff to
4fd4a38
Compare
primes2h
force-pushed
the
16.0-FIX-account_tax_stamp_behaviour
branch
from
4fd4a38 to
0bc6e43
Compare
primes2h
added
the
no stale
primes2h
force-pushed
the
16.0-FIX-account_tax_stamp_behaviour
branch
from
0bc6e43 to
e3548c4
Compare
primes2h
force-pushed
the
16.0-FIX-account_tax_stamp_behaviour
branch
from
e3548c4 to
b2e994a
Compare
Risolve #4528 per 16.0.
Vedi anche #4414 (comment) e relativa review della PR.