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

[fix]l10n_it_fatturapa_out risuzione problema fattura con anticipo #4381

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

matteoopenf
Copy link
Contributor

@matteoopenf matteoopenf commented Sep 24, 2024

Risolve #3670

Super seed di #3678

La differenza tra questa e l'altra è che nella vecchia veniva sempre sovrascritto il dizionario e in alcuni casi non funzionava quindi

@matteoopenf matteoopenf marked this pull request as draft September 24, 2024 16:01
@matteoopenf matteoopenf force-pushed the 16.0-FIX_fatturacon_anticipo branch 2 times, most recently from e7897cf to fb18fab Compare September 24, 2024 16:25
@matteoopenf matteoopenf marked this pull request as ready for review September 24, 2024 16:29
@matteoopenf matteoopenf changed the title [fix]l10n_it_fatturapa_out fix [fix]l10n_it_fatturapa_out risuzione problema fattura con anticipo Sep 24, 2024
before the fix  the dictionary it was always rewritten, now it update if the key of the tax is the same

Co-authored-by: Marcoapu <[email protected]>
Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Forse non mi è chiaro il funzionamento.

Steps:

  1. carico un ordine
  2. creo una fattura di acconto
  3. creo la fattura (con detrazione acconto)

Risultato:

  1. mi ritrovo con una fattura di acconto
  2. e una nota di credito per lo stesso imposto

Atteso:

  1. fattura di acconto
  2. fattura con indicato il totale detratto l'acconto (che venga accettata dallo SDI)

Allego video

PR4381.mp4

@matteoopenf
Copy link
Contributor Author

@MaurizioPellegrinet no.

I passaggi sono

Steps:

carico un ordine
creo una fattura di acconto
creo la fattura (con detrazione acconto)

Prima della pr la seconda fatture quella con detrazione di acconto veniva scarta, ora non viene scarta.
Questo accade se ci sono più righe nei movimenti contabili che riguardano la stessa imposta.
Perchè in quel caso il dizionario venendo ogni volta riscritto e non sommando i valori, i valori vengono poi errati.

@SirAionTech
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 16.0.

@matteoopenf
Copy link
Contributor Author

@SirAionTech vedo che il problema segnalato da @MaurizioPellegrinet compare anche dopo il rebase.
Tuttavia abbiamo visto in call su discord che compare anche nella tua pr di alcuni giorni fa.
Quindi il problema che crea una nota di credito è dovuto a qualcosa che sta altrove

Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional test: OK

@MaurizioPellegrinet
Copy link

Il problema da me indicato in seguito a verifica odierna in discord è legato all'impostazione del prodotto selezionato:
politica di fatturazione per la prova fatta deve essere impostato a "quantità ordinate",
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[16.0] l10n_it_fatturapa_out: Errore XML Su Fatture Con Anticipo
5 participants