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 CreditAttribution: CydeWeys commentedHere's the .patch that implements this feature.
Comment #2
CydeWeys CreditAttribution: CydeWeys commentedI 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 CreditAttribution: Cameron Tod commentedPatch 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 CreditAttribution: CydeWeys commentedUnfortunately 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.
Comment #5
NancyDruMenu 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.
Comment #6
CydeWeys CreditAttribution: CydeWeys commentedOK, all of those issues have been addressed in this .patch file.
Comment #7
CydeWeys CreditAttribution: CydeWeys commentedComment #8
NancyDruOkay, I'll check it out.
Comment #9
NancyDruCommitted, thanks for the patch.