Posted by hefox on August 2, 2012 at 9:04pm
2 followers
Jump to:
| Project: | Masquerade |
| Version: | 7.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
As title says, burdening bootstrap with the check seems unnecessary -- why does it need to be stored in a session and checked every page load? Wrapping it in a function allows it never be checked for certian checks/drush/etc.
Attaching patch, but for d6 version, so not putting it as needs review/work (assuming masquerade is get into 7 first than backport)
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| masquerade_hook_init.patch | 4.39 KB | Ignored | None | None |
Comments
#1
Idea is good but D7 has lazy-session creation so if nothing is stored in session no session would be stored/started. This needs manual review and testing