Closed (fixed)
Project:
Advanced Forum
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 May 2008 at 03:27 UTC
Updated:
21 May 2008 at 20:51 UTC
Hey-- I'm using the garland theme and updated to the new -dev module. Had a problem where no matter what I set the Advanced forum theme directory to in the preferences, it still could never find it and just went to the defaults along with a lot of warning messages.
A quick search of the module itself came up with this:
// If we had no luck finding a subtheme, find the theme path
// the normal way.
if (!empty($advanced_forum_theme_path)) {
$advanced_forum_theme_path = path_to_theme();
Could the problem be line 823 above-- namely you want to set a path to the theme if the variable *IS* empty, like this:
if (empty($advanced_forum_theme_path)) {
That is-- taking out the "!" before "empty" seemed to fix it for me.
W
Comments
Comment #1
michelleYes, that section of code is totally broken. I was starting to work on the Zen problem and didn't get it finished. I'm hoping to have that finished soon, but I needed to work on a client site tonight.
Michelle
Comment #2
michellePS: There are no database changes with advforum so you can safely go back to the last alpha.
Michelle
Comment #3
michelleActually, I guess it was as simple as that to fix it. I had written that code and never tested it and then accidentally committed it. No clue why I put a ! there. But fixed, now, thanks.
Michelle
Comment #4
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.