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/29 15:46:50 UTC
svn commit: r1343724 -
/ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/BsfEventHandler.java
Author: jacopoc
Date: Tue May 29 13:46:49 2012
New Revision: 1343724
URL: http://svn.apache.org/viewvc?rev=1343724&view=rev
Log:
Improved code that manages the cache:
* protected the UtilCache object (static field) by making it private and final
* removed unnecessary synchronization
Modified:
ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/BsfEventHandler.java
Modified: ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/BsfEventHandler.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/BsfEventHandler.java?rev=1343724&r1=1343723&r2=1343724&view=diff
==============================================================================
--- ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/BsfEventHandler.java (original)
+++ ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/BsfEventHandler.java Tue May 29 13:46:49 2012
@@ -43,7 +43,7 @@ import org.ofbiz.webapp.control.ConfigXM
public class BsfEventHandler implements EventHandler {
public static final String module = BsfEventHandler.class.getName();
- public static UtilCache<String, String> eventCache = UtilCache.createUtilCache("webapp.BsfEvents");
+ private static final UtilCache<String, String> eventCache = UtilCache.createUtilCache("webapp.BsfEvents");
/**
* @see org.ofbiz.webapp.event.EventHandler#init(javax.servlet.ServletContext)
@@ -85,38 +85,30 @@ public class BsfEventHandler implements
cacheName = event.invoke;
scriptString = eventCache.get(cacheName);
if (scriptString == null) {
- synchronized(eventCache) {
- if (scriptString == null) {
- if (Debug.verboseOn()) {
- Debug.logVerbose("Loading BSF Script at location: " + cacheName, module);
- }
- URL scriptUrl = FlexibleLocation.resolveLocation(cacheName);
- if (scriptUrl == null) {
- throw new EventHandlerException("BSF script not found at location [" + cacheName + "]");
- }
- scriptStream = scriptUrl.openStream();
- scriptString = IOUtils.getStringFromReader(new InputStreamReader(scriptStream));
- scriptStream.close();
- eventCache.put(cacheName, scriptString);
- }
+ if (Debug.verboseOn()) {
+ Debug.logVerbose("Loading BSF Script at location: " + cacheName, module);
}
+ URL scriptUrl = FlexibleLocation.resolveLocation(cacheName);
+ if (scriptUrl == null) {
+ throw new EventHandlerException("BSF script not found at location [" + cacheName + "]");
+ }
+ scriptStream = scriptUrl.openStream();
+ scriptString = IOUtils.getStringFromReader(new InputStreamReader(scriptStream));
+ scriptStream.close();
+ scriptString = eventCache.putIfAbsentAndGet(cacheName, scriptString);
}
} else {
// we are a script in the webapp - load by resource
cacheName = context.getServletContextName() + ":" + event.path + event.invoke;
scriptString = eventCache.get(cacheName);
if (scriptString == null) {
- synchronized(eventCache) {
- if (scriptString == null) {
- scriptStream = context.getResourceAsStream(event.path + event.invoke);
- if (scriptStream == null) {
- throw new EventHandlerException("Could not find BSF script file in webapp context: " + event.path + event.invoke);
- }
- scriptString = IOUtils.getStringFromReader(new InputStreamReader(scriptStream));
- scriptStream.close();
- eventCache.put(cacheName, scriptString);
- }
+ scriptStream = context.getResourceAsStream(event.path + event.invoke);
+ if (scriptStream == null) {
+ throw new EventHandlerException("Could not find BSF script file in webapp context: " + event.path + event.invoke);
}
+ scriptString = IOUtils.getStringFromReader(new InputStreamReader(scriptStream));
+ scriptStream.close();
+ scriptString = eventCache.putIfAbsentAndGet(cacheName, scriptString);
}
}