Hey all -

I couldn't figure out for a few minutes why my sidebars were all of a sudden disappearing on the access denied page when LoginToboggan was enabled. Well, it's because the sidebars are being unset in hook_page_alter. So I guess that explains why, but not really how.

Could that either not happen, or be user-configurable? Or at least documented? We've got blocks that need to be displayed, even on access denied pages, and without LT enabled it works fine.

Thanks,

Isaac

CommentFileSizeAuthor
#1 1316040-remove-sidebar-setting-1.patch1.22 KBhunmonk

Comments

hunmonk’s picture

Status: Active » Needs review
StatusFileSize
new1.22 KB

a setting for this seems like overkill. still, LT is making a pretty drastic move there that i don't think you can change with a theme override. so i'm toying with the idea in the attached patch as a reasonable solution. please try it out and let me know what you think.

isaac.niebeling’s picture

I'll try the patch in the near future -- at the moment, I've just commented out the unset line since I have to have this to the client this afternoon, but I'll definitely do it on my local copy and let you know. Just looking at it, it appears like it should work, though there's nothing in there yet for end users to actually be able to turn it on and off (without a quick bit of code or DB update).

hunmonk’s picture

the proper way to use this new variable is to set in in settings.php, no db stuff necessary.

this is a very common technique for putting in configuration stuff that is overkill to add to the UI.

isaac.niebeling’s picture

Ah, I hadn't thought of that. Thanks for the reminder.

hunmonk’s picture

Title: Don't unset sidebars on access denied pages » Setting for optional unsetting of sidebars on access denied pages
Status: Needs review » Fixed

committed #1 to 7.x-1.x-dev.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.