Skip to content

Fix issue with validation of invoices using SubInvoiceLines.#1153

Merged
langfr merged 1 commit into
ZUGFeRD:masterfrom
langfr:bugs/FACTUR-X_EXTENDED.xslt_250
Jun 23, 2026
Merged

Fix issue with validation of invoices using SubInvoiceLines.#1153
langfr merged 1 commit into
ZUGFeRD:masterfrom
langfr:bugs/FACTUR-X_EXTENDED.xslt_250

Conversation

@langfr

@langfr langfr commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

When using ZF_250 Extended invoices with SubInvoiceLines fail, because some rules do not take the LineStatusReasonCode into account, although mentioning that it should do so in the error message.

One mustangproject test fails:
LibraryTest.testSubInvoiceLineExportValidity:345 [XPath "/validation/xml/summary/@status" evaluated to value] expected:<"[]valid"> but was:<"[in]valid">

Thanks to @stefan6419846 for finding out the reason.

The header of the changed file contains his comment:

<!-- This is a copy of the original file available inside the archive at `Schema/4_Factur-X_1.09_EXTENDED/_XSLT_EXTENDED/FACTUR-X_EXTENDED.xslt`.
     It has been modified to match the updated Schematron file for the `BR-FXEXT-*-08(b|ini|rev)` rules.
     These were not updated with the "Corrigendum" issued on 2026-06-11 in the original distribution, see
     https://github.com/ZUGFeRD/mustangproject/issues/1141#issuecomment-4680523481 as well.

     Without these changes, the following examples fail to validate:
       * `4. EXTENDED/X02_SubInvoiceLines_Buero_Material_Bsp3`
       * `4. EXTENDED/X17_SubInvoiceLines_Hardware_Bsp2`
       * `4. EXTENDED/X20_SubInvoiceLines_Buero_Material_Bsp3__`

     Please note that these changes are not official and not acknowledged by upstream.

Added additional constraint

not(ram:AssociatedDocumentLineDocument/ram:LineStatusReasonCode) or ram:AssociatedDocumentLineDocument/ram:LineStatusReasonCode = 'DETAIL')

to the rules.

@langfr langfr merged commit ab66e6b into ZUGFeRD:master Jun 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant