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] XML di pagamento rifiutato: Valore SEPA non è un codice valido per Category Purpose #4379

Merged

Conversation

SirAionTech
Copy link
Contributor

Passi:

  1. Installare e configurare l10n_it_sct_cbi
  2. Inviare un XML di pagamento alla banca

Attuale:
Il file viene rifiutato:
image

Atteso:
Il file viene accettato

Altre info:
Nella documentazione1 è scritto:

Il campo Code del campo Category Purpose (<CtgyPurp>) deve assumere uno dei valori presenti nella lista esterna disponibile all'indirizzo http://www.iso20022.org/external_code_list.page. (“NARR”, “Category Purpose non valida”)

Dalla pagina linkata è possibile scaricare la lista dei codici accettati, tra questi c'è SUPP che viene descritto come:

Transaction is related to a payment to a supplier.

che direi essere abbastanza generico, quindi uso questo invece di SEPA per correggere.

La prima scelta sarebbe stata rimuovere il nodo CtgyPurp perché da schema è opzionale:

<xs:element name="CtgyPurp" type="CategoryPurpose1Choice" minOccurs="0"/>

però sempre nella documentazione1 è indicato come obbligatorio quando si fanno pagamenti verso IBAN italiani:

Il blocco Category Purpose deve essere obbligatoriamente presente in caso di IBAN del Creditor Account radicato su IT (primi due chrt IBAN = IT) (“NARR”, “Category Purpose non presente”)

Footnotes

  1. https://www.cbi-org.eu/My-Menu/Servizio-CBI-Documentazione/Servizio-CBI-Documentazione-Standard, il link è già nel README 2

@SirAionTech SirAionTech marked this pull request as ready for review September 19, 2024 09:19
@francesco-ooops
Copy link
Contributor

@SirAionTech se è stata già utilizzata in produzione dai vostri clienti fai fare review a un funzionale e mergiamo pure 👍

Copy link
Contributor

@andreampiovesana andreampiovesana left a comment

Choose a reason for hiding this comment

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

LGTM

@SirAionTech
Copy link
Contributor Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-4379-by-SirAionTech-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 466116d into OCA:16.0 Oct 7, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 9076c72. Thanks a lot for contributing to OCA. ❤️

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

Successfully merging this pull request may close these issues.

4 participants