svn commit: r656693 - in /ofbiz/trunk/framework: base/src/base/org/ofbiz/base/util/UtilHttp.java images/webapp/images/selectall.js

[adrianc]

Author: adrianc
Date: Thu May 15 08:05:38 2008
New Revision: 656693

URL: http://svn.apache.org/viewvc?rev=656693&view=rev
Log:
A couple of small fixups for the Ajax support.

Modified:
    ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilHttp.java
    ofbiz/trunk/framework/images/webapp/images/selectall.js

Modified: ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilHttp.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilHttp.java?rev=656693&r1=656692&r2=656693&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilHttp.java (original)
+++ ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilHttp.java Thu May 15 08:05:38 2008
@@ -692,15 +692,15 @@
     /** Returns the query string contained in a request target - basically everything
      * after and including the ? character.
      * @param target The request target
-     * @return The query string or null if none is found
+     * @return The query string
      */
     public static String getQueryStringFromTarget(String target) {
-        if (target == null || target.length() == 0) return null;
+        if (target == null || target.length() == 0) return "";
         int queryStart = target.indexOf('?');
         if (queryStart != -1) {
             return target.substring(queryStart);
         }
-        return null;
+        return "";
     }
 
     /** Removes the query string from a request target - basically everything

Modified: ofbiz/trunk/framework/images/webapp/images/selectall.js
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/selectall.js?rev=656693&r1=656692&r2=656693&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/selectall.js (original)
+++ ofbiz/trunk/framework/images/webapp/images/selectall.js Thu May 15 08:05:38 2008
@@ -225,7 +225,7 @@
 */
 function ajaxUpdateAreas(areaCsvString) {
     var areaArray = areaCsvString.split(",");
-    var numAreas = areaArray.length / 3;
+    var numAreas = parseInt(areaArray.length / 3);
     for (var i = 0; i < numAreas; i = i + 3) {
         new Ajax.Updater(areaArray[i], areaArray[i + 1], {parameters: areaArray[i + 2]});
     }