Opened 9 years ago

Closed 7 years ago

#11020 closed Uncategorized (wontfix)

models.permalink() does not pass through the url prefix

Reported by: Holger Just Owned by: nobody
Component: Database layer (models, ORM) Version: 1.0
Severity: Normal Keywords:
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: yes
Needs tests: yes Patch needs improvement: no
Easy pickings: UI/UX:


The permalink() decorator does not allow for passing a customized prefix to the reverse() function. The attached patch changes that behavior by allowing an additional positional parameter to be passed to the reverse function.

I originally wanted to use that feature to generate fully qualified URLs to switch between HTTPS and plain HTTP.

Attachments (1)

permalink.patch (401 bytes) - added by Holger Just 9 years ago.
Patch to fix the permalink behavior

Download all attachments as: .zip

Change History (3)

Changed 9 years ago by Holger Just

Attachment: permalink.patch added

Patch to fix the permalink behavior

comment:1 Changed 9 years ago by Chris Beaven

Needs documentation: set
Needs tests: set
Triage Stage: UnreviewedDesign decision needed

You don't have to use the permalink decorator. It feels like this is an edge case that could just use a reverse() call, but I'll push to a design decision rather than close outright.

comment:2 Changed 7 years ago by Chris Beaven

Resolution: wontfix
Severity: Normal
Status: newclosed
Type: Uncategorized

The prefix arg isn't even documented and it's definitely an edge case so I'm going to close this as won't fix.

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