[16.0] [FIX] l10n_it_split_payment: when editing invoice lines, move lines are not correctly computed #4233
Conversation
There was a problem hiding this comment.
Grazie della PR!
Puoi modificare il messaggio del commit in modo che segua le linee guida in https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message?
In particolare:
please check if the commit message is cut with ellipsis
Puoi aggiungere un test? Così si evitano regressioni.
| return res | ||
| for move in self: | ||
| if move.split_payment: | ||
| split_payment_total = 0 |
There was a problem hiding this comment.
Questo viene dichiarato qui e sommato poco sotto, ma non trovo dove viene usato: a cosa serve?
| write_off_line_vals = line._build_writeoff_line() | ||
| split_payment_total += write_off_line_vals["debit"] | ||
| line_sp = fields.first( | ||
| line.move_id.line_ids.filtered( |
There was a problem hiding this comment.
| line.move_id.line_ids.filtered( | |
| move.line_ids.filtered( |
| float_compare( | ||
| line_sp.price_unit, | ||
| write_off_line_vals["price_unit"], | ||
| precision_rounding=line.move_id.currency_id.rounding, |
There was a problem hiding this comment.
| precision_rounding=line.move_id.currency_id.rounding, | |
| precision_rounding=move.currency_id.rounding, |
| ): | ||
| line_sp.write(write_off_line_vals) | ||
| else: | ||
| if line.move_id.amount_sp: |
There was a problem hiding this comment.
| if line.move_id.amount_sp: | |
| if move.amount_sp: |
| if line.display_type == "tax" and not line.is_split_payment: | ||
| write_off_line_vals = line._build_writeoff_line() | ||
| split_payment_total += write_off_line_vals["debit"] | ||
| line_sp = fields.first( |
There was a problem hiding this comment.
Per ogni riga cerca tra tutte le righe della move e troverà sempre la stessa line_sp, non sarebbe meglio salvarla? Così almeno evitiamo di rifare la ricerca e il filtered che tra l'altro ora è anche su un campo calcolato.
Se poi riuscissimo anche a fare la scrittura fuori dal loop non sarebbe male.
In pratica, ora che siamo nella move si potrebbe riscrivere un po' questo loop per evitare di rifare le stesse operazioni per ogni riga: ad esempio ottenere in un colpo solo l'importo da mettere nel writeoff da tutte le righe che non sono is_split_payment, visto che _build_writeoff_line prende quasi tutti i dati dalla move; cosa ne pensi?
There was a problem hiding this comment.
Ho tirato fuori line_sp ma non mi è chiara l'altra proposta
| if line.move_id.amount_sp: | ||
| line.move_id.with_context( | ||
| skip_split_payment_computation=True | ||
| ).line_ids = [(0, 0, write_off_line_vals)] |
There was a problem hiding this comment.
Rispetto al codice rimosso sotto non c'è più _sync_dynamic_lines, sai mica come mai non serve più?
There was a problem hiding this comment.
Dovrebbe essere lui a scrivere le move lines indesiderate
|
|
||
|
|
||
| class AccountMoveLine(models.Model): | ||
| _inherit = "account.move.line" | ||
|
|
||
| is_split_payment = fields.Boolean() | ||
| is_split_payment = fields.Boolean(compute="_compute_is_split_payment") |
There was a problem hiding this comment.
Si potrebbe rendere storato e dipendente da account_id, che ne dici?
| if line.move_id.amount_sp: | ||
| line.move_id.with_context( | ||
| skip_split_payment_computation=True | ||
| ).line_ids = [(0, 0, write_off_line_vals)] |
There was a problem hiding this comment.
| ).line_ids = [(0, 0, write_off_line_vals)] | |
| ).line_ids = [Command.create(write_off_line_vals)] |
eLBati
force-pushed
the
16.0-fix-split_payment
branch
from
f6b5fe7
to
0e6a669
Compare
Fatto
Al momento non riesco |
eLBati
force-pushed
the
16.0-fix-split_payment
branch
from
0e6a669
to
c34606a
Compare
When editing invoice lines, move lines are not correctly computed
eLBati
force-pushed
the
16.0-fix-split_payment
branch
from
c34606a
to
1b72988
Compare
#4231