search

OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
149 stars 303 forks source link

l10n_it_intrastat: modifica nome variabili #2510

Open TheMule71 opened 2 years ago

TheMule71 commented 2 years ago

https://github.com/OCA/l10n-italy/blob/1fc7123eac6c0ec936a02f77431ea3c6ef8960ab/l10n_it_intrastat/models/account.py#L275-L284

Sebbene sia pressoché una questione estetica, il codice si basa sull'assunzione implicita che la valuta della company sia euro, sarebbe meglio prendere direttamente euro e usare quello per le conversioni, oppure cambiare nome alle variabili per essere un po' più agnostiche. Vd. anche

https://github.com/OCA/l10n-italy/pull/2228#discussion_r734567124

sergiocorato commented 2 years ago

Concordo che se si dà per scontato che siano Euro, sia meglio prendere direttamente la valuta ignorando l'impostazione dell'azienda. Per correttezza sarebbe da far apparire un errore quando l'utente tenta di inserire una valuta diversa dall'Euro all'azienda se sono installati i moduli intrastat.

github-actions[bot] commented 3 months ago

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.