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

CommentFileSizeAuthor
#3 updates_3.patch69.64 KBkilles@www.drop.org
updates_0.patch72.62 KBkilles@www.drop.org

Comments

TDobes’s picture

TDobes commented

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

chx commented

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

killes@www.drop.org’s picture

killes@www.drop.org commented
Title: delete from cache/ remove old updates » remove old updates
Status: Closed (won't fix) » Needs review
StatusFileSize
new updates_3.patch69.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

chx commented

/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

dries commented
Status: Needs review » Fixed

Committed to HEAD.

Patriciafish’s picture

Patriciafish commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)