You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ad...@apache.org on 2012/04/08 16:21:02 UTC
svn commit: r1311011 -
/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ScriptUtil.java
Author: adrianc
Date: Sun Apr 8 14:21:01 2012
New Revision: 1311011
URL: http://svn.apache.org/viewvc?rev=1311011&view=rev
Log:
ScriptUtil bug fix: the SCRIPT_NAMES field was initialized only when debug level was INFO. Removed enclosing debug level check so SCRIPT_NAMES will always be initialized.
Modified:
ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ScriptUtil.java
Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ScriptUtil.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ScriptUtil.java?rev=1311011&r1=1311010&r2=1311011&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ScriptUtil.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ScriptUtil.java Sun Apr 8 14:21:01 2012
@@ -77,31 +77,29 @@ public final class ScriptUtil {
static {
Set<String> writableScriptNames = new HashSet<String>();
- if (Debug.infoOn()) {
- ScriptEngineManager manager = new ScriptEngineManager();
- List<ScriptEngineFactory> engines = manager.getEngineFactories();
- if (engines.isEmpty()) {
- Debug.logInfo("No scripting engines were found.", module);
- } else {
- Debug.logInfo("The following " + engines.size() + " scripting engines were found:", module);
- for (ScriptEngineFactory engine : engines) {
- Debug.logInfo("Engine name: " + engine.getEngineName(), module);
- Debug.logInfo(" Version: " + engine.getEngineVersion(), module);
- Debug.logInfo(" Language: " + engine.getLanguageName(), module);
- List<String> extensions = engine.getExtensions();
- if (extensions.size() > 0) {
- Debug.logInfo(" Engine supports the following extensions:", module);
- for (String e : extensions) {
- Debug.logInfo(" " + e, module);
- }
+ ScriptEngineManager manager = new ScriptEngineManager();
+ List<ScriptEngineFactory> engines = manager.getEngineFactories();
+ if (engines.isEmpty()) {
+ Debug.logInfo("No scripting engines were found.", module);
+ } else {
+ Debug.logInfo("The following " + engines.size() + " scripting engines were found:", module);
+ for (ScriptEngineFactory engine : engines) {
+ Debug.logInfo("Engine name: " + engine.getEngineName(), module);
+ Debug.logInfo(" Version: " + engine.getEngineVersion(), module);
+ Debug.logInfo(" Language: " + engine.getLanguageName(), module);
+ List<String> extensions = engine.getExtensions();
+ if (extensions.size() > 0) {
+ Debug.logInfo(" Engine supports the following extensions:", module);
+ for (String e : extensions) {
+ Debug.logInfo(" " + e, module);
}
- List<String> shortNames = engine.getNames();
- if (shortNames.size() > 0) {
- Debug.logInfo(" Engine has the following short names:", module);
- for (String name : engine.getNames()) {
- writableScriptNames.add(name.concat(":"));
- Debug.logInfo(" " + name, module);
- }
+ }
+ List<String> shortNames = engine.getNames();
+ if (shortNames.size() > 0) {
+ Debug.logInfo(" Engine has the following short names:", module);
+ for (String name : engine.getNames()) {
+ writableScriptNames.add(name.concat(":"));
+ Debug.logInfo(" " + name, module);
}
}
}