Opened 19 years ago

Closed 19 years ago

Last modified 18 years ago

#634 closed defect (fixed)

[patch] admin shortcuts don't honor existing http:// absolute urls

Reported by: hugo Owned by: Adrian Holovaty
Component: contrib.admin Version:
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

When you have objects in your model that return serverabsolute urls with leading ​http://, they aren't honored by the current r/ shortcut view - it still prepends the site. The attached patch honors those URLs and if it starts with ​http:// (should maybe extended to ​https://, too?), it just returns the URL without attaching some site stuff.

I use a ​http:// carrying URL in one of my models where objects in the database return a get_<field>_url() result as their absolute url, because they are just media files listed in the database.

Attachments (1)

shortcut-fix.diff​ (1.1 KB ) - added by hugo 19 years ago.
fixes admin shortcuts to honor leading ​http://

Download all attachments as: .zip

Change History (3)

by hugo, 19 years ago

Attachment: shortcut-fix.diff​ added

fixes admin shortcuts to honor leading ​http://

comment:1 by hugo, 19 years ago

Summary: admin shortcuts don't honor existing http:// absolute urls → [patch] admin shortcuts don't honor existing http:// absolute urls

comment:2 by Adrian Holovaty, 19 years ago

Resolution: → fixed
Status: new → closed

(In [903]) Fixed #634 -- Changed shortcut view to accept get_absolute_url()s that return URLs starting with http. Thanks, Hugo

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