[Patches] [ python-Patches-740827 ] add urldecode() method to urllib

[SourceForge.net]

Patches item #740827, was opened at 2003-05-21 02:43
Message generated for change (Comment added) made by loewis
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=740827&group_id=5470

Category: Modules
Group: None
>Status: Closed
>Resolution: Rejected
Priority: 5
Submitted By: Matthew Boedicker (mboedick)
Assigned to: Nobody/Anonymous (nobody)
Summary: add urldecode() method to urllib

Initial Comment:
Adds a urldecode() method to urllib. The method does
the opposite of urlencode, turns a url-encoded string
into a list of two-tuples of name/value pairs.

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

>Comment By: Martin v. Löwis (loewis)
Date: 2005-02-24 21:02

Message:
Logged In: YES 
user_id=21627

It appears that all reviewers agree that the patch should
not be applied. Rejecting it.

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

Comment By: Irmen de Jong (irmen)
Date: 2005-01-16 16:41

Message:
Logged In: YES 
user_id=129426

I've always found it strange that url-manipulating methods
were present in the cgi module.
I agree that it would be nicer to have them grouped together
in a single module such as urlparse.
So I wouldn't add urldecode to urllib.

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

Comment By: Brett Cannon (bcannon)
Date: 2003-05-21 18:00

Message:
Logged In: YES 
user_id=357491

I think so, but I don't know which one.  I only use urllib.urlopen ever so if 
you have a specific suggestion from experience please make it and submit a 
patch.  Since I don't use either modules extensively someone else is going to 
need to step in and help make any final decisions.

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

Comment By: Matthew Boedicker (mboedick)
Date: 2003-05-21 16:25

Message:
Logged In: YES 
user_id=119895

It is not different than cgi.parse_qsl.  Must have missed
that module when I was looking for this functionality.  To
me it seems like urlencode and urldecode (as well as quote,
quote_plus, unquote, etc.) belong in urlparse.  Do you think
it would be valuable to include a pointer to cgi.parse_qsl
in urllib or urllparse?

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

Comment By: Brett Cannon (bcannon)
Date: 2003-05-21 05:21

Message:
Logged In: YES 
user_id=357491

How is this different from cgi.parse_sql ?  If it is not different, consider just 
having a wrapper for urldecode that calls cgi.parse_sql .

Do you think urllib is the best place for this?  What about urlparse?

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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=740827&group_id=5470