This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new fffa395d17 Improved: Update to Apache Tika 3.0.0 (OFBIZ-13155)
fffa395d17 is described below

commit fffa395d17a930bfc6c10b6986876b48ebc63c6d
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Mon Oct 21 18:47:44 2024 +0200

    Improved: Update to Apache Tika 3.0.0 (OFBIZ-13155)
    
    This reverts commit 7c3a622f08e37d6a68125844383820dac1592f23.
    
    When running testSendOrderConfirmation and testFopMacroLibrary
    there are a lot of issues with dependencies related to
    org.apache.xmlgraphics Tika, FOP and PDFBox
    
    To be checked before going further using PDFBox 33.0.3 and FOP 2.10
    
    I'll put the the error stack for testSendOrderConfirmation in the Jira
---
 dependencies.gradle                                              | 9 ++++-----
 .../java/org/apache/ofbiz/entity/serialize/XmlSerializer.java    | 2 +-
 .../src/main/java/org/apache/ofbiz/security/SecuredUpload.java   | 3 +--
 3 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/dependencies.gradle b/dependencies.gradle
index 621b9b4327..f902364959 100644
--- a/dependencies.gradle
+++ b/dependencies.gradle
@@ -53,15 +53,14 @@ dependencies {
     implementation 'org.apache.logging.log4j:log4j-api:2.20.0' // the API of 
log4j 2
     implementation 'org.apache.logging.log4j:log4j-core:2.20.0' // Somehow 
needed by Buildbot to compile OFBizDynamicThresholdFilter.java
     implementation 'org.apache.poi:poi:5.3.0'
-    implementation 'org.apache.pdfbox:pdfbox:3.0.3'
+    implementation 'org.apache.pdfbox:pdfbox:2.0.32' // 3.0.1 does not compile
     implementation 'org.apache.shiro:shiro-core:1.13.0'
     implementation 'org.apache.shiro:shiro-crypto-cipher:2.0.0'
     implementation 'org.apache.sshd:sshd-core:2.13.1'
     implementation 'org.apache.sshd:sshd-sftp:2.13.1'
-    implementation 'jakarta.xml.bind:jakarta.xml.bind-api:4.0.2'
-    implementation 'org.apache.tika:tika-core:3.0.0'
-    implementation 'org.apache.tika:tika-parsers:3.0.0'
-    implementation 'org.apache.tika:tika-parser-pdf-module:3.0.0'
+    implementation 'org.apache.tika:tika-core:2.9.2'
+    implementation 'org.apache.tika:tika-parsers:2.9.2'
+    implementation 'org.apache.tika:tika-parser-pdf-module:2.9.2'
     implementation 'org.apache.cxf:cxf-rt-frontend-jaxrs:3.6.4' // 4.x+ 
requires javax.xml.bind -> jakarta.xml.bind namespace change
     implementation 'org.apache.tomcat:tomcat-catalina-ha:9.0.91' // Remember 
to change the version number (9 now) in javadoc block if needed.
     implementation 'org.apache.tomcat:tomcat-jasper:9.0.91'
diff --git 
a/framework/entity/src/main/java/org/apache/ofbiz/entity/serialize/XmlSerializer.java
 
b/framework/entity/src/main/java/org/apache/ofbiz/entity/serialize/XmlSerializer.java
index e7e9eadd04..ce954005a7 100644
--- 
a/framework/entity/src/main/java/org/apache/ofbiz/entity/serialize/XmlSerializer.java
+++ 
b/framework/entity/src/main/java/org/apache/ofbiz/entity/serialize/XmlSerializer.java
@@ -44,7 +44,7 @@ import java.util.TreeSet;
 import java.util.Vector;
 import java.util.WeakHashMap;
 
-import jakarta.xml.bind.DatatypeConverter;
+import javax.xml.bind.DatatypeConverter;
 import javax.xml.parsers.ParserConfigurationException;
 
 import org.apache.ofbiz.base.util.Debug;
diff --git 
a/framework/security/src/main/java/org/apache/ofbiz/security/SecuredUpload.java 
b/framework/security/src/main/java/org/apache/ofbiz/security/SecuredUpload.java
index 7e7670ed88..8e83b452bb 100644
--- 
a/framework/security/src/main/java/org/apache/ofbiz/security/SecuredUpload.java
+++ 
b/framework/security/src/main/java/org/apache/ofbiz/security/SecuredUpload.java
@@ -76,7 +76,6 @@ import org.apache.ofbiz.base.util.UtilValidate;
 import org.apache.ofbiz.base.util.UtilXml;
 import org.apache.ofbiz.entity.Delegator;
 import org.apache.ofbiz.entity.util.EntityUtilProperties;
-import org.apache.pdfbox.Loader;
 import org.apache.pdfbox.pdmodel.PDDocument;
 import org.apache.pdfbox.pdmodel.PDDocumentNameDictionary;
 import org.apache.pdfbox.pdmodel.PDEmbeddedFilesNameTreeNode;
@@ -531,7 +530,7 @@ public class SecuredUpload {
             }
             // OK no JS code, pass to check 2: detect if the document has any 
embedded files
             PDEmbeddedFilesNameTreeNode efTree = null;
-            try (PDDocument pdDocument = Loader.loadPDF(file)) {
+            try (PDDocument pdDocument = PDDocument.load(file)) {
                 PDDocumentNameDictionary names = new 
PDDocumentNameDictionary(pdDocument.getDocumentCatalog());
                 efTree = names.getEmbeddedFiles();
             }

Reply via email to