node

    moderate: unexpected column in database
    indexes changed: unexpected (not an error)
    indexes node_moderate: unexpected (not an error)

node_revision

    column status - difference on: default
    declared: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'not null' => TRUE, 'default' => 1)
    actual: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'not null' => TRUE, 'default' => 0)
    indexes timestamp: unexpected (not an error)

node_type

    column type - difference on: default
    declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 32, 'not null' => TRUE)
    actual: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => '32', 'not null' => TRUE, 'default' => '')
    column module - difference on: default
    declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 255, 'not null' => TRUE)
    actual: array('description' => 'The module defining this node type.', 'type' => 'varchar', 'length' => '255', 'not null' => TRUE, 'default' => '')
    column has_title - difference on: default
    declared: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'unsigned' => TRUE, 'size' => 'tiny', 'not null' => TRUE)
    actual: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'unsigned' => TRUE, 'size' => 'tiny', 'not null' => TRUE, 'default' => 0)
CommentFileSizeAuthor
#1 node-upgrade-schema-1154808.patch880 byteslyricnz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lyricnz’s picture

Version: 7.0 » 7.x-dev
Status: Active » Needs review
FileSize
880 bytes

Fix issues reported by current schema module.

catch’s picture

Priority: Normal » Major
Issue tags: +D7 upgrade path
marcingy’s picture

Status: Needs review » Reviewed & tested by the community

Looks good.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x. Thanks!

Status: Fixed » Closed (fixed)

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

David_Rothstein’s picture

I found this issue while looking for something else, but note that {node}->moderate was deliberately left in the table for sites upgrading from D6, since contrib modules use it and the intention was not to destroy any data. (See discussion in #564462: Remove moderate column from node_schema().)

I'm not sure if that reasoning still applies, but if so, that part of the patch here should be reverted and/or pushed to D8.