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.

CommentFileSizeAuthor
ctm.info.patch425 bytesTBarregren

Comments

rickvug’s picture

rickvug commented

I agree with all points.

Gary Feldman’s picture

Gary Feldman commented

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.

vasike’s picture

vasike
Location Ramnicu Valcea
commented
Status: Active » Closed (fixed)