-
Notifications
You must be signed in to change notification settings - Fork 76
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.
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
Aspose library upgrade #5640
Aspose library upgrade #5640
Conversation
aspose/src/main/java/nl/nn/adapterframework/extensions/aspose/AsposeLicenseLoader.java
Outdated
Show resolved
Hide resolved
licenseWrapper.loadLicense(inputStream); | ||
log.info("loaded Aspose [{}] license", library.name()); | ||
} catch (Exception e) { | ||
log.error("failed to load Aspose [" + library.name() + "] license", e); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
log.error("failed to load Aspose [{}] license", library.name(), e);
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bedankt, over het hoofd gezien!
...va/nl/nn/adapterframework/extensions/aspose/services/conv/impl/CisConversionServiceImpl.java
Outdated
Show resolved
Hide resolved
...l/nn/adapterframework/extensions/aspose/services/conv/impl/convertors/PdfAttachmentUtil.java
Outdated
Show resolved
Hide resolved
.../nl/nn/adapterframework/extensions/aspose/services/conv/impl/convertors/SlidesConvertor.java
Outdated
Show resolved
Hide resolved
aspose/src/main/java/nl/nn/adapterframework/extensions/aspose/services/util/FileUtil.java
Outdated
Show resolved
Hide resolved
Now we have lots of failing tests
… `double` so there is a bit more feedback on how big the image differences are.
Add option to overwrite test-expectations to make it easier to re-calibrate output after updates.
…-xml; update test-specifications.
…s are released per library and we cannot really have one global "Aspose Version". At least this way all versions are together in a single block with link to the repo.
…services/conv/impl/convertors/PdfAttachmentUtil.java Co-authored-by: Niels Meijer <nielsmeijer@hotmail.com>
…services/conv/impl/CisConversionServiceImpl.java Co-authored-by: Niels Meijer <nielsmeijer@hotmail.com>
…slate an error message from Dutch to English.
18878f6
to
51faa6d
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs 66.9% Coverage The version of Java (11.0.21) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. |
Re-apply Aspose version upgrades and changes from @Laurens-makel against master.
This PR includes changes from and supercedes the following PRs:
This PR is against 7.9-release branch.