Remove 5.x-6.x updates

catch - July 4, 2008 - 15:58
Project:Drupal
Version:7.x-dev
Component:install system
Category:task
Priority:normal
Assigned:catch
Status:patch (code needs review)
Description

During the 6.x release cycle, we removed all 4.7-5.x updates to enforce the one major version upgrade at a time guideline more strictly, reduce memory usage on update.php, etc.

Here's a patch which does that, and also a bit of tidying in .install files - adding update phpdoc if it's missing, getting hooks in the same order etc.

AttachmentSize
old-updates.patch85.59 KB

#1

webernet - July 5, 2008 - 12:49
Status:patch (code needs review)» patch (code needs work)

There are a couple blocks of code in update.php for the 5.x-6.x update as well.

#2

catch - July 7, 2008 - 14:46
Status:patch (code needs work)» patch (code needs review)

Well spotted, new patch.

AttachmentSize
old_updates.patch89.16 KB

#3

webernet - July 7, 2008 - 21:19
Status:patch (code needs review)» patch (code needs work)

Looks like there's (at least) one more...

<?php
/**
* Create the batch table.
*
* This is part of the Drupal 5.x to 6.x migration.
*/
function update_create_batch_table() {
?>

#4

catch - July 7, 2008 - 21:32
Status:patch (code needs work)» patch (code needs review)

Got that one. Did a grep, found some in blogapi.install - this should be everything now.

Hopefully this is all of them. Thanks for keeping your eyes peeled webernet.

AttachmentSize
old_update.patch91.75 KB

#5

lilou - August 23, 2008 - 20:12

Patch #4 re-rolled against CVS HEAD.

AttachmentSize
old_update_1.patch92.37 KB

#6

catch - September 16, 2008 - 21:35
Status:patch (code needs review)» patch (code needs work)

testing the testing bot

#7

catch - September 16, 2008 - 21:35
Status:patch (code needs work)» patch (code needs review)

and again.

 
 

Drupal is a registered trademark of Dries Buytaert.