[12.0] FIX l10n_it_reverse_charge: reconcile_supplier_invoice setting currency to avoid redundant exchange_diff_move when not needed #3183
Conversation
eLBati
changed the title
| @@ -233,6 +233,7 @@ def rc_payment_credit_line_vals(self, invoice): | |||
| 'account_id': self.get_rc_inv_line_to_reconcile( | |||
| invoice).account_id.id, | |||
| 'partner_id': invoice.partner_id.id, | |||
| 'currency_id': self.currency_id.id, | |||
There was a problem hiding this comment.
Forse:
| 'currency_id': self.currency_id.id, | |
| 'currency_id': invoice.currency_id.id, |
…change_diff_move when not needed
eLBati
force-pushed
the
12.0-fix-reconcile_supplier_invoice-currency
branch
from
7722539 to
50101b4
Compare
There was a problem hiding this comment.
Grazie della PR!
Per facilitare la tracciatura delle correzioni/migliorie sulle varie versioni, puoi aprire una issue aggiungendo la descrizione, tramite use case, del problema? (parafrasando https://github.com/OCA/l10n-italy/wiki/Team-di-sviluppo#apertura-issue)
Hai modo di aggiungere un test?
Puoi adattare il messaggio del commit a https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message? Ad esempio: ci sarebbe da menzionare il nome del modulo modificato e la prima riga andrebbe accorciata (puoi mettere tutti i dettagli nelle righe successive).
|
@eLBati può essere che c'è anche nella 16?
|
@odooNextev sì: #3864 |
| @@ -211,6 +210,7 @@ def rc_debit_line_vals(self, amount=None): | |||
| 'credit': credit, | |||
| 'account_id': self.get_inv_line_to_reconcile().account_id.id, | |||
| 'partner_id': self.partner_id.id, | |||
| 'currency_id': self.currency_id.id, | |||
There was a problem hiding this comment.
Da fare lo stesso su rc_credit_line_vals()?
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
#3863