l10n_it_intrastat: modifica nome variabili

[TheMule71]

l10n-italy/l10n_it_intrastat/models/account.py

Lines 275 to 284 in 1fc7123

	amount_euro = invoice_currency._convert(
	amount_currency,
	company_currency,
	self.invoice_id.company_id,
	fields.Date.today())
	statistic_amount_euro = amount_euro
	res.update({
	'amount_currency': amount_currency,
	'amount_euro': amount_euro,
	'statistic_amount_euro': statistic_amount_euro})

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

#2228 (comment)

• 12.0
• 14.0
• 16.0

[sergiocorato]

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]

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.