As requested on the themes list, I have corrected some of the errors in the themeswitcher.module.

Attached is a patch vs. cvs. I will also attach the entire new file, just for simplicity.

CommentFileSizeAuthor
#10 82340_themeswitcher_module.patch.txt8.65 KBBèr Kessels
#9 themeswitcher_3.module9.2 KBagentrickard
#8 themeswitcher_2.module9.2 KBagentrickard
#7 themeswitcher_1.module9.16 KBagentrickard
#4 ts.png116.68 KBagentrickard
#3 themeswitcher_0.module9.15 KBagentrickard
#2 themeswitcher_page.patch8.68 KBagentrickard
#1 themeswitcher.module5.42 KBagentrickard
themeswitcher.patch4.68 KBagentrickard

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new themeswitcher.module5.42 KB

The new code has some substantial changes from prior (since my handling of arrays is more simplistic).

it may be easier, then, to just replace the whole file. Attached is the new module version.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new themeswitcher_page.patch8.68 KB

I went ahead and coded a themeswitcher_page function and menu element.

This gives an overview page that allows for theme selection.

This patch replaces the above.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new themeswitcher_0.module9.15 KB

New module attached.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new ts.png116.68 KB

Screenshot of themeswitcher_page callback.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Active » Needs review
Bèr Kessels’s picture

Bèr Kessels commented

Add a comment to become subscribed.
The code looks very nice, but I'd need to review it a bit closer (and test it! ) :)

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new themeswitcher_1.module9.16 KB

There's at least one error. In themeswitcher_get_current() line 297, I need to invoke global $user as well.

 -   global $custom_theme;
+   global $custom_theme, $user;

This function grabs the user's current theme setting. Failing to invoke global $user may overlook a custom setting.

Attached minor correction to the module file.

The other 'big' code question is whether this module is now themed correctly. The only theme function I use format the individual table cells on the overview page. Drupal standards might mean that the whole themeswitcher_page() function go to a theme function.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new themeswitcher_2.module9.2 KB

Fixed a dumb math error in the themeswitcher_page() logic.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new themeswitcher_3.module9.2 KB

Fixed a missing & on line 281.

Bèr Kessels’s picture

Bèr Kessels commented
StatusFileSize
new 82340_themeswitcher_module.patch.txt8.65 KB

First, lets make a real patch.

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Component: web site » Other

Is this issue misfiled? There's no theme switching on d.o that I'm aware of.

Michelle

mikey_p’s picture

mikey_p commented

Could this be from when the themegarden was hosted on d.o infrastructure?

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

Yes, it is a legacy of themegarden. The issue can likely be closed.

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Status: Needs review » Closed (fixed)

Thanks, Ken. Closing.

Michelle