-
-
Notifications
You must be signed in to change notification settings - Fork 309
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
[14.0][IMP] supporto per la conversione degli importi in EUR per fatture in valuta estera #2563
Conversation
testata funzionante. |
<Divisa>EUR</Divisa> | ||
<Data>2021-12-16</Data> | ||
<Numero>INV/2021/12/0001</Numero> | ||
<ImportoTotaleDocumento>14.00</ImportoTotaleDocumento> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eventuale alternativa: Campo Divisa
in USD e ImportoTotaleDocumento
in USD e il resto tutto uguale
1b351d0
to
d3f3aff
Compare
Implementate entrambe le opzioni. Ho aggiunto una cosa, le info dei pagamenti (che non c'erano) e ho deciso di trasformare in EUR anche quelli se la Divisa viene forzata a EUR, lasciare gli importi originali altrimenti. Aggiunto anche un test per il caso. Cortesemente validatemi i due XML: |
Fatto. |
63f9df7
to
9ac46a1
Compare
Gli XML sono validi per ADE. |
Io l'ho provata e direi ok anche dal mio punto di vista |
Ho provato in locale e quando genero un file xml mi viene questo errore se manca la città del cliente. Traceback (most recent call last): The above exception was the direct cause of the following exception: Traceback (most recent call last): Error to render compiling AST |
Quando è impostato a EUR tutto Ok |
Il problema è di fatturapa_out, non riguarda il codice di questa PR, ma già che ci siamo (visto che ho già in canna un commit separato di fix su problema analogo, integro anche questa fix). Edit: concetto giusto, PR sbagliata, la fix la metto in #2606. |
Come mai 2 commit? |
Il commit sarebbe uno, ma ho separato le modifiche per modulo uno per fatturapa_out e l'altro per fatturapa_out_sp. #2559 ha spaccato un paio di cose, ma lo sapevamo. Va fatto un refactoring. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Funziona come concordato in ambedue le casistiche
9ac46a1
to
7648b2c
Compare
Adesso sistema fatturapa_out, fatturapa_out_sp e fatturapa_out_rc. A proposito, toccando 3 moduli, ho splittato il commit in 3 parti. Questo dovrebbe agevolare il porting futuro (se non altro in fase di |
@eLBati riesci a fare la review di questa? |
…i in EUR per fatture in valuta estera
…orti in EUR per fatture in valuta estera
59c10af
to
480f766
Compare
…orti in EUR per fatture in valuta estera
env.ref("product.product_product_10") is a demo data we use in tests Prior to commit 86febae278f08864e83017d43f6aa9d67165d664 in odoo the original price (14.0 USD) was taken as EUR (from the company currency), and converted to 16.38 USD when added to a invoice with currency USD. This module used to convert it back to 14.0 EUR. After 86febae278f08864e83017d43f6aa9d67165d664, the product is added to a invoice with currency == USD with the correct price, 14.00, which is later converted to 11.97 EUR by this module.
480f766
to
e1875f3
Compare
@TheMule71 Gli importi non vanno convertiti in EURO, vanno lasciati in valuta indicando la divisa e il cambio utilizzati |
scusate se arrivo solo adesso. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
review tecnica 👍
La discussione c'è da 1 anno e passa. Vd. #2042. La parte rilevante è:
|
Si ho visto rileggendo bene tutto :)! |
Quindi approvata? |
Stò facendo la review |
Riesci a collegarti su discord? Noi avevamo il dito sul merge... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Mi pare ok 👍 |
/ocabot merge patch |
On my way to merge this fine PR! |
Congratulations, your PR was merged at d1a2fe4. Thanks a lot for contributing to OCA. ❤️ |
Vd. discussione in #2042, in particolare #2042 (comment)
Qui l'XML viene generato con Divisa EUR e gli importi convertiti. Per le righe fattura in AltriDatiGestionali vengono riportati i valori originali.
#2590
Aggionamento
Include #2753 e #2763