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 2012/05/24 18:03:30 UTC

svn commit: r1342308 - in /ofbiz/trunk/framework: common/webcommon/WEB-INF/common-controller.xml webtools/webapp/webtools/WEB-INF/controller.xml

Author: jacopoc
Date: Thu May 24 16:03:29 2012
New Revision: 1342308

URL: http://svn.apache.org/viewvc?rev=1342308&view=rev
Log:
Removed the definition of the FreeMarkerViewHandler handler from common-controller and added to the only controller that still uses it (the one of the "webtools" component, and only for one test page).
The main motivation for this is that FreeMarkerViewHandler creates (via OfbizCacheStorage) an unused static instance of UtilCache per object (i.e. for each webapp at init time), wasting system resources and cluttering the list of cache objects in Webtools.

Modified:
    ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml
    ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/controller.xml

Modified: ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml?rev=1342308&r1=1342307&r2=1342308&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml (original)
+++ ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml Thu May 24 16:03:29 2012
@@ -46,7 +46,6 @@ under the License.
     <handler name="screentext" type="view" class="org.ofbiz.widget.screen.ScreenTextViewHandler"/-->
     <handler name="screenfop" type="view" class="org.ofbiz.widget.screen.ScreenFopViewHandler"/>
     <handler name="jsp" type="view" class="org.ofbiz.webapp.view.JspViewHandler"/>
-    <handler name="ftl" type="view" class="org.ofbiz.webapp.ftl.FreeMarkerViewHandler"/>
     <handler name="http" type="view" class="org.ofbiz.webapp.view.HttpViewHandler"/>
     <handler name="birt" type="view" class="org.ofbiz.birt.webapp.view.BirtViewHandler"/>
 

Modified: ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/controller.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/controller.xml?rev=1342308&r1=1342307&r2=1342308&view=diff
==============================================================================
--- ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/controller.xml (original)
+++ ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/controller.xml Thu May 24 16:03:29 2012
@@ -26,7 +26,8 @@ under the License.
     <include location="component://common/webcommon/WEB-INF/tempexpr-controller.xml"/>
     <include location="component://testtools/webapp/testtools/WEB-INF/controller.xml"/>
     <description>WebTools Site Configuration File</description>
-    
+
+    <handler name="ftl" type="view" class="org.ofbiz.webapp.ftl.FreeMarkerViewHandler"/>
     <handler name="datavision" type="view" class="org.ofbiz.webapp.view.DataVisionViewHandler"/>
 
     <!-- Events to run on every request before security (chains exempt) -->