Closed (fixed)
Project:
Backup and Migrate
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Mar 2013 at 18:00 UTC
Updated:
30 Apr 2016 at 08:58 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
micahw156Here's a patch that replaces variable_get for file_private_path with a call to config('system.file') and then uses method get('path.private') to retrieve the file. This only moves the problem. backup_migrate_create_destination() doesn't create the target directories, so that's probably the next thing to fix.
Edit: Also, submitted patch isn't the one I meant to post, as it introduces an unnecessary variable. I'll fix that on my next update.
Comment #2
Anonymous (not verified) commentedComment #3
micahw156Thanks for the review of work done so far.
The patch in #1 fixes the original error, but backups still do not work. This needs more changes before it's ready.
Comment #4
ronan commentedPatch applied and I also fixed an issue with the local file security check now that drupal_http_request is gone. Backup to server seems to be working now.
Comment #5.0
(not verified) commentedAdded img tag.
Comment #6
lcgondellesm commentedHello, Can anyone guide me to install this patch? Thanks in advance.
Comment #7
dddave commentedhttps://www.drupal.org/patch/apply