Author: deepak
Date: Thu Nov  9 08:25:36 2017
New Revision: 1814708

URL: http://svn.apache.org/viewvc?rev=1814708&view=rev
Log:
Improved: Use FlexibleLocation.resolveLocation to resolve the fop.path and 
fop.font.base.url (OFBIZ-9974)

Modified:
    ofbiz/ofbiz-framework/trunk/framework/webapp/config/fop.properties
    
ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/view/ApacheFopWorker.java

Modified: ofbiz/ofbiz-framework/trunk/framework/webapp/config/fop.properties
URL: 
http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/webapp/config/fop.properties?rev=1814708&r1=1814707&r2=1814708&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/webapp/config/fop.properties 
(original)
+++ ofbiz/ofbiz-framework/trunk/framework/webapp/config/fop.properties Thu Nov  
9 08:25:36 2017
@@ -18,13 +18,13 @@
 ###############################################################################
 
 #Set fop.xconf path
-fop.path=/framework/webapp/config
+fop.path=component://webapp/config
 
 #Set default font family
 fop.font.family=NotoSans
 
 #Set font base url
-fop.font.base.url=/framework/webapp/config/
+fop.font.base.url=component://webapp/config/
 
 ###############################################################################
 # FOP Encryption Parameters

Modified: 
ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/view/ApacheFopWorker.java
URL: 
http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/view/ApacheFopWorker.java?rev=1814708&r1=1814707&r2=1814708&view=diff
==============================================================================
--- 
ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/view/ApacheFopWorker.java
 (original)
+++ 
ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/view/ApacheFopWorker.java
 Thu Nov  9 08:25:36 2017
@@ -105,14 +105,16 @@ public final class ApacheFopWorker {
                 }
 
                 try {
-                    String ofbizHome = System.getProperty("ofbiz.home");
-                    File userConfigFile = FileUtil.getFile(ofbizHome + fopPath 
+ "/fop.xconf");
+                    URL configFilePath = 
FlexibleLocation.resolveLocation(fopPath + "/fop.xconf");
+                    File userConfigFile = 
FileUtil.getFile(configFilePath.getFile());
                     if (userConfigFile.exists()) {
                         fopFactory = FopFactory.newInstance(userConfigFile);
                     } else {
                         Debug.logWarning("FOP configuration file not found: " 
+ userConfigFile, module);
                     }
-                    File fontBaseFile = FileUtil.getFile(ofbizHome + 
fopFontBaseProperty);
+                    URL fontBaseFileUrl = 
FlexibleLocation.resolveLocation(fopFontBaseProperty);
+                    File fontBaseFile = 
FileUtil.getFile(fontBaseFileUrl.getFile());
+
                     if (fontBaseFile.isDirectory()) {
                         
fopFactory.getFontManager().setResourceResolver(ResourceResolverFactory.createDefaultInternalResourceResolver(fontBaseFile.toURI()));
                     } else {


Reply via email to