This patch adds the option to display Internal Names of modules on the Modules List page. The internal name is what Drupal knows the module as -- it's what all of the source files are pre-pended with, what all of its implemented hooks begin with, the name of the directory it's located in (and the name of the tarball file when it's downloaded), etc. This solves the problem that every Drupal user runs into sooner or later of knowing what a module's internal name is (because they just downloaded it), but then being unable to locate it on the Modules List page because it goes by a different name.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

CydeWeys’s picture

Here's the .patch that implements this feature.

CydeWeys’s picture

I defaulted my new feature to FALSE, but I don't think it'd be unreasonable to have it default to TRUE. Let me know your thoughts.

Cameron Tod’s picture

Status: Needs review » Reviewed & tested by the community

Patch looks good and works well. I guess it could be implemented as a plugin in contribs rather than as an add on to system_module, but otherwise, looks great and very useful!

CydeWeys’s picture

Unfortunately it can't be a separate contrib plugin because the code it uses to do its actual work needs to necessarily reside in the overridden theme_system_modules_theme function, which is already overridden in system_module. If you want to combine the functionality of the two they need to be together in that one definition.

NancyDru’s picture

Status: Reviewed & tested by the community » Needs work

Menu titles and descriptions are automatically passed through t(), so should that not be used in hook_menu().

I don't like single checkboxes, please change it to radios with '#options' => array(t('No'), t('Yes'))

Please don't mess with my closing parenthesis alignment.

CydeWeys’s picture

OK, all of those issues have been addressed in this .patch file.

CydeWeys’s picture

Status: Needs work » Reviewed & tested by the community
NancyDru’s picture

Status: Reviewed & tested by the community » Needs review

Okay, I'll check it out.

NancyDru’s picture

Status: Needs review » Fixed

Committed, thanks for the patch.

Status: Fixed » Closed (fixed)

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