With modern PHP there's no need to track which files are loaded, just use include_once() when needed and the rest will be taken care of.

CommentFileSizeAuthor
#4 backup_migrate-n3154166-4.patch43.9 KBDamienMcKenna
#3 backup_migrate-n3154166-3.patch2.53 KBDamienMcKenna
#2 backup_migrate-n3154166-2.patch42.65 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna 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-n3154166-2.patch42.65 KB

Let's see if this works.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
backup_migrate-n3154166-3.patch2.53 KB

This fixes the tests locally, hopefully it'll work ok with the testbot.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
backup_migrate-n3154166-4.patch43.9 KB

Doh. #3 is the interdiff, this is the full patch.

Status: Needs review » Needs work

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

  • DamienMcKenna committed ad4b5ed on 7.x-3.x 
    Issue #3154166 by DamienMcKenna: Remove use of backup_migrate_include()...
DamienMcKenna’s picture

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

Finally.

Committed.

Status: Fixed » Closed (fixed)

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