I have got these notice messages on page /admin/modules in Drupal 7 when I have downloaded package patterns:

Notice: Undefined index: name in system_sort_modules_by_info_name() (line 933 of /var/aegir/platforms/D-tests-7.01/modules/system/system.admin.inc).
Notice: Undefined index: name in system_sort_modules_by_info_name() (line 933 of /var/aegir/platforms/D-tests-7.01/modules/system/system.admin.inc).
Notice: Undefined index: name in system_sort_modules_by_info_name() (line 933 of /var/aegir/platforms/D-tests-7.01/modules/system/system.admin.inc).
Notice: Undefined index: name in system_sort_modules_by_info_name() (line 933 of /var/aegir/platforms/D-tests-7.01/modules/system/system.admin.inc).
Notice: Undefined index: name in system_sort_modules_by_info_name() (line 933 of /var/aegir/platforms/D-tests-7.01/modules/system/system.admin.inc).
Notice: Undefined index: name in system_sort_modules_by_info_name() (line 933 of /var/aegir/platforms/D-tests-7.01/modules/system/system.admin.inc).
Notice: Undefined index: name in system_sort_modules_by_info_name() (line 933 of /var/aegir/platforms/D-tests-7.01/modules/system/system.admin.inc).
Warning: uasort(): Array was modified by the user comparison function in system_modules() (line 784 of /var/aegir/platforms/D-tests-7.01/modules/system/system.admin.inc).
Notice: Undefined index: name in _system_modules_build_row() (line 966 of /var/aegir/platforms/D-tests-7.01/modules/system/system.admin.inc).

These warning messages are very annoying when I need to administer modules and they show even when I have all patterns package modules disabled.

I think I have found the reason: Drupal core is unable to read informations about submodule patterns_d2d from patterns_d2d/patterns_d2d.info file because you have ";" mark at begin of every line in the info file. I see the solution in removing ";" marks (patch is included) or in removing this module from the package (because I think you have good reason to have the marks in the info file).

Steps to reproduce this issue:

  1. Download Drupal 7
  2. Download patterns module in any official version for Drupal 7 (beta1 or dev). You don't need to enable any modules.
CommentFileSizeAuthor
patterns_d2d.info.patch605 byteskrato
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

csko’s picture

Assigned: Unassigned » csko
Status: Active » Needs review

Hi krato,

Thanks for testing out Patterns and filing this bug report.

That module is meant to be hidden and apparently we have done a poor job at that. Somehow the errors never came up until I installed a fresh Drupal.

Anyway, I renamed the .info file for now, it seems to be OK. It should be up in a new release -beta2 soon.
You can update to the newest version or just refrain from installing that module. Or, you can try it out if you are really adventurous (you'll need https://github.com/cschwirz/drupaltodrupal).

krato’s picture

It is all right now, There is no errors anymore on modules page. Thanks for bug fixing and we look forward to unhidden versions of your new module :-) Maybe me or someone else will test your github version soon :-)

csko’s picture

Status: Needs review » Closed (fixed)

Thanks, I'll close this issue then.

The Github version of Patterns 7.x-1.x is actually in sync with drupal.org right now (we push here from time to time). It's just there are extra modules here and there.