Skip to content

[FIX] account_edi_ubl_cii: fix FacturX validation issues#219799

Closed
fw-bot wants to merge 1 commit intoodoo:saas-18.1from
odoo-dev:saas-18.1-17.0-opw-4907827-fix_invalid_facturx-mcou-449801-fw
Closed

[FIX] account_edi_ubl_cii: fix FacturX validation issues#219799
fw-bot wants to merge 1 commit intoodoo:saas-18.1from
odoo-dev:saas-18.1-17.0-opw-4907827-fix_invalid_facturx-mcou-449801-fw

Conversation

@fw-bot
Copy link
Copy Markdown
Contributor

@fw-bot fw-bot commented Jul 20, 2025

Issue:

There are several issues with the XML of FacturX.

Steps to reproduce:

  • Create a partner with a direct debit mandate
  • In the Accounting tab of the partner select "FacturX" as eInvoice format
  • Create an invoice with this partner, pay with SDD, confirm
  • Send to FacturX
  • Verify the PDF with https://www.portinvoice.com/
    1 Missing namespaces
  1. In the XML the code for the payment means does not adapt to the payment method: 42 (Payment to bank account) and not 59 (SDD)

Cause:

  1. The namespaces are not there (doc).
  2. We always input "42" as payment mean

Solution:

  1. Added namespaces xmlns:qdt="urn:un:unece:uncefact:data:standard:QualifiedDataType:100" and xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  2. Added the code "59" for SDD, a lot of codes could be added but this commit only handles this one

Ticket link
opw-4907827

Forward-Port-Of: #218686
Forward-Port-Of: #217763

Issue:
There are several issues with the XML of FacturX.

Steps to reproduce:
- Create a partner with a direct debit mandate
- In the Accounting tab of the partner select "FacturX" as eInvoice format
- Create an invoice with this partner, pay with SDD, confirm
- Send to FacturX
- Verify the PDF with https://www.portinvoice.com/
	- Missing namespace
- In the XML the code for the payment means does not adapt to the payment method: 42 (Payment to bank account) and not 59 (SDD)

Cause:
- The namespaces are not there (It's not mandatory).
- We always input "42" as payment mean

Solution:
- Added namespace `xmlns:qdt="urn:un:unece:uncefact:data:standard:QualifiedDataType:100"` and `xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"`
- Added the code "59" for SDD, as a lot of codes could be added we add a dictionnary that could be used to add codes. This commit only handles the SDD code

opw-4907827

X-original-commit: 17a1fc4
@robodoo
Copy link
Copy Markdown
Contributor

robodoo commented Jul 20, 2025

Pull request status dashboard

@fw-bot
Copy link
Copy Markdown
Contributor Author

fw-bot commented Jul 20, 2025

This PR targets saas-18.1 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Jul 20, 2025
robodoo pushed a commit that referenced this pull request Jul 21, 2025
Issue:
There are several issues with the XML of FacturX.

Steps to reproduce:
- Create a partner with a direct debit mandate
- In the Accounting tab of the partner select "FacturX" as eInvoice format
- Create an invoice with this partner, pay with SDD, confirm
- Send to FacturX
- Verify the PDF with https://www.portinvoice.com/
	- Missing namespace
- In the XML the code for the payment means does not adapt to the payment method: 42 (Payment to bank account) and not 59 (SDD)

Cause:
- The namespaces are not there (It's not mandatory).
- We always input "42" as payment mean

Solution:
- Added namespace `xmlns:qdt="urn:un:unece:uncefact:data:standard:QualifiedDataType:100"` and `xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"`
- Added the code "59" for SDD, as a lot of codes could be added we add a dictionnary that could be used to add codes. This commit only handles the SDD code

opw-4907827

closes #219799

X-original-commit: 17a1fc4
Signed-off-by: Paolo Gatti (pgi) <pgi@odoo.com>
Signed-off-by: Mathieu Coutant (mcou) <mcou@odoo.com>
@robodoo robodoo closed this Jul 21, 2025
@fw-bot fw-bot deleted the saas-18.1-17.0-opw-4907827-fix_invalid_facturx-mcou-449801-fw branch July 28, 2025 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

forwardport This PR was created by @fw-bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants