D7.22, shema 7.x-1.0-rc1

backup_migrate

backup_migrate_profiles

•column profile_id - difference on: length
declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 255, 'not null' => TRUE, 'default' => '0')
actual: array('description' => 'The primary identifier for a profile.', 'type' => 'varchar', 'length' => '32', 'not null' => TRUE, 'default' => '0')

backup_migrate_destinations

•column destination_id - difference on: length
declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 255, 'not null' => TRUE, 'default' => '0')
actual: array('description' => 'The primary identifier for a profile.', 'type' => 'varchar', 'length' => '32', 'not null' => TRUE, 'default' => '0')

backup_migrate_schedules

•column schedule_id - difference on: length
declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 255, 'not null' => TRUE, 'default' => '0')
actual: array('description' => 'The primary identifier for a profile.', 'type' => 'varchar', 'length' => '32', 'not null' => TRUE, 'default' => '0')
•column source_id - difference on: length
declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 255, 'not null' => TRUE, 'default' => 'db')
actual: array('description' => 'The backup_migrate_destination.destination_id of the source to backup from.', 'type' => 'varchar', 'length' => '32', 'not null' => TRUE, 'default' => 'db')
•column destination_id - difference on: length
declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 255, 'not null' => TRUE, 'default' => '0')
actual: array('description' => 'The backup_migrate_destination.destination_id of the destination to back up to.', 'type' => 'varchar', 'length' => '32', 'not null' => TRUE, 'default' => '0')
•column profile_id - difference on: length
declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 255, 'not null' => TRUE, 'default' => '0')
actual: array('description' => 'The primary identifier for a profile.', 'type' => 'varchar', 'length' => '32', 'not null' => TRUE, 'default' => '0')

CommentFileSizeAuthor
#5 backup_migrate-1997166.patch422 bytestobiasb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ronan’s picture

Ah, quite so. This shouldn't cause any issue on your site, but I'll put an update hook in to clean up this discrepancy.

ronan’s picture

Status: Active » Fixed

There's an update hook in the latest dev that should fix this.

devad’s picture

I tried last 7.x-2.x-dev. No schema mismatshes. Fixed. Thnx.

Status: Fixed » Closed (fixed)

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

tobiasb’s picture

Issue summary: View changes
Status: Closed (fixed) » Needs work
FileSize
422 bytes

I take a look at this update hook and saw that $schema = backup_migrate_schema(); overrides the added schema in http://drupalcode.org/project/backup_migrate.git/commitdiff/594e9d79fac9... . I believe this was forgotten to remove, right?

* The attached patch removes that again.

tobiasb’s picture

Version: 7.x-2.7 » 7.x-2.x-dev
Status: Needs work » Needs review
couturier’s picture

Status: Needs review » Closed (outdated)

Ronan marked this issue as fixed and didn't respond to further comments. The 7.x-2.x branch is being deprecated. If this is still a problem in newer versions, please re-open.