#5964 closed New feature (fixed)
utils: urlunquote
| Reported by: | Thomas Güttler (Home) < > | Owned by: | Aymeric Augustin |
|---|---|---|---|
| Component: | Core (Other) | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | hv@… | Triage Stage: | Accepted |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Hi,
Using only urllib.unquote() for the result of django.utils.http.urlquote() does
not work. That's why I think an urlunquote() would be good.
A patch inclusive unittest is attached.
Attachments (2)
Change History (11)
by , 17 years ago
| Attachment: | urlunquote.diff added |
|---|
comment:1 by , 17 years ago
| Triage Stage: | Unreviewed → Design decision needed |
|---|
comment:2 by , 16 years ago
| Component: | Tools → Core framework |
|---|
comment:3 by , 14 years ago
| Severity: | → Normal |
|---|---|
| Type: | → New feature |
comment:4 by , 13 years ago
| Easy pickings: | unset |
|---|---|
| Triage Stage: | Design decision needed → Accepted |
| UI/UX: | unset |
comment:5 by , 13 years ago
by , 13 years ago
comment:6 by , 13 years ago
| Owner: | changed from to |
|---|
comment:7 by , 13 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
In [17407]:
(The changeset message doesn't reference this ticket)
Note:
See TracTickets
for help on using tickets.
Makes sense to me, though we ought to provide
unquote_plussince we providequote_plus. I'll attach patch that covers both, though someone with deep unicode knowledge should double-check whether we should be using encode/decode instead, and whetherutf-8will suffice here.