[drupal-devel] Archive Module

[Dries Buytaert]

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/