You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2014/10/13 09:56:26 UTC
svn commit: r1631309 - in /ofbiz/branches/release13.07: ./
framework/base/lib/ framework/base/src/org/ofbiz/base/util/template/
framework/widget/src/org/ofbiz/widget/screen/
Author: jacopoc
Date: Mon Oct 13 07:56:25 2014
New Revision: 1631309
URL: http://svn.apache.org/r1631309
Log:
Applied fix from trunk:
===
Upgraded to the latest, now official, release of Freemarker 2.3.21
Added:
ofbiz/branches/release13.07/framework/base/lib/freemarker-2.3.21.jar
- copied unchanged from r1631300, ofbiz/trunk/framework/base/lib/freemarker-2.3.21.jar
Removed:
ofbiz/branches/release13.07/framework/base/lib/freemarker-2.3.19-null-wildcards.jar
Modified:
ofbiz/branches/release13.07/ (props changed)
ofbiz/branches/release13.07/.classpath
ofbiz/branches/release13.07/LICENSE
ofbiz/branches/release13.07/framework/base/src/org/ofbiz/base/util/template/FreeMarkerWorker.java
ofbiz/branches/release13.07/framework/widget/src/org/ofbiz/widget/screen/HtmlWidget.java
Propchange: ofbiz/branches/release13.07/
------------------------------------------------------------------------------
Merged /ofbiz/trunk:r1605029,1621683,1628130,1631299-1631300
Modified: ofbiz/branches/release13.07/.classpath
URL: http://svn.apache.org/viewvc/ofbiz/branches/release13.07/.classpath?rev=1631309&r1=1631308&r2=1631309&view=diff
==============================================================================
--- ofbiz/branches/release13.07/.classpath (original)
+++ ofbiz/branches/release13.07/.classpath Mon Oct 13 07:56:25 2014
@@ -24,7 +24,7 @@
<classpathentry kind="lib" path="framework/base/lib/clhm-release-1.0-lru.jar"/>
<classpathentry kind="lib" path="framework/base/lib/hamcrest-all-1.2.jar"/>
<classpathentry kind="lib" path="framework/base/lib/fop-1.1.jar"/>
- <classpathentry kind="lib" path="framework/base/lib/freemarker-2.3.19-null-wildcards.jar"/>
+ <classpathentry kind="lib" path="framework/base/lib/freemarker-2.3.21.jar"/>
<classpathentry kind="lib" path="framework/base/lib/httpclient-4.2.1.jar"/>
<classpathentry kind="lib" path="framework/base/lib/httpclient-cache-4.2.1.jar"/>
<classpathentry kind="lib" path="framework/base/lib/httpcore-4.2.1.jar"/>
Modified: ofbiz/branches/release13.07/LICENSE
URL: http://svn.apache.org/viewvc/ofbiz/branches/release13.07/LICENSE?rev=1631309&r1=1631308&r2=1631309&view=diff
==============================================================================
--- ofbiz/branches/release13.07/LICENSE (original)
+++ ofbiz/branches/release13.07/LICENSE Mon Oct 13 07:56:25 2014
@@ -19,6 +19,7 @@ framework/base/lib/avalon-framework-4.2.
framework/base/lib/batik-all-1.7.jar
framework/base/lib/barcode4j-2.1-barcode4j-fop-ext-complete.jar
framework/base/lib/clhm-release-1.0-lru.jar
+framework/base/lib/freemarker-2.3.21.jar
framework/base/lib/httpclient-4.2.1.jar
framework/base/lib/httpclient-cache-4.2.1.jar
framework/base/lib/httpcore-4.2.1.jar
@@ -448,7 +449,6 @@ BSD License:
applications/content/lib/dom4j-1.6.1.jar
framework/base/lib/antisamy-bin.1.2.jar
framework/base/lib/bsh-engine-modified.jar
-framework/base/lib/freemarker-2.3.19-null-wildcards.jar
framework/base/lib/hamcrest-all-1.2.jar
framework/base/lib/httpunit-1.7.jar
framework/base/lib/ical4j-1.0-rc2.jar
Modified: ofbiz/branches/release13.07/framework/base/src/org/ofbiz/base/util/template/FreeMarkerWorker.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release13.07/framework/base/src/org/ofbiz/base/util/template/FreeMarkerWorker.java?rev=1631309&r1=1631308&r2=1631309&view=diff
==============================================================================
--- ofbiz/branches/release13.07/framework/base/src/org/ofbiz/base/util/template/FreeMarkerWorker.java (original)
+++ ofbiz/branches/release13.07/framework/base/src/org/ofbiz/base/util/template/FreeMarkerWorker.java Mon Oct 13 07:56:25 2014
@@ -58,6 +58,7 @@ import freemarker.cache.TemplateLoader;
import freemarker.core.Environment;
import freemarker.ext.beans.BeanModel;
import freemarker.ext.beans.BeansWrapper;
+import freemarker.ext.beans.BeansWrapperBuilder;
import freemarker.template.Configuration;
import freemarker.template.SimpleHash;
import freemarker.template.SimpleScalar;
@@ -66,6 +67,7 @@ import freemarker.template.TemplateExcep
import freemarker.template.TemplateExceptionHandler;
import freemarker.template.TemplateModel;
import freemarker.template.TemplateModelException;
+import freemarker.template.Version;
/** FreeMarkerWorker - Freemarker Template Engine Utilities.
*
@@ -74,22 +76,19 @@ public class FreeMarkerWorker {
public static final String module = FreeMarkerWorker.class.getName();
+ public static final Version version = new Version(2, 3, 21);
+
// use soft references for this so that things from Content records don't kill all of our memory, or maybe not for performance reasons... hmmm, leave to config file...
private static final UtilCache<String, Template> cachedTemplates = UtilCache.createUtilCache("template.ftl.general", 0, 0, false);
- private static final BeansWrapper defaultOfbizWrapper = configureBeansWrapper(new BeansWrapper());
+ private static final BeansWrapper defaultOfbizWrapper = new BeansWrapperBuilder(version).build();
private static final Configuration defaultOfbizConfig = makeConfiguration(defaultOfbizWrapper);
public static BeansWrapper getDefaultOfbizWrapper() {
return defaultOfbizWrapper;
}
- public static <T extends BeansWrapper> T configureBeansWrapper(T wrapper) {
- wrapper.setNullWildcards(true);
- return wrapper;
- }
-
public static Configuration makeConfiguration(BeansWrapper wrapper) {
- Configuration newConfig = new Configuration();
+ Configuration newConfig = new Configuration(version);
newConfig.setObjectWrapper(wrapper);
newConfig.setSharedVariable("Static", wrapper.getStaticModels());
Modified: ofbiz/branches/release13.07/framework/widget/src/org/ofbiz/widget/screen/HtmlWidget.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release13.07/framework/widget/src/org/ofbiz/widget/screen/HtmlWidget.java?rev=1631309&r1=1631308&r2=1631309&view=diff
==============================================================================
--- ofbiz/branches/release13.07/framework/widget/src/org/ofbiz/widget/screen/HtmlWidget.java (original)
+++ ofbiz/branches/release13.07/framework/widget/src/org/ofbiz/widget/screen/HtmlWidget.java Mon Oct 13 07:56:25 2014
@@ -49,6 +49,7 @@ import freemarker.template.Template;
import freemarker.template.TemplateException;
import freemarker.template.TemplateModel;
import freemarker.template.TemplateModelException;
+import freemarker.template.Version;
/**
* Widget Library - Screen model HTML class.
@@ -58,10 +59,14 @@ public class HtmlWidget extends ModelScr
public static final String module = HtmlWidget.class.getName();
private static final UtilCache<String, Template> specialTemplateCache = UtilCache.createUtilCache("widget.screen.template.ftl.general", 0, 0, false);
- protected static Configuration specialConfig = FreeMarkerWorker.makeConfiguration(FreeMarkerWorker.configureBeansWrapper(new ExtendedWrapper()));
+ protected static Configuration specialConfig = FreeMarkerWorker.makeConfiguration(new ExtendedWrapper(FreeMarkerWorker.version));
// not sure if this is the best way to get FTL to use my fancy MapModel derivative, but should work at least...
public static class ExtendedWrapper extends BeansWrapper {
+ public ExtendedWrapper(Version version) {
+ super(version);
+ }
+
@SuppressWarnings("unchecked")
@Override
public TemplateModel wrap(Object object) throws TemplateModelException {