Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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
Comment #1
MichelleWhoops. 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
Comment #2
MichelleBy 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
Comment #3
(not verified) CreditAttribution: commentedAutomatically closed -- issue fixed for two weeks with no activity.