Running Backup Migrate against 8.x-dev (commit 957b556), I get the message "You must specify a private file system path in the file system settings to backup to the server." even though this value is set in the configuration.

Comments

micahw156’s picture

Here'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.

Anonymous’s picture

Status: Active » Reviewed & tested by the community
micahw156’s picture

Status: Reviewed & tested by the community » Needs work

Thanks 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.

ronan’s picture

Status: Needs work » Fixed

Patch 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.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

Added img tag.

lcgondellesm’s picture

Hello, Can anyone guide me to install this patch? Thanks in advance.

dddave’s picture