Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add support for PDFBOX 3.0 #1311

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Add support for PDFBOX 3.0 #1311

wants to merge 7 commits into from

Conversation

@tresf
Copy link
Contributor

@tresf tresf commented Jan 29, 2025 •
edited
Loading

Switches the PDFBOX library from 2.x to 3.x

TODO:

  • Regression testing
  • Update workarounds for PDFBOX 3.x

@akberenz can you help me with the workaround? getContext() is null and it's expected to be passed in now.

tray/src/qz/printer/action/pdf/ParamPdfRenderer.java

Lines 41 to 44 in b8cab39

// Note: These must match PdfFontPageDrawer's ignoreTransparency condition
// TODO: These will throw NPE: Fix for PDFBOX 3.x
addOperator(new OpaqueDrawObject(null));
addOperator(new OpaqueGraphicStateParameters(null));

@tresf tresf added this to the 2.2.5 milestone Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@akberenz akberenz

@tresf tresf

Labels

enhancement pdfbox

Projects

None yet

Milestone

2.2.5

Development

Successfully merging this pull request may close these issues.

3 participants

@tresf @akberenz