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)

masquerade_hook_init.patch4.39 KBhefox


andypost’s picture

Status: Active » Needs review

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