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.

Support from Acquia helps fund testing for Drupal Acquia logo

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.

victor-shelepen’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