(the issue's title might be ironic, but it is meant to emphasize the problem) I have spotted these:

'Publishing options' during node create
- 'Expiration Date:' field name
In content type 'Workflow settings':
- 'Expiration Date:' field name (should be 'Default expiration date:' btw)
- 'Expiration Date Limit:' field name
- 'Expiration Date Required' checkbox

...might be others as well. Please help spot them and fix them all.

I know, I know... a lot of other modules do this. It is Drupal standard to not capitalize module/theme names except the first letter of the first word though. So it should also be 'Node expire' instead of 'Node Expire'. Drupal capitalization standard:

name (Required)
The displayed name of your module. It should follow the Drupal capitalization standard: only the first letter of the first word is capitalized ("Example module", not "example module" or "Example Module"). Spaces are allowed as the name is used mainly for the display purposes.

Comments

klonos’s picture

Issue tags: +do not capitalize name

...added a tag to help follow up same issues in other modules.

vikramy’s picture

Assigned: Unassigned » vikramy
Status: Active » Fixed

Fixed in latest dev and in head. If you find more feel free to reopen. Thanks by the way for making me follow standards.

klonos’s picture

No problem mate and thanx for fixing it in a snap! We should be thanking you for your time and effort on this module. I will let you know if I find any other bits that need fixing ;)

...BTW, could you take a look at this while you're at it?: #870042: Add the module to a (new) package category (a.k.a. unclutter the default 'Other' category)

klonos’s picture

...I think you should also fix capitalization of the name in the module's front page too.

vikramy’s picture

You mean in README.txt?

klonos’s picture

Yeah some places in the README.txt too, but I was referring to the title in the project's page: http://drupal.org/project/node_expire (it still reads 'Node Expire' instead of 'Node expire')

Status: Fixed » Closed (fixed)

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

michelle’s picture

klonos’s picture

Title: Unnecessary Capitalization Of Words Throughout Forms + Module Name » Unnecessary Capitalization Of Words Throughout Forms
Status: Closed (fixed) » Active

...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.

Nafes’s picture

Issue summary: View changes
Status: Active » Closed (works as designed)

  • vikramy committed d79c504 on feature-cleanup
    #869394: Drupal coding standards