Opened 15 years ago

Closed 14 years ago

Last modified 13 years ago

#12217 closed (fixed)

FlatpageFallbackMiddleware means view middleware is not applied

Reported by: Carl Meyer Owned by: nobody
Component: Documentation Version: dev
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

If flatpages are installed according to the docs (that is, using FlatpageFallbackMiddleware), any view middleware installed will not affect flatpages (because the flatpage view is not called in the usual fashion via URLconf).

It's not clear to me that this is a bug (it's perfectly logical once you realize what's happening), but it certainly can be a cause of confusing bugs ("why is this middleware I just installed not working on certain parts of my site?").

Perhaps it's just something to note in the flatpages docs? I'll put together a patch if someone can confirm that's the best solution here.

Change History (5)

comment:1 by Russell Keith-Magee, 14 years ago

Component: Contrib apps → Documentation
milestone: → 1.2

Yeah - I think this is a documentation issue.

comment:2 by Russell Keith-Magee, 14 years ago

Triage Stage: Unreviewed → Accepted

comment:3 by James Bennett, 14 years ago

Resolution: → fixed
Status: new → closed

(In [12671]) Fixed #12217: Documented the fact that serving flatpages from the flatpage middleware does not apply view middleware methods.

comment:4 by James Bennett, 14 years ago

(In [12672]) [1.1.X] Fixed #12217: Documented the fact that serving flatpages from the flatpage middleware does not apply view middleware methods. Backport of [12671] from trunk.

comment:5 by Jacob, 13 years ago

milestone: 1.2

Milestone 1.2 deleted

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