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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TDobes’s picture

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 as well.

chx’s picture

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

killes@www.drop.org’s picture

Title: delete from cache/ remove old updates » remove old updates
Status: Closed (won't fix) » Needs review
FileSize
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.

chx’s picture

/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.

Dries’s picture

Status: Needs review » Fixed

Committed to HEAD.

Patriciafish’s picture

Anonymous’s picture

Anonymous’s picture

Anonymous’s picture

Status: Fixed » Closed (fixed)