Hi.
I noticed something weird around line 139 in backup_migrate.drush.inc:

  $settings = array('source_id' => $source_id);
  backup_migrate_perform_restore($destination_id, $file_id);

In a nutshell the settings array is correctly initialised but not passed to backup_migrate_perform_restore, which will cause the source parameter to be simply ignored.
Needless to say this bug may cause an unwanted database restore (since db is the default source_id).

Thanks for the awasome module.

CommentFileSizeAuthor
backup_migrate_drush_restore_source.patch520 bytesepifab
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ronan’s picture

Status: Active » Fixed

Looks good. Commited. Thanks!

Status: Fixed » Closed (fixed)

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