Skip to content

#2432 Add missing case in the switch for PDFA3U. Validation is ok cor… - #2433

Merged
merks merged 1 commit into
eclipse-birt:masterfrom
giangianoulas:#2432-PDF.A3U
Jun 16, 2026
Merged

#2432 Add missing case in the switch for PDFA3U. Validation is ok cor…#2433
merks merged 1 commit into
eclipse-birt:masterfrom
giangianoulas:#2432-PDF.A3U

Conversation

@giangianoulas

@giangianoulas giangianoulas commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Added the missing case. Validation does not complain now and it should fix #2432

      <validationReport jobEndStatus="normal" profileName="PDF/A-3u validation profile" statement="PDF file is compliant with Validation Profile requirements." isCompliant="true">
        <details passedRules="148" failedRules="0" passedChecks="26513" failedChecks="0"></details>
      </validationReport>

@hvbtup hvbtup left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Might be a silly question, but did you also check other places in the code where PDFA* is used for decisions and which should be extended? Maybe there are some, I don't know.

That said, this change certainly cannot do harm and obviously improves PDF/A-3u support.

@giangianoulas

Copy link
Copy Markdown
Contributor Author

It is not really that silly because I did not check prior to this PR. I did check now and it seems to the only place it was missing

@merks
merks merged commit 9b3c26a into eclipse-birt:master Jun 16, 2026
3 checks 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.

Validation failure for PDF.A3U

3 participants