Opened 16 years ago

Closed 14 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: no UI/UX: no

Description

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 16 years ago.
Patch to fix the permalink behavior

Download all attachments as: .zip

Change History (3)

by Holger Just, 16 years ago

Attachment: permalink.patch added

Patch to fix the permalink behavior

comment:1 by Chris Beaven, 15 years ago

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 by Chris Beaven, 14 years ago

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