[drupal-devel] [task] remove old updates

chx drupal-devel at drupal.org
Tue Aug 16 14:01:10 UTC 2005

Issue status update for 
Post a follow up: 

 Project:      Drupal
 Version:      cvs
 Component:    base system
 Category:     tasks
 Priority:     normal
 Assigned to:  killes at www.drop.org
 Reported by:  killes at www.drop.org
 Updated by:   chx
 Status:       patch (code needs review)

/me feels stupid for his May reaction. Especially that when this issue
was raised again on IRC, I was the first to mention the CVS repos...
baaaah! So, I am fine with removing updates.


Previous comments:

Sun, 31 Oct 2004 22:48:40 +0000 : killes at www.drop.org

Attachment: http://drupal.org/files/issues/updates_0.patch (72.62 KB)

The attached patch removes the updates 80 through 109 from updates.inc
and adds a "delete from cache" statement after updating.


Sun, 31 Oct 2004 23:27:40 +0000 : TDobes

big, big, big -1 on removing the old updates.

Having these updates around makes life much easier for those upgrading
from older versions and provides quick code samples on how to populate
certain fields and tables for those migrating from other CMS'.  I know,
in particular, that the PHP code for the node_comment_statistics table
and the thread field of the comments table have been infinitely
valuable to me when navigating sites from other CMS'.

I like the idea of clearing the cache after each run of update.php. 
There is a specific issue for it [1] as well.
[1] http://drupal.org/node/12350


Sun, 13 Mar 2005 19:08:27 +0000 : chx

tdobes is right there for I set this to "won't fix".


Tue, 16 Aug 2005 13:44:26 +0000 : killes at www.drop.org

Attachment: http://drupal.org/files/issues/updates_3.patch (69.64 KB)

Don't recall why I wanted the delete from cache, but this patch removes
all update prior to 4.5.0.
Peopel who want inspiration from old updates still have plenty of them
in the file and can look into our cvs repos.

More information about the drupal-devel mailing list