According to these:

Writing .info files (Drupal 7.x)
#1346158: Decide on coding standard for capitalization of module names [policy, no patch]

Proper Case should be used. So, "Meta Tags" instead of "Meta tags". Unless you choose to go with "Metatags" of course ;)

Comments

damienmckenna’s picture

StatusFileSize
new14 KB

How would this look? Given the module's internal name is "metatag" it makes sense that the module's proper name be "Metatag".

damienmckenna’s picture

Status: Active » Needs review
dave reid’s picture

Hrm, I don't think it makes sense to make the internal name match exactly the the module name. 'Meta tag' is the proper term and I don't think it should be changed.

klonos’s picture

Well, I personally don't have any preference - I was merely pointing the fact that the project needs to "comply" with d.o standards set. At the end I guess it's up to you (since you are the maintainer).

PS: whatever you choose to go with, remember to also change the project title here in d.o too ;)

damienmckenna’s picture

The module name is a little confusing, e.g. the first paragraph of http://buildamodule.com/video/advanced-site-building-in-drupal-7-how-to-...

damienmckenna’s picture

Current status:

  • We're confusing people by using two words in the project's full name but not using an underline in the short name.
  • We're confusing people by using the plural form of the word "tag" in the project's full name but not in the short name.

I recommend we rename the module's full name to "Metatag", thus resolving the two reasons for confusion, and be done with it.

damienmckenna’s picture

I think we'd get by using "Metatag" for the project name instead of the arguably more accurate "Meta tags" because we're using it as a proper noun, so we can name it whatever we want. Also, it's too late to rename the short name to "meta_tag". And I won't even mention how the hooks use the string "metatags" rather than "metatag" or "meta_tags" (oh wait, I did).

klonos’s picture

... (oh wait, I did).

:D

I'm afraid I've opened a whole bigger can of worms than I thought when I filed this issue :/

dave reid’s picture

Hah, which hooks should be fixed?

damienmckenna’s picture

@Dave: Correction, there's only one hook with the incorrect string: hook_metatag_metatags_view_alter. However, there are fourteen internal functions use that name, but fixing that won't resolve the confusion people have over the project name.

damienmckenna’s picture

StatusFileSize
new9.14 KB

Rerolled. I really want to include this in the next release.

damienmckenna’s picture

StatusFileSize
new11.33 KB

This adds one or two additional changes of "Meta tags" to just say "Metatag".

damienmckenna’s picture

Status: Needs review » Fixed

I've committed this as I really do think this will help simplify the site builder's user experience.

klonos’s picture

Do not forget the project name here on d.o too.

damienmckenna’s picture

@klonos: I'm going to do that as soon a I release the next beta.

damienmckenna’s picture

Status: Fixed » Closed (fixed)

Now that v7.x-1.0-beta7 is out, I'm closing this in the interest of keeping the issue queue clean.

klonos’s picture

Thanx!