I am having a problem with this module in conjunction with the switchtheme module. It checks to see if the $custom_theme global variable has been set, and bails out if it has.

The problem is that when a user without permission to the admin_theme views the site, $custom_theme is set to 0, so switchtheme never attempts to set the new theme.

I think this is best solved in this module, by unsetting the custom_theme rather than assigning it a 0 value. This would be more compatible with other modules, since null is also false.

CommentFileSizeAuthor
admin_theme.custom_theme.patch740 byteszroger

Comments

markus_petrux’s picture

markus_petrux commented
Status: Needs review » Reviewed & tested by the community

Agree on this fix.

meba’s picture

meba commented

Just found the same issue independently. The proposed patch fixes my problem too. RTBC?

davyvdb’s picture

davyvdb commented
Status: Reviewed & tested by the community » Fixed

This has been fixed in CVS now. Thanks!

Status: Fixed » Closed (fixed)

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