Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#14377 closed (fixed)

Add site to the context of the logout view

Reported by: Hraban <hraban@…> Owned by: nobody
Component: contrib.auth Version: dev
Severity: Keywords: logout site view
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

This should be a really simple addition. ;)

In my app's root template I change the style according to site settings; everything's ok until the user logs out, then he gets a "completely different" site.
Of course I can fix this e.g. with a middleware, but I guess it can't hurt to populate the logout view's context in the same way as the login view's.

Change History (3)

comment:1 by Gabriel Hurley, 14 years ago

Triage Stage: UnreviewedAccepted

Valid issue, marking accepted. However, the fix for this issue is in the patch on #14386. When that patch is committed this issue can be closed.

comment:2 by Luke Plant, 14 years ago

Resolution: fixed
Status: newclosed

(In [13980]) Fixed #14386, #8960, #10235, #10909, #10608, #13845, #14377 - standardize Site/RequestSite usage in various places.

Many thanks to gabrielhurley for putting most of this together. Also to
bmihelac, arthurk, qingfeng, hvendelbo, petr.pulc@…, Hraban for
reports and some initial patches.

The patch also contains some whitespace/PEP8 fixes.

comment:3 by Luke Plant, 14 years ago

(In [13987]) [1.2.X] Fixed #14386, #8960, #10235, #10909, #10608, #13845, #14377 - standardize Site/RequestSite usage in various places.

Many thanks to gabrielhurley for putting most of this together. Also to
bmihelac, arthurk, qingfeng, hvendelbo, petr.pulc@…, Hraban for
reports and some initial patches.

The patch also contains some whitespace/PEP8 fixes.

Backport of [13980] from trunk. Some items could be considered features
(i.e. supporting RequestSite in various contrib apps), others are definite
bugs, but it was much more robust to backport all these tightly related
changes together.

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