Author: jonesde
Date: Sat Feb 14 08:23:13 2009
New Revision: 744420
URL:
http://svn.apache.org/viewvc?rev=744420&view=revLog:
Fixed issue reported by Ashish Nagar where the condition was checking for the same character twice
Modified:
ofbiz/trunk/framework/base/src/org/ofbiz/base/util/StringUtil.java
Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/StringUtil.java
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/StringUtil.java?rev=744420&r1=744419&r2=744420&view=diff==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/StringUtil.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/StringUtil.java Sat Feb 14 08:23:13 2009
@@ -484,8 +484,8 @@
}
// check for "<", ">"
- if (value.indexOf("<") >= 0 || value.indexOf("<") >= 0) {
- errorMessageList.add("In field [" + valueName + "] greater-than (>) and less-than (<) symbols are not allowed.");
+ if (value.indexOf("<") >= 0 || value.indexOf(">") >= 0) {
+ errorMessageList.add("In field [" + valueName + "] less-than (<) and greater-than (>) symbols are not allowed.");
}
// check for & not followed by a space (can be used for escaping chars)