You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@chemistry.apache.org by fm...@apache.org on 2011/03/21 23:54:35 UTC

svn commit: r1084010 - /chemistry/opencmis/trunk/chemistry-opencmis-server/chemistry-opencmis-server-bindings/src/main/java/org/apache/chemistry/opencmis/server/shared/HttpUtils.java

Author: fmui
Date: Mon Mar 21 22:54:35 2011
New Revision: 1084010

URL: http://svn.apache.org/viewvc?rev=1084010&view=rev
Log:
fixed CMIS-334: Optional AtomPub parameters that are set to "" can cause a CmisInvalidArgumentException

Modified:
    chemistry/opencmis/trunk/chemistry-opencmis-server/chemistry-opencmis-server-bindings/src/main/java/org/apache/chemistry/opencmis/server/shared/HttpUtils.java

Modified: chemistry/opencmis/trunk/chemistry-opencmis-server/chemistry-opencmis-server-bindings/src/main/java/org/apache/chemistry/opencmis/server/shared/HttpUtils.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-server/chemistry-opencmis-server-bindings/src/main/java/org/apache/chemistry/opencmis/server/shared/HttpUtils.java?rev=1084010&r1=1084009&r2=1084010&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-server/chemistry-opencmis-server-bindings/src/main/java/org/apache/chemistry/opencmis/server/shared/HttpUtils.java (original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-server/chemistry-opencmis-server-bindings/src/main/java/org/apache/chemistry/opencmis/server/shared/HttpUtils.java Mon Mar 21 22:54:35 2011
@@ -158,7 +158,7 @@ public class HttpUtils {
      */
     public static boolean getBooleanParameter(HttpServletRequest request, String name, boolean def) {
         String value = getStringParameter(request, name);
-        if (value == null) {
+        if ((value == null) || (value.length() == 0)) {
             return def;
         }
 
@@ -170,7 +170,7 @@ public class HttpUtils {
      */
     public static Boolean getBooleanParameter(HttpServletRequest request, String name) {
         String value = getStringParameter(request, name);
-        if (value == null) {
+        if ((value == null) || (value.length() == 0)) {
             return null;
         }
 
@@ -194,7 +194,7 @@ public class HttpUtils {
      */
     public static BigInteger getBigIntegerParameter(HttpServletRequest request, String name) {
         String value = getStringParameter(request, name);
-        if (value == null) {
+        if ((value == null) || (value.length() == 0)) {
             return null;
         }