As for #1464554: Upgrade path for image style configuration image module has now hook_update_N functions which is affecting the upgrade path testing (see #1292470-138: Convert user pictures to Image Field) so the schema version in the testing databases should be also changed.

The only change is

->values(array(
  'filename' => 'modules/image/image.module',
  'name' => 'image',
  'type' => 'module',
  'owner' => '',
  'status' => '0',
  'bootstrap' => '0',
  'schema_version' => '8000', <-- From -1 to 8000
  'weight' => '0',

Patch attached.

CommentFileSizeAuthor
add_schema_version_image_upgrade.patch28.95 KBpcambra
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

valthebald’s picture

valthebald’s picture

Status: Needs review » Reviewed & tested by the community

That's a valid point, and the test bot is happy. Marking RTBC...

alexpott’s picture

Status: Reviewed & tested by the community » Needs review

Is this really the right thing to do... surely a drupal 7 site will have a schema version of -1???? no?

pcambra’s picture

It made sense in the context of #1464554: Upgrade path for image style configuration at the time, I'm not sure if this is needed anymore, minimal profile doesn't include image module, does it?

alansaviolobo’s picture

Issue summary: View changes

the code doesn't seem to show any changes after applying the patch.

jhedstrom’s picture

Status: Needs review » Closed (works as designed)

The minimal install does not include the image module. Closing this out.