[16.0][FIX] l10n_it_account_stamp: add stamp by comparing price in company currency #3713
Conversation
Borruso
changed the title
|
@Borruso l'obiettivo di questa PR è risolvere l'errore nel test InvoicingTest.test_amount_total_changing_currency ? E' da fare anche per la 14? |
credo di si, l'altro giorno a una persona si rompeva a una persona su 14 leggevo su discord |
|
ok, in tal caso @Borruso apri una issue per favore |
Borruso
force-pushed
the
16.0-fix-l10n_it_account_stamp-company_currency
branch
3 times, most recently
from
781d53d
to
1082b19
Compare
|
@Borruso puoi aprire la issue di tracciamento in cui descrivi il problema che questa PR vuole correggere? Grazie. |
Borruso
force-pushed
the
16.0-fix-l10n_it_account_stamp-company_currency
branch
from
1082b19
to
43c6024
Compare
| inv_tax.price_subtotal | ||
| abs(inv_tax.balance) |
There was a problem hiding this comment.
Si suppone quindi che la valuta dell'azienda sia EUR, ma se non fosse così?
Credo sarebbe più corretto controllare prima che la valuta dell'azienda sia EUR, oppure controllare il valore convertito in EUR.
There was a problem hiding this comment.
balance è nella valuta della company che non per forza deve essere EUR
stamp_apply_min_total_base questo campo in prodotto è un float che dovresti scrivere in base alla tua company, forse è proprio questo campo che ti può far supporre che l'azienda sia EUR
| @@ -32,14 +32,13 @@ def is_tax_stamp_applicable(self): | |||
| return total_tax_base >= stamp_product_id.stamp_apply_min_total_base | |||
|
|
|||
| @api.depends( | |||
| "invoice_line_ids.price_subtotal", | |||
| "line_ids.price_total", | |||
| "line_ids.balance", | |||
| "currency_id", | |||
There was a problem hiding this comment.
| "currency_id", |
Probabile che anche questo non serva più
| price_unit = inv.company_currency_id._convert( | ||
| stamp_product_id.list_price, inv.currency_id, inv.company_id, inv.date | ||
| ) |
There was a problem hiding this comment.
Credo che il campo list_price del prodotto sia nella valuta del prodotto quindi andrebbe convertito dalla valuta del prodotto alla valuta della fattura, analogamente a quanto fatto in https://github.com/odoo/odoo/blob/03d2e58f0d9e3beaf6ff8c4486675b4613b88283/addons/sale/models/product_template.py#L266-L269
Borruso
force-pushed
the
16.0-fix-l10n_it_account_stamp-company_currency
branch
from
43c6024
to
c68d778
Compare
|
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
Borruso
force-pushed
the
16.0-fix-l10n_it_account_stamp-company_currency
branch
from
c68d778
to
b5fd4b1
Compare
github-actions
bot
removed
the
stale
--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing