Issues with the .info-file [#451556]

First of all, thanks for this much appreciated module. It should really be in core. Now, to the issues:

The .info-file doesn't reflect that this module depends on the Menu module. Add dependencies[] = menu to the .info-file.

Also, project should not be in the .info-file.

Finally, I suggest that the package Other is used instead of Administration.

The attached patch solves all these issues.

Comment	File	Size	Author
	ctm.info.patch	425 bytes	TBarregren

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

rickvug CreditAttribution: rickvug commented 6 June 2009 at 20:54

I agree with all points.

Comment #2

Gary Feldman CreditAttribution: Gary Feldman commented 30 June 2009 at 14:08

I agree with everything except that I really dislike using Other as a dumping ground for all modules that may not be part of their own collection, but still have an identifiable subject area. In this case, the Administration category is already being used by a number of other stand-alone modules, and makes sense. Of course, if the feature gets rolled into core for D7, the issue is moot; it probably shouldn't even be a module, just a feature made available by core.