Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0] [IMP] l10n_it_fatturapa_out: EUR values from journal #3284

Conversation

TheMule71
Copy link
Contributor

@TheMule71 TheMule71 commented Apr 21, 2023

Su suggerimento di @As400it , invece di convertire da valuta estera a EUR si va a prendere il valore nelle righe contabili.

Visto a posteriori il codice, non saprei dire quanto ne vale la pena, visto che la maggior parte dei valori convertiti non viene da righe contabili direttamente. Il riepilogo tasse viene ricalcolato, l'importo totale lo stesso, rimangono solo i subtotali di riga.

Vedi #3295

@TheMule71 TheMule71 force-pushed the 16.0-imp-l10n_it_fatturapa_out-eur-from-journal branch from fdda216 to 10d19b2 Compare April 21, 2023 11:12
@matteoopenf
Copy link
Contributor

@TheMule71 scusa 10d19b2 questo commit non sarebbe utile anche su 14?

@TheMule71
Copy link
Contributor Author

TheMule71 commented Apr 21, 2023

@TheMule71 scusa 10d19b2 questo commit non sarebbe utile anche su 14?

Anche l'altro mi sa.

Sarebbe tutto più utile se i valori usati nell'XML corrispondessero a quelli in fattura, ma di fatto ciò vale solo per le singole righe. Riepiloghi, totali, vengono tutti ricalcolati. L'idea era quella di non avere discrepanze ma di fatto queste ci sono comunque.

Considera che se mai si arrivarà ad aggregare le righe fattura per conto, anche quella corrispondenza verrà persa, e non ci sarà più nessun legame (diretto) tra le journal entry ed i valori esposti nell'XML (a meno di fare le aggregazioni manipolando le journal entry stesse, ovviamente).

@matteoopenf
Copy link
Contributor

@TheMule71 scusa 10d19b2 questo commit non sarebbe utile anche su 14?

Anche l'altro mi sa.

Sarebbe tutto più utile se i valori usati nell'XML corrispondessero a quelli in fattura, ma di fatto ciò vale solo per le singole righe. Riepiloghi, totali, vengono tutti ricalcolati. L'idea era quella di non avere discrepanze ma di fatto queste ci sono comunque.

Considera che se mai si arrivarà ad aggregare le righe fattura per conto, anche quella corrispondenza verrà persa, e non ci sarà più nessun legame (diretto) tra le journal entry ed i valori esposti nell'XML (a meno di fare le aggregazioni manipolando le journal entry stesse, ovviamente).

Grazie Mille del chiarimento

@MarcoCalcagni
Copy link
Contributor

Ciao @TheMule71 ,

nel file xml il riepilogo iva è sbagliato , viene riconvertito nuovamente .

per correggere dovresti modificare
<ImponibileImporto t-out="format_monetary(fpa_to_eur(tax_data['ImponibileImporto'], invoice=record), euro)"/>
<Imposta t-out="format_monetary(fpa_to_eur(tax_data['Imposta'], invoice=record), euro)" />
con
<ImponibileImporto t-out="format_monetary(tax_data['ImponibileImporto'], euro)"/>
<Imposta t-out="format_monetary(tax_data['Imposta'], euro)" />

questo evita la seconda conversione

@Borruso
Copy link
Contributor

Borruso commented Apr 26, 2023

Ciao @TheMule71 ,

l'errore c'è quando metto l'aliquota in fattura con valuta usd

@eLBati
Copy link
Member

eLBati commented Apr 28, 2023

Ho creato #3295

Copy link

github-actions bot commented May 5, 2024

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.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 5, 2024
@github-actions github-actions bot closed this Jun 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
16.0 stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

l10n_it_fatturapa_out - fattura in valuta estera con IVA
6 participants