backup_migrate_update_7305()

if (!db_field_exists($table, $machine_name)) {

$machine_name is never declared.

CommentFileSizeAuthor
#2 backup_migrate-n3143362-2.patch478 bytesDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

solideogloria created an issue. See original summary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Active » Needs review
FileSize
backup_migrate-n3143362-2.patch478 bytes

Gah! Sorry about that.

I think it's supposed to be just `machine_name`.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: » #3116785: Plan for Backup and Migrate 7.x-3.8

Status: Needs review » Needs work

The last submitted patch, 2: backup_migrate-n3143362-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review

The test failure was unrelated.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed

Committed. Thank you.

Status: Fixed » Closed (fixed)

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