Return-Path: Mailing-List: contact tomcat-dev-help@jakarta.apache.org; run by ezmlm Delivered-To: mailing list tomcat-dev@jakarta.apache.org Received: (qmail 3855 invoked by uid 2016); 28 Oct 1999 22:54:58 -0000 Delivered-To: apcore-jakarta-tomcat-cvs@apache.org Received: (qmail 3827 invoked by uid 257); 28 Oct 1999 22:54:57 -0000 Date: 28 Oct 1999 22:54:57 -0000 Message-ID: <19991028225457.3824.qmail@hyperreal.org> From: gonzo@hyperreal.org To: jakarta-tomcat-cvs@apache.org Subject: cvs commit: jakarta-tomcat/src/share/org/apache/tomcat/core Request.java gonzo 99/10/28 15:54:56 Modified: src/share/org/apache/tomcat/core Tag: TOMCAT_J2EE_10F_102199 Request.java Log: fix (again) for request.getParameter() accessing queryString data - j2ee costin - can we possibly keep the request/response re-tooling your doing out of the j2ee tree? i'm guessing RequestImpl will disappear someday so i copied the setQueryString() method to bridge the QueryString and Parameter constructs vs a super call. i'm going to try and add some tests for this stuff as well so that we should feel a bit better about making request/response changes. thx, - james Revision Changes Path No revision No revision 1.1.2.1 +14 -3 jakarta-tomcat/src/share/org/apache/tomcat/core/Request.java Index: Request.java =================================================================== RCS file: /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/core/Request.java,v retrieving revision 1.1 retrieving revision 1.1.2.1 diff -u -r1.1 -r1.1.2.1 --- Request.java 1999/10/09 00:30:15 1.1 +++ Request.java 1999/10/28 22:54:54 1.1.2.1 @@ -1,7 +1,7 @@ /* - * $Header: /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/core/Request.java,v 1.1 1999/10/09 00:30:15 duncan Exp $ - * $Revision: 1.1 $ - * $Date: 1999/10/09 00:30:15 $ + * $Header: /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/core/Request.java,v 1.1.2.1 1999/10/28 22:54:54 gonzo Exp $ + * $Revision: 1.1.2.1 $ + * $Date: 1999/10/28 22:54:54 $ * * ==================================================================== * @@ -380,6 +380,17 @@ public void setQueryString(String queryString) { this.queryString = queryString; + + Hashtable parameters = null; + + // catch any parse exceptions + + try { + parameters = HttpUtils.parseQueryString(queryString); + } catch (Exception e) { + } + + this.parameters = parameters; } public String getRemoteUser() {