tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From billbar...@apache.org
Subject svn commit: r382659 - /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java
Date Fri, 03 Mar 2006 05:01:17 GMT
Author: billbarker
Date: Thu Mar  2 21:01:16 2006
New Revision: 382659

URL: http://svn.apache.org/viewcvs?rev=382659&view=rev
Log:
Return the empty String for an empty query-string instead of null.

Fix for Bug #38113

I'm agreeing with the reasoning in the bug, if not the patch offered.


Modified:
    tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java

Modified: tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java
URL: http://svn.apache.org/viewcvs/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java?rev=382659&r1=382658&r2=382659&view=diff
==============================================================================
--- tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java
(original)
+++ tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/Request.java
Thu Mar  2 21:01:16 2006
@@ -1893,11 +1893,7 @@
      */
     public String getQueryString() {
         String queryString = coyoteRequest.queryString().toString();
-        if (queryString == null || queryString.equals("")) {
-            return (null);
-        } else {
-            return queryString;
-        }
+	return queryString;
     }
 
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message