Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It should be 'Comment notify' instead of 'Comment Notify'. Please see this related issue I filed in Coder's issue queue for more info: #1103208: Projects' names should be checked in .info files for correct capitalization.
Comment | File | Size | Author |
---|---|---|---|
#8 | 1103232_title_case.patch | 2.76 KB | greggles |
Comments
Comment #1
gregglesFixed, thanks - http://drupalcode.org/project/comment_notify.git/commit/726886e
I also fixed the project page on d.o.
Comment #2
klonosThanx Greg ;)
Comment #4
MichelleSee #1346158: Decide on coding standard for capitalization of module names [policy, no patch] and #1430452: Use Proper Name Case for core modules for current information on this policy.
Comment #5
klonos...yep, the standard has changed and it now allows (more likely suggests) Proper Name Case instead of the Sentence case for module/theme names. The Sentence case policy remains for UI text like widget labels for example.
Comment #6
Dave ReidThe module is not really a proper name and is meant to be generic. I don't see any reason to revert this but I'll leave it up to Greg.
Comment #7
gregglesThe argument that sways me the most is that "Using the Comment Notify module to send e-mails" is clear while "Using the Comment notify module to send e-mails" is not clear.
Patch, anyone?
And if anyone wants to work on the docs pages please go for it.
Comment #8
gregglesOk, found a few more spots that were inconsistent on the project page and elsewhere.
http://drupalcode.org/project/comment_notify.git/commit/7b91583