Opened 17 years ago

Closed 17 years ago

#6167 closed (duplicate)

Flat pages do not honour APPEND_SLASH

Reported by: dkbg <dgrose@…> Owned by: nobody
Component: Contrib apps Version: dev
Severity: Keywords: flatpages
Cc: semente@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Flat pages do not append a slash if they were defined as /page/ and /page is requested.

Attachments (1)

flatpages_append_slash.diff​ (5.6 KB ) - added by Chris Beaven 17 years ago.

Download all attachments as: .zip

Change History (6)

by Chris Beaven, 17 years ago

Attachment: flatpages_append_slash.diff​ added

comment:1 by Chris Beaven, 17 years ago

Has patch: set
Triage Stage: Unreviewed → Ready for checkin

Note that if you're using flat pages currently (using a pattern rather than the middleware), APPEND_SLASH is not even used because flat pages catches everything and a slash is now only appended when the non-slash path doesn't resolve.

Also added in (the start of) a test suite for flatpages!

comment:2 by Chris Beaven, 17 years ago

See also: #6213

comment:3 by Malcolm Tredinnick, 17 years ago

Triage Stage: Ready for checkin → Accepted

Seems to be some possibility that #6213 fixes the problem differently (better?) if I'm reading the comments there correctly. Some should sort out a unified approach here so that we're left with one patch and one ticket open (and the other one closed as a dupe or something).

comment:4 by Guilherme M. Gondim <semente@…>, 17 years ago

Cc: semente@… added

comment:5 by Jason Davies, 17 years ago

Resolution: → duplicate
Status: new → closed

I've updated the patch in #6213 to include test cases based on the ones in the patch for this ticket. Closing this ticket as a duplicate of #6213 as suggested by SmileyChris.

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