Let's bump things like core = 7.x in .info files!

CommentFileSizeAuthor
#1 1089320-bump-version.patch84.88 KBdave reid

Comments

dave reid’s picture

Status: Active » Needs review
StatusFileSize
new84.88 KB

Summary:
- Changes all the 'core = 7.x' lines in .info files to 8.x
- Updates core compatability constants
- Fixes CHANGELOG.txt, MAINTAINERS.txt (to remove Angie - sadness), and UPDATE.txt

quicksketch’s picture

Thanks for getting on this Dave! Unless there's already an issue for it, I'll start working on removing all the 7xxx updates so we can add in the 8000 ones (when they exist).

quicksketch’s picture

wizonesolutions’s picture

Tested applying this - applies cleanly, all changes look to be there. +1.

Also, this is my first core patch review! Start at the beginning, right? :D

donSchoe’s picture

It's applying cleanly and working. But "avaibale updates" page tells me:

Drupal core 8.0-dev Unknown release date
Recommended version: 8.x-dev (2011-Mar-10)

You might wish to change 8.0-dev to 8.x-dev? :)
I'm not 100% sure but I think 8.0 is the number of a final release, it should be 8.x.

But hey, patch is looking good so far. :)

Edit: changed it to 8.x-dev manually and it's still complaining "Unknown release date". Not sure why. (I need to mention I have a live website running the "8.x-dev" release as of Mar-10, 2011 and that's not complaining :)

dries’s picture

Status: Needs review » Fixed

Nice one. Committed to CVS HEAD.

damien tournoud’s picture

Status: Fixed » Needs work
   $system_schema = drupal_get_installed_schema_version('system');
   if ($system_schema < 7000) {
-    $has_required_schema = $system_schema >= REQUIRED_D6_SCHEMA_VERSION;
+    $has_required_schema = $system_schema >= REQUIRED_D7_SCHEMA_VERSION;
     $requirements = array(

The 7000 here probably need to be bumped to 8000 too.

--- a/modules/simpletest/tests/upgrade/drupal-6.filled.database.php
+++ b/modules/simpletest/tests/upgrade/drupal-6.filled.database.php

Please revert this change.

-    // During a Drupal 6.x to 7.x update, drupal_get_schema() does not contain
+    // During a Drupal 6.x to 8.x update, drupal_get_schema() does not contain
     // the queue table yet, so we cannot rely on drupal_write_record().
     $query = db_insert('queue')
       ->fields(array(

7.x to 8.x?

quicksketch’s picture

The 7000 here probably need to be bumped to 8000 too.

DamZ: Could you look at #1097100: Remove all 7xxx update functions and tests (D6 to D7 upgrade path)? This fixes most of the issues you raise and deals with the update functions and the "drupal-6.*.database.php" files by removing them (since we don't need to test upgrading from D6 to D8 anyway). It might be a bit aggressive, but right now there's nothing to test when it comes to upgrading from D7 to D8.

donSchoe’s picture

Status: Needs work » Fixed

See #8.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.