I needed an option to disable certain themes in the select list.

Since #735638: Option to select which of the enabled themes will be available for switching only applies to D6 i made this little patch.

CommentFileSizeAuthor
#8 1159884.8-switchtheme-disable-option.patch1.22 KBmrfelton
#5 switchtheme-disable_themes-1159884.patch836 bytesRobin Millette
add-disable-option.patch2.75 KBdonniewiko

Comments

wernercd’s picture

wernercd commented

Just what I was looking for... I'll try this tonight.

Anonymous’s picture

Anonymous (not verified) commented

Works perfect.

Only in the page layout I would preferre a table view of all the themes (theme per row), to have all settings in one pageview, withouth the closed fieldsets per theme. But like I said: it works great.

raydenxxx’s picture

raydenxxx commented

Works great!

Be aware about this correction

-    $options[$name] = variable_get('switchtheme_' . $name, $label);
+	if(!variable_get('disable_' . $name)){
+      $options[$name] = variable_get('switchtheme_' . $label, $label);
+	}

You replace the $name var by $label which will do name changing dosn't work.
So you should write this

-    $options[$name] = variable_get('switchtheme_' . $name, $label);
+	if(!variable_get('disable_' . $name)){
+      $options[$name] = variable_get('switchtheme_' . $name, $label);
+	}
Robin Millette’s picture

Robin Millette commented

That's a big patch!

Instead of another set of variables, in this patch, blanked labels disable themes.

Robin Millette’s picture

Robin Millette commented
StatusFileSize
new switchtheme-disable_themes-1159884.patch836 bytes
mr_wookie’s picture

mr_wookie commented

@#5: Works greate for me. Thanks!

rolfmeijer’s picture

rolfmeijer commented

Works like a charm.
Maybe the help text needs an update?
Something like:

Set a label for each enabled theme. This is what will be displayed to the user in the selection box. An empty label will hide it from the selection box.

It’s just a suggestion, maybe it’s clear as it is, I’m just saying a new (dev-)version would be great. :-)

mrfelton’s picture

mrfelton commented
StatusFileSize
new 1159884.8-switchtheme-disable-option.patch1.22 KB

Patch updated to apply cleanly against latest dev code

m-patate’s picture

m-patate commented

Hello,

Thanks for this usefull enhancement !
Do you know when it will be available in a new release module version ?
Thanks

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Reviewed & tested by the community

#8 works for me!
Marked #1948174: I need a way to remove a theme from the pool. as a duplicate.