In admin/build/modules, this module's name shows up as Phone - CCK. The name should just be Phone to match other CCK module names.

CommentFileSizeAuthor
#5 phone.info351 bytesChristefano-oldaccount

Comments

vm’s picture

vm commented

you can easily change this yourself in the modules .info file

aren cambre’s picture

aren cambre commented

You are correct, but this needs to be corrected in the module itself if consistency is a priority for the broader Drupal project.

vm’s picture

vm commented

Submitting a patch would help the developer make the corrections.

aren cambre’s picture

aren cambre commented

I would, but I haven't learned the patch or .info system enough to feel comfortable doing that yet.

Christefano-oldaccount’s picture

Christefano-oldaccount commented
StatusFileSize
new phone.info351 bytes

Aside from unified patches that created .info files, I don't think I've ever seen a patch just for a .info file. Developers just attach them to the issue.

Christefano-oldaccount’s picture

Christefano-oldaccount commented
Category: bug » feature
aren cambre’s picture

aren cambre commented
Version: 5.x-1.5 » 6.x-2.14

Now even more important if this module will be ported to D7 since CCK has been integrated to core and renamed Fields.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

The project's name can remain as is (plain "Phone") and it could be "Phone - CCK" for the 6.x .info and "Phone field"/"Phone fields" for the 7.x .info file. This would both make sense and at the same time clearly imply/explain why the one requires CCK while the other requires D7 core Field API.