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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

VM’s picture

VM CreditAttribution: VM commented

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

Aren Cambre’s picture

Aren Cambre CreditAttribution: 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 CreditAttribution: VM commented

Submitting a patch would help the developer make the corrections.

Aren Cambre’s picture

Aren Cambre CreditAttribution: 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 CreditAttribution: Christefano-oldaccount commented
FileSize
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 CreditAttribution: Christefano-oldaccount commented
Category: bug » feature
Aren Cambre’s picture

Aren Cambre CreditAttribution: 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
CreditAttribution: klonos 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.