Opened 7 years ago

Closed 6 years ago

#28635 closed Bug (fixed)

admin's preserved filters don't work if the URL has non-ASCII characters in it

Reported by: Wenli Tsai Owned by: nobody
Component: contrib.admin Version: 1.11
Severity: Normal Keywords: add_preserved_filters
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Wenli Tsai)

The preserved_filters won't be correctly recovered when url contains unicode prefix.

When save the change form and return to the changelist,
the changelist url should be /admin/myApp/台灣Model/?cityID=a.

But we got /admin/myApp/台灣Model/?_changelist_filters=cityID%3Da,
and 302 redirected to /admin/myApp/台灣Model/?e=1.

Change History (7)

comment:1 by Wenli Tsai, 7 years ago

Description: modified (diff)

comment:2 by Wenli Tsai, 7 years ago

Summary: Non-ASCII url prefixPreserved filters with non-ASCII url prefix

comment:3 by Wenli Tsai, 7 years ago

Summary: Preserved filters with non-ASCII url prefixPreserved filters with unicode url prefix

comment:4 by Wenli Tsai, 7 years ago

Description: modified (diff)

comment:5 by Tim Graham, 7 years ago

Has patch: set
Summary: Preserved filters with unicode url prefixadmin's preserved filters don't work if the URL has non-ASCII characters in it
Triage Stage: UnreviewedAccepted

comment:6 by Carlton Gibson, 6 years ago

Triage Stage: AcceptedReady for checkin

PR looks good.

comment:7 by Tim Graham <timograham@…>, 6 years ago

Resolution: fixed
Status: newclosed

In ba37ee9e:

Fixed #28635 -- Fixed admin's preserved filters if the URL contains non-ASCII characters.

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