In #2774487: Enable using bz2 for backup files I introduce a function that can be used to remove some duplicate code from the hosting_tasks_extra module.

CommentFileSizeAuthor
#6 0001-Issue-2778443-Fix-for-missing-include.patch1.03 KBmemtkmcc
#2 refactor_backup-2778443-2.patch941 byteshelmo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

helmo created an issue. See original summary.

helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commented
Status: Active » Needs review
FileSize
refactor_backup-2778443-2.patch941 bytes

  • helmo committed 83352fa on 7.x-3.x 
    Issue #2778443 by helmo: refactor backup filename code to use core...
helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commented
Status: Needs review » Fixed
memtkmcc’s picture

memtkmcc CreditAttribution: memtkmcc commented
Category: Feature request » Bug report
Priority: Normal » Critical
Status: Fixed » Needs work

This breaks the db update task completely.

You can't call drush_provision_drupal_provision_backup_suggest_filename() without adding include_once($provision_dir . '/platform/backup.provision.inc'); first.

memtkmcc’s picture

memtkmcc CreditAttribution: memtkmcc at Omega8.cc commented
Status: Needs work » Needs review
FileSize
0001-Issue-2778443-Fix-for-missing-include.patch1.03 KB

Attached patch fixes the problem.

  • helmo committed 3a8a5fd on 7.x-3.x authored by memtkmcc 
    Issue #2778443 by helmo, memtkmcc: refactor backup filename code to use...
helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commented
Status: Needs review » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

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