search

OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
142 stars 297 forks source link

[MIG] l10n_it_sepa_bonifici -> l10n_it_sct_cbi #4189

Open SirAionTech opened 4 weeks ago

SirAionTech commented 4 weeks ago

Migrazione da https://github.com/OCA/l10n-italy/tree/ce2daa663db1e2e009dc85021fac2f6379238f43/l10n_it_sepa_bonifici (8.0).

Il modulo è stato in buona parte riscritto come conseguenza dei cambiamenti nella dipendenza account_banking_pain_base, ad esempio https://github.com/OCA/bank-payment/commit/574c258f9ef1cf18691b3d70379c4eafec4b7560.

Ho inglesizzato il nome del modulo, non ho fatto script di migrazione perché credo che nessuno voglia fare una migrazione da 8.0 a 16.0; se vuole farlo, i dati di questo modulo sono il minore dei suoi problemi. Per le traduzioni ho fatto un semplice find/replace, lascerei il compito il più possibile ai nostri ottimi traduttori :innocent:.

Il modulo migrato rispetta i nuovi schema in vigore dal 18/03/2024 ^1. È possibile verificare la validità dei file generati su https://www.cbiservice.com/correttoresepa.php, selezionando l'opzione: SCT - CBIBdyPaymentRequest 00.04.01 (new).

Fatemi sapere cosa ne pensate :smile:

SirAionTech commented 4 weeks ago

I test falliscono per https://github.com/OCA/l10n-italy/pull/4192, ho aggiunto la sua fix https://github.com/OCA/l10n-italy/pull/4193 come dipendenza

SirAionTech commented 2 weeks ago

/ocabot migration l10n_it_sepa_bonifici

francesco-ooops commented 2 weeks ago

@SirAionTech rilancia pure

SirAionTech commented 2 weeks ago

@SirAionTech rilancia pure

Cosa? Il comando

/ocabot migration l10n_it_sepa_bonifici

era andato bene

SirAionTech commented 2 weeks ago

Fatto rebase dopo il merge di https://github.com/OCA/l10n-italy/pull/4193.

francesco-ooops commented 2 weeks ago

@matteoopenf dato che è il porting di un modulo creato da openforce, potresti fare review? Grazie! :)

MaurizioConte commented 1 week ago

image Test Pagamento a Fornitore quando si genera il file si riceve l'errore in slide

SirAionTech commented 1 week ago

image Test Pagamento a Fornitore quando si genera il file si riceve l'errore in slide

Grazie della revisione! Puoi riportare i passi che hai fatto per ottenere l'errore? Oppure un video se ti viene più comodo. Nel log dovresti poi trovare il contenuto del file prodotto, puoi riportare anche quello?

MaurizioConte commented 1 week ago

https://dinamicheaziendalisrl-my.sharepoint.com/:v:/g/personal/mconte_dinamicheaziendali_it/EawmBl4H1M9PvGWET9KlYtcBBqe-sgMVz-eC7zuU8jSCnw?e=nN6Z8y&nav=eyJyZWZlcnJhbEluZm8iOnsicmVmZXJyYWxBcHAiOiJTdHJlYW1XZWJBcHAiLCJyZWZlcnJhbFZpZXciOiJTaGFyZURpYWxvZy1MaW5rIiwicmVmZXJyYWxBcHBQbGF0Zm9ybSI6IldlYiIsInJlZmVycmFsTW9kZSI6InZpZXcifX0%3D

E' chiaro che il file non è stato generato

MaurizioConte commented 1 week ago

Grazie della revisione! Puoi riportare i passi che hai fatto per ottenere l'errore? Oppure un video se ti viene più comodo. Nel log dovresti poi trovare il contenuto del file prodotto, puoi riportare anche quello?

Lo sto provando su runbot e non so da dove prendere il log e non sono nemmeno tecnico

SirAionTech commented 1 week ago

E' chiaro che il file non è stato generato

Il contenuto del file è stato generato e stampato nel log, lo dice anche il messaggio di errore che hai mostrato in https://github.com/OCA/l10n-italy/pull/4189#issuecomment-2189282023:

The generated XML file and the full error have been written in the server logs.

Il file non è stato poi presentato all'utente perché non era corretto; però nel log dovresti comunque trovarne il contenuto, puoi riportarlo qui?

SirAionTech commented 1 week ago

Grazie della revisione! Puoi riportare i passi che hai fatto per ottenere l'errore? Oppure un video se ti viene più comodo. Nel log dovresti poi trovare il contenuto del file prodotto, puoi riportare anche quello?

Lo sto provando su runbot e non so da dove prendere il log e non sono nemmeno tecnico

Ok se è su runboat allora lo posso recuperare io, riporto qui sotto l'ultimo errore che vedo:

2024-06-25 16:02:51,592 18 WARNING bedca60bd-a74f-41aa-9da7-e999d50bc8f8 odoo.addons.account_banking_pain_base.models.account_payment_order: The XML file is invalid against the XML Schema Definition 
2024-06-25 16:02:51,592 18 WARNING bedca60bd-a74f-41aa-9da7-e999d50bc8f8 odoo.addons.account_banking_pain_base.models.account_payment_order: b'<?xml version=\'1.0\' encoding=\'UTF-8\'?>\n<CBIBdyPaymentRequest xmlns="urn:CBI:xsd:CBIBdyPaymentRequest.00.04.01" xmlns:PMRQ="urn:CBI:xsd:CBIPaymentRequest.00.04.01">\n  <CBIEnvelPaymentRequest>\n    <CBIPaymentRequest>\n      <PMRQ:GrpHdr>\n        <PMRQ:MsgId>PAY0001</PMRQ:MsgId>\n        <PMRQ:CreDtTm>2024-06-25T16:02:51</PMRQ:CreDtTm>\n        <PMRQ:NbOfTxs>1</PMRQ:NbOfTxs>\n        <PMRQ:CtrlSum>1725.00</PMRQ:CtrlSum>\n        <PMRQ:InitgPty>\n          <PMRQ:Nm>My Company (San Francisco)</PMRQ:Nm>\n          <PMRQ:Id>\n            <PMRQ:OrgId>\n              <PMRQ:Othr>\n                <PMRQ:Id>0130502B</PMRQ:Id>\n                <PMRQ:Issr>CBI</PMRQ:Issr>\n              </PMRQ:Othr>\n            </PMRQ:OrgId>\n          </PMRQ:Id>\n        </PMRQ:InitgPty>\n      </PMRQ:GrpHdr>\n      <PMRQ:PmtInf>\n        <PMRQ:PmtInfId>PAY0001-20240625-NORM</PMRQ:PmtInfId>\n        <PMRQ:PmtMtd>TRF</PMRQ:PmtMtd>\n        <PMRQ:BtchBookg>false</PMRQ:BtchBookg>\n        <PMRQ:PmtTpInf>\n          <PMRQ:InstrPrty>NORM</PMRQ:InstrPrty>\n          <PMRQ:SvcLvl>\n            <PMRQ:Cd>SEPA</PMRQ:Cd>\n          </PMRQ:SvcLvl>\n        </PMRQ:PmtTpInf>\n        <PMRQ:ReqdExctnDt>\n          <PMRQ:Dt>2024-06-25</PMRQ:Dt>\n        </PMRQ:ReqdExctnDt>\n        <PMRQ:Dbtr>\n          <PMRQ:Nm>My Company (San Francisco)</PMRQ:Nm>\n          <PMRQ:PstlAdr>\n            <PMRQ:Ctry>IT</PMRQ:Ctry>\n            <PMRQ:AdrLine>Via Milano, 1</PMRQ:AdrLine>\n          </PMRQ:PstlAdr>\n        </PMRQ:Dbtr>\n        <PMRQ:DbtrAcct>\n          <PMRQ:Id>\n            <PMRQ:IBAN>FR7642424242424242424242424</PMRQ:IBAN>\n          </PMRQ:Id>\n        </PMRQ:DbtrAcct>\n        <PMRQ:DbtrAgt>\n          <PMRQ:FinInstnId>\n            <PMRQ:BIC>PSSTFRPPXXX</PMRQ:BIC>\n            <PMRQ:ClrSysMmbId>\n              <PMRQ:MmbId>24242</PMRQ:MmbId>\n            </PMRQ:ClrSysMmbId>\n          </PMRQ:FinInstnId>\n        </PMRQ:DbtrAgt>\n        <PMRQ:ChrgBr>SLEV</PMRQ:ChrgBr>\n        <PMRQ:CdtTrfTxInf>\n          <PMRQ:PmtId>\n            <PMRQ:InstrId>19</PMRQ:InstrId>\n            <PMRQ:EndToEndId>19</PMRQ:EndToEndId>\n          </PMRQ:PmtId>\n          <PMRQ:PmtTpInf>\n            <PMRQ:CtgyPurp>\n              <PMRQ:Cd>SEPA</PMRQ:Cd>\n            </PMRQ:CtgyPurp>\n          </PMRQ:PmtTpInf>\n          <PMRQ:Amt>\n            <PMRQ:InstdAmt Ccy="EUR">1725.00</PMRQ:InstdAmt>\n          </PMRQ:Amt>\n          <PMRQ:CdtrAgt>\n            <PMRQ:FinInstnId>\n              <PMRQ:BIC>CPHBBE75</PMRQ:BIC>\n            </PMRQ:FinInstnId>\n          </PMRQ:CdtrAgt>\n          <PMRQ:Cdtr>\n            <PMRQ:Nm>B2C Customer IT</PMRQ:Nm>\n            <PMRQ:PstlAdr>\n              <PMRQ:Ctry>IT</PMRQ:Ctry>\n              <PMRQ:AdrLine>Via Roma, 1</PMRQ:AdrLine>\n            </PMRQ:PstlAdr>\n          </PMRQ:Cdtr>\n          <PMRQ:CdtrAcct>\n            <PMRQ:Id>\n              <PMRQ:IBAN>BE61310126985517</PMRQ:IBAN>\n            </PMRQ:Id>\n          </PMRQ:CdtrAcct>\n          <PMRQ:RmtInf>\n            <PMRQ:Ustrd>CCC</PMRQ:Ustrd>\n          </PMRQ:RmtInf>\n        </PMRQ:CdtTrfTxInf>\n      </PMRQ:PmtInf>\n    </CBIPaymentRequest>\n  </CBIEnvelPaymentRequest>\n</CBIBdyPaymentRequest>\n' 
2024-06-25 16:02:51,592 18 WARNING bedca60bd-a74f-41aa-9da7-e999d50bc8f8 odoo.addons.account_banking_pain_base.models.account_payment_order: Element '{urn:CBI:xsd:CBIPaymentRequest.00.04.01}BIC': This element is not expected. Expected is one of ( {urn:CBI:xsd:CBIPaymentRequest.00.04.01}BICFI, {urn:CBI:xsd:CBIPaymentRequest.00.04.01}ClrSysMmbId )., line 49 
2024-06-25 16:02:51,594 18 WARNING bedca60bd-a74f-41aa-9da7-e999d50bc8f8 odoo.http: The generated XML file is not valid against the official XML Schema Definition. The generated XML file and the full error have been written in the server logs. Here is the error, which may give you an idea on the cause of the problem : Element '{urn:CBI:xsd:CBIPaymentRequest.00.04.01}BIC': This element is not expected. Expected is one of ( {urn:CBI:xsd:CBIPaymentRequest.00.04.01}BICFI, {urn:CBI:xsd:CBIPaymentRequest.00.04.01}ClrSysMmbId )., line 49

Giusto per info, lo trovi in https://runboat.odoo-community.org/api/v1/builds/bedca60bd-a74f-41aa-9da7-e999d50bc8f8/log

primes2h commented 2 days ago

@SirAionTech Questa PR va ad implementare https://github.com/OCA/l10n-italy/issues/3203 come alternativa a https://github.com/OCA/l10n-italy/pull/3301?

francesco-ooops commented 2 days ago

@SirAionTech Questa PR va ad implementare #3203 come alternativa a #3301?

@primes2h chiusa la vecchia PR e linkata la nuova

SirAionTech commented 2 days ago

@SirAionTech Questa PR va ad implementare #3203 come alternativa a #3301?

Io l'ho vista più come migrazione del modulo in oggetto, però sì direi che la issue #3203 viene risolta da questo modulo, quindi si può vedere anche così. Nota che #3301 è per 14.0 mentre questa è per 16.0.

primes2h commented 2 days ago

@SirAionTech Questa PR va ad implementare #3203 come alternativa a #3301?

Io l'ho vista più come migrazione del modulo in oggetto, però sì direi che la issue #3203 viene risolta da questo modulo, quindi si può vedere anche così. Nota che #3301 è per 14.0 mentre questa è per 16.0.

Si certo, intendevo come modulo alternativo. Grazie!

primes2h commented 2 days ago

@SirAionTech Questa PR va ad implementare #3203 come alternativa a #3301?

@primes2h chiusa la vecchia PR e linkata la nuova

:+1:

SirAionTech commented 2 days ago

image Test Pagamento a Fornitore quando si genera il file si riceve l'errore in slide

@MaurizioConte Puoi riprovare e lasciare una revisione? Se non sai come fare trovi delle istruzioni in https://www.odoo-italia.org/documentazione/14.0/sviluppo/review.html#revisione-funzionale.