None of my module's files use classes or interfaces, yet Coder Review gives the following message:
Line 5: Module .info files must now specify all loadable code files explicitly.

The line indicated is always the last line in my .info file.

According to https://drupal.org/node/542202, the files[] property should be optional. Am I misunderstanding the files[] property or is this a bug with Coder Review?

Comments

solotandem’s picture

solotandem commented
Component: Coder Review » Review/Rules

Your understanding is spot on; the review rule needs to be updated. This API change (i.e. what is included in the Drupal code registry) initially included all code files. It was changed to include just files with classes and interfaces, but the review rule was not updated. Thanks for making an issue of this.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

sub

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Issue summary: View changes

Corrected nomenclature.

kenorb’s picture

kenorb commented

See:
https://drupal.org/update/modules/6/7#registry
https://drupal.org/comment/4395614#comment-4395614

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Issue summary: View changes
Status: Active » Closed (won't fix)

Coder 7.x-1.x is frozen now and will not receive any updates. Coder 8.x-2.x can be used to check code for any Drupal version, Coder 8.x-2.x also supports the phpcbf command to automatically fix conding standard errors. Please check if this issue is still relevant and reopen against that version if necessary.