craigmcc    2002/07/01 18:51:10

  Modified:    src/share/org/apache/struts/util RequestUtils.java
  Log:
  computeURL() - when rendering a URL with multiple query parameters, use the
  reserved '&' separator between each name=value pair, rather tha "&".
  
  PR: Bugzila #5942
  Submitted by: Jeff Butler <jbutler at ciber.com>
  
  Revision  Changes    Path
  1.41      +10 -10    
jakarta-struts/src/share/org/apache/struts/util/RequestUtils.java
  
  Index: RequestUtils.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/util/RequestUtils.java,v
  retrieving revision 1.40
  retrieving revision 1.41
  diff -u -r1.40 -r1.41
  --- RequestUtils.java 23 Jun 2002 01:38:31 -0000      1.40
  +++ RequestUtils.java 2 Jul 2002 01:51:10 -0000       1.41
  @@ -235,7 +235,7 @@
        * @param paramProperty Property (of bean named by <code>paramName</code>
        *  containing single-value parameter value
        * @param paramScope Scope containing bean named by
  -     *  <code>paramScope</code>
  +     *  <code>paramName</code>
        *
        * @param name Bean containing multi-value parameters Map (if any)
        * @param property Property (of bean named by <code>name</code>
  @@ -462,7 +462,7 @@
                           url.append('?');
                           question = true;
                       } else {
  -                        url.append("&amp;");
  +                        url.append("&");
                       }
                       url.append(URLEncoder.encode(key));
                       url.append('='); // Interpret null as "no value"
  @@ -471,7 +471,7 @@
                           url.append('?');
                           question = true;
                       } else {
  -                        url.append("&amp;");
  +                        url.append("&");
                       }
                       url.append(URLEncoder.encode(key));
                       url.append('=');
  @@ -483,7 +483,7 @@
                               url.append('?');
                               question = true;
                           } else {
  -                            url.append("&amp;");
  +                            url.append("&");
                           }
                           url.append(URLEncoder.encode(key));
                           url.append('=');
  @@ -494,7 +494,7 @@
                           url.append('?');
                           question = true;
                       } else {
  -                        url.append("&amp;");
  +                        url.append("&");
                       }
                       url.append(URLEncoder.encode(key));
                       url.append('=');
  @@ -505,7 +505,7 @@
               // Re-add the saved anchor (if any)
               if (anchor != null) {
                   url.append('#');
  -                url.append(anchor);
  +                url.append(URLEncoder.encode(anchor));
               }
   
           }
  
  
  

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to