Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#1 | backup_migrate-file_private_path-1938270-01.patch | 767 bytes | micahw156 |
backup_migrate-8x-private-file-system-bug.png | 113.75 KB | micahw156 |
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
victor-shelepen CreditAttribution: victor-shelepen 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 CreditAttribution: 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) CreditAttribution: commentedAdded img tag.
Comment #6
lcgondellesm CreditAttribution: lcgondellesm commentedHello, Can anyone guide me to install this patch? Thanks in advance.
Comment #7
dddave CreditAttribution: dddave commentedhttps://www.drupal.org/patch/apply