svn commit: r744420 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/StringUtil.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r744420 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/StringUtil.java

jonesde
Author: jonesde
Date: Sat Feb 14 08:23:13 2009
New Revision: 744420

URL: http://svn.apache.org/viewvc?rev=744420&view=rev
Log:
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)