Pursuant to the discussion on the devel list here: http://lists.drupal.org/pipermail/development/2006-November/021069.html it looks like we need to do this quickly before this gets spiralling out of hand.
The gist is:
- The "package = " entry in .info files should be restricted to modules which are actually part of a package. For example, "Views Bonus Pack" module is useless without "Views" module, thus is part of the Views package.
- "Uncategorized" makes it sound like the maintainer was too lazy to categorize their module, and therefore module maintainers are going bonkers making up every package name under the sun, when in actual fact very few modules should actually have packages.
- The package attribute should specifically not be for categorization, both because categories are in many cases a many-to-many relationship, which is not desirable nor even feasible, and also because people really tend to suck at categorizing things.
I have almost marked this "critical" because this is already getting out of hand and there hasn't been a release yet. But at the very least it's very high priority and needs to get sorted.
Comment | File | Size | Author |
---|---|---|---|
#3 | system-remove-all-doubt.patch | 635 bytes | bdragon |
Comments
Comment #1
webchickOh and because I somehow managed to leave this out entirely, this issue is referring to the admin/build/modules page.
Comment #2
webchickBtw, there is already documentation that alludes to this in the updating docs:
http://drupal.org/node/64279#info
However, it should probably be changed to make it more explicit, that the only reason you should add a package name is if your module is part of a larger collection of modules.
Comment #3
bdragon CreditAttribution: bdragon commented+1.
Comment #4
merlinofchaos CreditAttribution: merlinofchaos commentedI don't think it should necessarily be bottom of list.
I definitely agree Uncategorized causes confusion.
Comment #5
webchickWorks for me. RTBC.
Comment #6
moshe weitzman CreditAttribution: moshe weitzman commentedYes. Nice.
I repent for my mistake. I changed it to Uncategorized a while ago.
Comment #7
Morbus IffWorks for me. +1.
We need to update the handbook docs however - we should NOT use the word "category" or "categorized" anywhere.
Comment #8
Dries CreditAttribution: Dries commentedGood stuff! Committed to CVS HEAD.
Comment #9
(not verified) CreditAttribution: commented