[drupal-devel] Archive Module
dries at buytaert.net
Wed Apr 20 13:03:33 UTC 2005
On 20 Apr 2005, at 14:38, Jim Riggs wrote:
> Although I like this module a lot, if it is going to be in contrib, it
> has to behave like a contrib module. That is, it should not patch
> drupal.css, but use its own css file inside the module's directory.
> And, it should not have a name that conflicts with a core module.
> This will cause a bunch of problems and support issues as
> (non-developer) users stumble across it and try it. If you want to
> leave it as is (patching core and conflicting with a core module), it
> should stay in your sandbox.
> I suggest making it a true contrib module by changing the name of the
> cvs directory/module and accessing the css from the module's
> directory. This way people can use it until it -- hopefully -- makes
> it into core, replacing archive.
The module is meant for core, and therefore belongs in a sandbox (IMO).
Once ready for inclusion, please provide a patch and upload it to the
patch queue. In the mean time, everyone is encourage to review/discuss
the poroposed changes.
Dries Buytaert :: http://www.buytaert.net/
More information about the drupal-devel