Switch Theme option name [#1194344]

Hi everybody,

It seems that there is a issue when I try to change the display name on the admin/config/user-interface/switchtheme page.

I want to allow user to change theme between the new one(Pixture Reloaded) and the old one (Garland). So i change the display name by ' new theme' and 'old theme'.

On the switchtheme_select() function, it tries to get the variable like that :
$options[$name] = variable_get('switchtheme_' . $label, $label);
The switch theme variable looks like that :
switchtheme_garland or switchtheme_pixture_reloaded.
But the $label look like Garland and Pixture Reloaded, so it could not find the variable...

Solution !!
Replace the $options[$name] = variable_get('switchtheme_' . $label, $label);
by $options[$name] = variable_get('switchtheme_' . $name, $label);

Elodie

Comments

Comment #1

Rob C CreditAttribution: Rob C commented 15 August 2011 at 05:47

Indeed, this works.
Looks like it's already in the dev, seems to work okay.

Comment #2

jean-luc-1 CreditAttribution: jean-luc-1 commented 18 September 2013 at 09:15

I also need to change the theme names in the user switch block and it doesn't work with 7.x-1.0 version of Switchname.
Jean-Luc

Comment #3

B.X CreditAttribution: B.X commented 27 January 2014 at 03:13

//Replace the 
$options[$name] = variable_get('switchtheme_' . $label, $label);
//by 
$options[$name] = variable_get('switchtheme_' . $name, $label);

Thanks friend, this code works, now I can change the name. Wow!

Comment #4

webdev100 CreditAttribution: webdev100 commented 6 July 2014 at 21:24

Issue summary:

View changes

Hi,

where can I find the switchtheme_select() function?

-D

Comment #5

jwilson3

English

Ecuador

CreditAttribution: jwilson3 commented 16 July 2014 at 14:26

Status:

Active

» Fixed

@DrupalNew1: its in the switchtheme.module file. This issue has been fixed in the development version of the code. you can download a dev snapshot on the project page, which should fix this issue in Drupal 7. not sure about d6, so if anyone needs a backport, change the status.