contrib.redirects has a small error in new_path handling
|Reported by:||cephelo@…||Owned by:||adrian|
|Cc:||Triage Stage:||Ready for checkin|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
As per http://www.djangoproject.com/documentation/redirects/, contrib.redirects should issue a HTTP GONE response when the new_path is empty. I'm probably the only one that uses this, but there's a small error in the Django code, where the redirect object is compared to an empty string instead of the new_path attribute.
Attached patch fixes it.
Change History (3)
Changed 7 years ago by cephelo@…
comment:1 Changed 7 years ago by SmileyChris
- Needs documentation unset
- Needs tests unset
- Patch needs improvement unset
- Triage Stage changed from Unreviewed to Ready for checkin