Seems like your $advforum_path isn't set in advanced_forum.module. 

if (round((time()-$account->access)/60) < 15) {
       $vars['online_icon'] = theme_image($advforum_path . '/images/status_online.png', $alt = 'User is online', $title = 'User is online', $attributes = NULL, $getsize = TRUE);
    } else {
       $vars['online_icon'] = theme_image($advforum_path . '/images/status_offline.png', $alt = 'User is offline', $title = 'User is offline', $attributes = NULL, $getsize = TRUE);
    }

I think you assume $advforum_path is set after you call advanced_forum_add_css() but it isn't a global variable. All I did was added the line below before that code above:

$advforum_path = drupal_get_path('module', 'advanced_forum');

Comments

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Status: Active » Fixed

Whoops. Actually what happened is that variable used to be set up at the top of the function. Then I moved the CSS stuff into its own function, forgetting that the variable was also used for the online status. Thanks for catching that. Fixed in the CVS version now.

Michelle

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

By the way, I should mention that this bit is going to change. My plan is to add a configuration screen so you can you can set where you want to pull the icons from so you can change them out without having to hack the module. Since I haven't built that screen, yet, I hardcoded in the location for now.

Michelle

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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