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

SourceForge.net noreply at sourceforge.net
Thu Feb 24 21:02:27 CET 2005

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

More information about the Patches mailing list