[SourceForge.net <noreply@so...>]

Bugs item #1486299, was opened at 2006-05-11 04:26
Message generated for change (Comment added) made by ye238
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1486299&group_id=15494

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Please add URLEncoder.encode() to all generateURL() methods.

Initial Comment:
Hello.
   I am an Chinese in Japan. I found a bug in every 
generateURL() methods of XXXXXXURLGenerator classes.

now source code in generateURL() are following:

-------------------------------------------------
result += this.seriesParameterName + "=" + 
seriesKey.toString();
-------------------------------------------------

CAN YOU CHANGE THEM TO FOLLOWING???

-------------------------------------------------
result += this.seriesParameterName + "=" + 
URLEncoder.encode(seriesKey.toString());
-------------------------------------------------

thaks.

gjx1108@...

----------------------------------------------------------------------

Comment By: ye238 (ye238)
Date: 2007-04-14 17:55

Message:
Logged In: YES 
user_id=1756739
Originator: NO

This is definitely an big issue, and I'm not sure why this has not been
fixed.

Although sometimes web browsers are smart about spaces in the query
string, double or single quotes will difinitely cause problems even for
non-international users.

I volunteer to fix this and submit a patch.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1486299&group_id=15494