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

[14.0][FIX] l10n_it_withholding_tax pagamento fatture multiple con registrazione spese bancarie #4235

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

Conversation

sergiocorato
Copy link
Contributor

@sergiocorato sergiocorato commented Jun 26, 2024

Soluzione aggiuntiva a #3781
Questa PR copre la casistica di un pagamento di una fattura con ritenuta con contemporanea registrazione di spese bancarie.

Nella situazione attuale una fattura registrata di 1000€ + 220€ IVA con ritenuta di 200€, con un debito verso il fornitore quindi di 1020€, se viene pagata per 1021€ genera una registrazione con un write-off di 199€.

Con questa PR viene generato un write-off corretto di 1€.

@sergiocorato sergiocorato force-pushed the 14.0-fix-l10n_it_withholding_tax-payment-with-writeoff branch from 09cad28 to 0b97a74 Compare June 26, 2024 20:40
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.

LGTM

@sergiocorato sergiocorato force-pushed the 14.0-fix-l10n_it_withholding_tax-payment-with-writeoff branch from de38523 to 004685e Compare June 28, 2024 12:30
@sergiocorato
Copy link
Contributor Author

Ho esteso la PR per coprire la casistica di un pagamento di diverse fatture con ritenuta di acconto con una spesa bancaria (quindi utilizzando l'opzione raggruppa pagamenti).

@sergiocorato sergiocorato force-pushed the 14.0-fix-l10n_it_withholding_tax-payment-with-writeoff branch from 3956c0b to 3e48c12 Compare June 28, 2024 13:16
@MaurizioPellegrinet
Copy link

Ciao, ho appena provato con il pagamento raggruppato. Sembrerebbe tutto a posto, anche nella generazione dei movimenti di rit.acc. però nelle due fatture che ho pagato mi ritrovo questo netto a pagare
image

@sergiocorato sergiocorato force-pushed the 14.0-fix-l10n_it_withholding_tax-payment-with-writeoff branch 2 times, most recently from 716d3b7 to 907bc73 Compare June 28, 2024 13:48
@sergiocorato
Copy link
Contributor Author

Ciao, ho appena provato con il pagamento raggruppato. Sembrerebbe tutto a posto, anche nella generazione dei movimenti di rit.acc. però nelle due fatture che ho pagato mi ritrovo questo netto a pagare image

Ho fatto diverse modifiche e aggiunto un test in quanto ho visto che non gestiva bene fatture di valori molto diversi, adesso mi risulta ok.

@sergiocorato sergiocorato changed the title [14.0][FIX] l10n_it_withholding_tax pagamento con spese Jul 1, 2024
@sergiocorato sergiocorato force-pushed the 14.0-fix-l10n_it_withholding_tax-payment-with-writeoff branch from 907bc73 to 0dd2936 Compare July 1, 2024 14:38
@sergiocorato
Copy link
Contributor Author

@MaurizioPellegrinet ho aggiornato il branch della PR, vedo che adesso non riesco a replicare l'errore, se vuoi riprovare.

@MaurizioPellegrinet
Copy link

Questo è quanto riscontro ora:
a. creo 2 fatture stesso fornitore con rit.acc.
b. pago con raggruppamento, con importo maggiorato (che faccio confluire su write-off)

  • le fatture risultano entrambe regolarmente pagate OK
  • su movimenti RdA le 2 rit.acc. sono correttamente indicate OK
  • su mastrino viene rilevata solo 1 delle 2 rit.acc. pertanto il fornitore rimane aperto ERRATO

Allego screen delle fatture, dei movimenti RdA, dei mastrini
pr4235.odt

@sergiocorato sergiocorato force-pushed the 14.0-fix-l10n_it_withholding_tax-payment-with-writeoff branch 9 times, most recently from 8bab699 to 8170b31 Compare July 8, 2024 16:53
@sergiocorato
Copy link
Contributor Author

sergiocorato commented Jul 8, 2024

Questo è quanto riscontro ora: a. creo 2 fatture stesso fornitore con rit.acc. b. pago con raggruppamento, con importo maggiorato (che faccio confluire su write-off)

* le fatture risultano entrambe regolarmente pagate OK

* su movimenti RdA le 2 rit.acc. sono correttamente indicate OK

* su mastrino viene rilevata solo 1 delle 2 rit.acc. pertanto il fornitore rimane aperto  ERRATO

Allego screen delle fatture, dei movimenti RdA, dei mastrini pr4235.odt

Dovrei aver corretto anche questa casistica. Da quanto ho provato, risulta corretto anche la rimozione delle riconciliazioni e il ri-collegamento, indipendentemente dall'ordine in cui le righe vengono collegate.

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.

Un gran bel lavoro: LGTM

@sergiocorato sergiocorato force-pushed the 14.0-fix-l10n_it_withholding_tax-payment-with-writeoff branch from 8170b31 to bb715e4 Compare July 11, 2024 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants