Add option to display internal module names on Modules List page [#1760892]

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.

Comment	File	Size	Author
#6	util-system-module-display-internal-names-1760892-6.patch	5.98 KB	CydeWeys
#1	util-system-module-display-internal-names-1760892-1.patch	6.76 KB	CydeWeys
	util_system_module_show_internal_names.png	47.68 KB	CydeWeys

Comments

Comment #1

CydeWeys commented 28 August 2012 at 20:19

Status	File	Size
new	util-system-module-display-internal-names-1760892-1.patch	6.76 KB

Here's the .patch that implements this feature.

Comment #2

CydeWeys commented 28 August 2012 at 20:22

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.

Comment #3

cameron tod commented 28 August 2012 at 21:39

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!

Comment #4

CydeWeys commented 28 August 2012 at 21:50

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.