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 2009/04/13 19:08:15 UTC
svn commit: r764534 -
/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java
Author: adrianc
Date: Mon Apr 13 17:08:15 2009
New Revision: 764534
URL: http://svn.apache.org/viewvc?rev=764534&view=rev
Log:
Small fixup for last commit - made the FlexibleStringExpander code comments more accurate. No functional change.
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=764534&r1=764533&r2=764534&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 Mon Apr 13 17:08:15 2009
@@ -221,19 +221,20 @@
Debug.logWarning("Found a ${ without a closing } (curly-brace) in the String: " + original, module);
break;
}
- // check for escaped expression
+ // Check for escaped expression
boolean escapedExpression = (start - 1 >= 0 && original.charAt(start - 1) == '\\');
if (start > currentInd) {
- // append everything from the current index to the start of the var
+ // append everything from the current index to the start of the expression
strElems.add(new ConstElem(original.substring(currentInd, escapedExpression ? start -1 : start)));
}
if (original.indexOf("bsh:", start + 2) == start + 2 && !escapedExpression) {
- // checks to see if this starts with a "bsh:", if so treat the rest of the string as a bsh scriptlet
+ // checks to see if this starts with a "bsh:", if so treat the rest of the expression as a bsh scriptlet
strElems.add(new BshElem(original.substring(start + 6, end)));
} else if (original.indexOf("groovy:", start + 2) == start + 2 && !escapedExpression) {
- // checks to see if this starts with a "groovy:", if so treat the rest of the string as a groovy scriptlet
+ // checks to see if this starts with a "groovy:", if so treat the rest of the expression as a groovy scriptlet
strElems.add(new GroovyElem(original.substring(start + 9, end)));
} else {
+ // Scan for matching closing bracket
int ptr = original.indexOf(openBracket, start + 2);
while (ptr != -1 && end != -1 && ptr < end) {
end = original.indexOf(closeBracket, end + 1);
@@ -254,14 +255,14 @@
strElems.add(new VarElem(expression));
}
}
- // reset the current index to after the var, and the start to the beginning of the next var
+ // reset the current index to after the expression, and the start to the beginning of the next expression
currentInd = end + 1;
if (currentInd > origLen) {
currentInd = origLen;
}
start = original.indexOf(openBracket, currentInd);
}
- // append the rest of the original string, ie after the last variable
+ // append the rest of the original string, ie after the last expression
if (currentInd < origLen) {
strElems.add(new ConstElem(original.substring(currentInd)));
}