All the third party module integrations in ThemeKey's module folder might cause (fatal) errors if such a module gets dectivated.

Comments

mkalkbrenner’s picture

Status: Active » Needs review
StatusFileSize
new1.21 KB

possible fix for ThemeKey 6 attached

chawl’s picture

Status: Needs review » Reviewed & tested by the community

Patch works perfectly, tx :)

mkalkbrenner’s picture

Version: 6.x-3.x-dev » 7.x-1.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Thx for testing.

Now, we need something similar for D7.

mkalkbrenner’s picture

Version: 7.x-1.x-dev » 6.x-3.x-dev
Status: Patch (to be ported) » Needs review
StatusFileSize
new1.55 KB

I wonder why it worked @chawl because I discovered a serious bug regarding the module names while porting this to ThemeKey 7.

Attached is a patch against current 6.x-3.x branch in git.

mkalkbrenner’s picture

StatusFileSize
new5.26 KB

Complete patch for ThemeKey 7.x-1.x

chawl’s picture

Hmm, I hope I understand.

As I only tried disabling wurfl and "if" conditional always returned false because of path problem, themekey_scan_modules() was always invoked, thus variables table was always ok but I didn't recognise that no inc was really included as I was not using TK actively at test time whereat I had no rules defined :S

Sorry for casual testing.

mkalkbrenner’s picture

@chawl: no problem. Current version in git should work like expected now.

mkalkbrenner’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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