Please don't use .info package = 'Custom Modules by Himerus'

Dave Reid - October 24, 2008 - 23:39
Project:ShareThis
Version:6.x-1.x-dev
Component:Miscellaneous
Category:task
Priority:minor
Assigned:Unassigned
Status:closed
Description

In the sharethis.info file, please don't use 'Custom Modules by Himerus' as the module's package name. This module is not part of a large package like CCK or Views (intended use), and the package should not be provided so that it can be classified under 'Other.' The current package name makes the modules list page more complex than it should be.

See http://drupal.org/node/231036 for D-6 module .info files:

If your module comes with other modules or is meant to be used exclusively with other modules, enter the name of the package here. If left blank, the module will be listed as 'Other'. In general, this field should only be used by large multi-module packages, or by modules meant to extend these packages, such as CCK, Views, E-Commerce, Organic Groups and the like. All other modules should leave this blank. As a guideline, four or more modules that depend on each other (or all on a single module) make a good candidate for a package. Fewer probably do not.

#1

Dave Reid - November 16, 2008 - 06:11

Any update on this?

#2

Rob Loach - November 26, 2008 - 19:33
Status:active» needs review

Here you go...

AttachmentSize
325870.patch 686 bytes

#3

Dave Reid - December 4, 2008 - 19:36
Status:needs review» reviewed & tested by the community

Looks good.

#4

Rob Loach - December 16, 2008 - 22:24
Status:reviewed & tested by the community» fixed

http://drupal.org/cvs?commit=159316

#5

System Message - December 30, 2008 - 22:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.