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 2008/12/09 18:31:29 UTC

svn commit: r724792 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java

Author: adrianc
Date: Tue Dec  9 09:31:28 2008
New Revision: 724792

URL: http://svn.apache.org/viewvc?rev=724792&view=rev
Log:
Fixed a bug I introduced in FlexibleStringExpander - the special ${env...} syntax wasn't being expanded properly.

Modified:
    ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java?rev=724792&r1=724791&r2=724792&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java Tue Dec  9 09:31:28 2008
@@ -368,19 +368,22 @@
             this.bracketedOriginal = openBracket + original + closeBracket;
         }
         public void append(StringBuilder buffer, Map<String, ? extends Object> context, TimeZone timeZone, Locale locale) {
+            Object obj = null;
             try {
-                Object obj = UelUtil.evaluate(context, this.bracketedOriginal);
-                if (obj == null) {
-                    if (this.original.startsWith("env.")) {
-                        obj = System.getProperty(this.original.substring(4));
-                    }
-                }
-                if (obj != null) {
-                    buffer.append((String) ObjectType.simpleTypeConvert(obj, "String", null, timeZone, locale, false));
-                }
+                obj = UelUtil.evaluate(context, this.bracketedOriginal);
             } catch (Exception e) {
                 Debug.logVerbose("Error evaluating expression: " + e, module);
             }
+            if (obj == null) {
+                if (this.original.startsWith("env.")) {
+                    obj = System.getProperty(this.original.substring(4));
+                }
+            }
+            if (obj != null) {
+                try {
+                    buffer.append((String) ObjectType.simpleTypeConvert(obj, "String", null, timeZone, locale, false));
+                } catch (Exception e) {}
+            }
         }
     }