I have clean installation on Drupal 7.4.
In the Module help index page (/admin/advanced_help) I have the warning:

Notice: Undefined index: name in advanced_help_index_page() (line 209 in /autoinstalator/drupal/sites/all/modules/advanced_help/advanced_help.module).

CommentFileSizeAuthor
#11 advanced_help-1209182-11.patch525 bytesmvc
#4 1209182.patch525 byteslambic
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Higarigh’s picture

same problem here, but with Drupal 7.2, any suggestions?

elfur’s picture

same problem here, D7.5.

BenStallings’s picture

This is not a serious error, it's just annoying, and it probably only shows up with certain error-reporting settings, so many people may not be able to replicate the problem.

The error can be suppressed by changing line 209 from

$modules[$info->name] = $module_info['name'];

to

$modules[$info->name] = @$module_info['name'];

See http://php.net/manual/en/language.operators.errorcontrol.php for explanation.

lambic’s picture

FileSize
525 bytes

Here's a patch that uses the value from the name column if name isn't set in info.

lambic’s picture

Status: Active » Needs review
porg’s picture

The fix as described in #3 worked ony my Drupal 7.8 installation.

StephenRobinson’s picture

I have the same error in Drupal 7.9 on /admin/advanced_help with 7.x-1.0

Notice: Undefined index: name in advanced_help_index_page() (line 208 of /www/www-staging/drupal.sandbox.sanger.ac.uk/htdocs/sites/all/modules/contrib/advanced_help/advanced_help.module).

debug shows is profile module:

    *  info

      stdClass Object
      (
          [filename] => profiles/default/default.profile
          [name] => default
          [type] => module
          [owner] => 
          [status] => 0
          [bootstrap] => 0
          [schema_version] => 0
          [weight] => 1000
          [info] => a:6:{s:12:"dependencies";a:0:{}s:11:"description";s:0:"";s:7:"package";s:5:"Other";s:7:"version";N;s:3:"php";s:5:"5.2.4";s:5:"files";a:0:{}}
      )

    * module_info

      Array
      (
          [dependencies] => Array
              (
              )

          [description] => 
          [package] => Other
          [version] => 
          [php] => 5.2.4
          [files] => Array
              (
              )

      )

    * info info

      a:6:{s:12:"dependencies";a:0:{}s:11:"description";s:0:"";s:7:"package";s:5:"Other";s:7:"version";N;s:3:"php";s:5:"5.2.4";s:5:"files";a:0:{}}

easy fix:

      if(isset($module_info['name'])){
        $modules[$info->name] = $module_info['name'];
      }
axle_foley00’s picture

This error still seems to show up in 7.x-1.0. Will this fix be rolled into an official release soon?

gippy’s picture

To echo axle_foley00 in #8, this is a minor problem, but it's an easy fix that has been out there since Sept 2011. It would be nice to at least get it out in the dev version. The fix in the patch in #4 when applied to dev resolved the problem for me.

mvc’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

patch in #4 works for me.

mvc’s picture

Version: 7.x-1.0-beta1 » 7.x-1.x-dev
FileSize
525 bytes

gah, a new version came out yesterday without this patch -- that's what i get for not marking this as RTBC earlier.

lambic's one line patch still fixes the problem. re-rolled against latest dev, so it can be committed and included in the next stable release, if there ever even is one for D7.

redndahead’s picture

Status: Reviewed & tested by the community » Fixed

Sorry about that. I have committed the patch. At least it will be in dev.

Status: Fixed » Closed (fixed)

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

apperceptions’s picture

fyi, still and open issue.
line mentioned in #3 is now on line 233