Hi! Im getting this notice level error with t he latest dev of drupal. I believe if all notice lexel errors were fixed, drupal would run better with less resource use maybe.
Thanks for any help!

notice: Undefined variable: output in /home/cyberfan/htdocs/sites/all/modules/user_visits_adv/user_visits_adv.module on line 23.

CommentFileSizeAuthor
#2 user_visits_adv_934186_amitgoyal_1_d6_1.x.patch1.89 KBamitgoyal
#1 user-visits-advanced-934186-killua99-1-D6-1.x.patch433 byteskillua99

Comments

killua99’s picture

killua99 commented
StatusFileSize
new user-visits-advanced-934186-killua99-1-D6-1.x.patch433 bytes

This is what you need

/**
 * Implementation of hook_help();
 */
function user_visits_adv_help($path, $arg) {
  $output = '';
  switch ($path) {
    case "admin/help#user-visits-advaced":
      $output = '<p>'.  t('Extends the user visits module.') .'</p>';
      break;
  }
  return $output;
}
amitgoyal’s picture

amitgoyal commented
Status: Active » Needs review
StatusFileSize
new user_visits_adv_934186_amitgoyal_1_d6_1.x.patch1.89 KB

Please accept updated patch which also takes care of coder review issues.

sanchi.girotra’s picture

sanchi.girotra commented
Status: Needs review » Reviewed & tested by the community

@amitgoyal - I have tested this patch and it worked as required.

amitgoyal’s picture

amitgoyal commented
Status: Reviewed & tested by the community » Fixed

This has been fixed in the 6.x-1.x-dev version.

Status: Fixed » Closed (fixed)

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