Opened 15 years ago

Closed 12 years ago

#12140 closed Bug (fixed)

urlencode empty list encoding bug & fix

Reported by: aneil Owned by: nobody
Component: HTTP handling Version: dev
Severity: Normal Keywords: urlencode
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Julien Phalip)

django.utils.http.urlencode doesn't behave the same as urllib.urlencode for empty lists:

urllib.urlencode({'foo':[]},True) => ''

django.utils.http.urlencode({'foo':[]},True) => 'foo=%5B%5D'

The attached diff has a change at line 557 in trunk/django/utils/http.py:

-         isinstance(v, (list,tuple)) and [smart_str(i) for i in v] or smart_str(v))
+         [smart_str(i) for i in v] if isinstance(v, (list,tuple)) else smart_str(v))

Attached.

Attachments (2)

urlencode.diff​ (504 bytes ) - added by aneil 15 years ago.
urlencode empty list bug patch
12140-2.diff​ (1.6 KB ) - added by Claude Paroz 13 years ago.
Patch updated to current trunk

Download all attachments as: .zip

Change History (12)

by aneil, 15 years ago

Attachment: urlencode.diff​ added

urlencode empty list bug patch

comment:1 by aneil, 15 years ago

Change is at line 41, not line 557.

comment:2 by Russell Keith-Magee, 15 years ago

Needs tests: set
Triage Stage: Unreviewed → Accepted

comment:3 by floguy, 14 years ago

Needs tests: unset
Version: 1.1 → SVN

I've updated the patch to include tests for this. It's all isolated in a branch here: ​http://github.com/ericflo/django/compare/ticket12140

comment:4 by Matt McClanahan, 14 years ago

Severity: → Normal
Type: → Bug

comment:5 by Julien Phalip, 14 years ago

Patch needs improvement: set

The tests would need to be rewritten using unittests since this is now Django's preferred way.

comment:6 by Julien Phalip, 14 years ago

Description: modified (diff)

Reformatted description.

comment:7 by Aymeric Augustin, 13 years ago

UI/UX: unset

Change UI/UX from NULL to False.

comment:8 by Aymeric Augustin, 13 years ago

Easy pickings: unset

Change Easy pickings from NULL to False.

by Claude Paroz, 13 years ago

Attachment: 12140-2.diff​ added

Patch updated to current trunk

comment:9 by Claude Paroz, 13 years ago

Patch needs improvement: unset

comment:10 by Claude Paroz <claude@…>, 12 years ago

Resolution: → fixed
Status: new → closed

In [fe873e276527534b3c0c22f457ee314cf029ced4]:

Fixed #12140 -- Fixed http.urlencode result for empty lists

Thanks aneil for the report and the initial patch.

Note: See TracTickets for help on using tickets.
Back to Top