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

michelle’s picture

Yes, 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

michelle’s picture

PS: There are no database changes with advforum so you can safely go back to the last alpha.

Michelle

michelle’s picture

Status: Active » Fixed

Actually, 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

Anonymous’s picture

Status: Fixed » Closed (fixed)

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