There are too many files being loaded in the info file. For example, the module file is already loaded by Drupal so it should not be in the info file. This is also the case for the info file itself.

Further, the admin file will be added by hook_menu when needed so you should not add it to the info file. Now the admin file is loaded on every Drupal request but not used at all. Same for the install file, only used when the module is enabled or on cache clear (I guess) but now is also loaded on every request.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

h3rj4n’s picture

Status: Active » Needs review
FileSize
593 bytes

And the patch. The unneeded files are removed from the files array.

h3rj4n’s picture

Also removed the nodereview_node_nodereview.inc file because it's needed and already loaded in the module file. Changed the module_load_include to a require_once. See comment in patch.

h3rj4n’s picture

Remove the theme file from the files array. It's loaded trough hook_theme.

jay.lee.bio’s picture

Status: Needs review » Reviewed & tested by the community

Everything still working fine after applying patch #3. :D

yaworsk’s picture

Status: Reviewed & tested by the community » Closed (fixed)

Patch applied to 7.x-1.x-dev version.

For reference, information regarding files to include in .info files is here: http://drupal.org/node/350780. Drupal 7 introduces a code registry - an inventory of all classes and interfaces for all enabled modules and Drupal's core files. The registry stores the path to the file a given class or interface is defined in, and loads the file when necessary.