backdrop-contrib / backup_migrate

Port of the Drupal backup_migrate module to Backdrop
GNU General Public License v2.0
7 stars 14 forks source link

Wrongly uses `file_private_path` from file `backup_migrate.settings.json` #99

Closed argiepiano closed 1 year ago

argiepiano commented 1 year ago

When the module was first ported, the variable file_private_path (which is a Drupal 7 variable, handled by D7's file system) was wrongly ported as a backup_migrate proprietary config setting. This resulted in the backup_migrate_update_1000() creating that config setting, and a few spots in the code doing things like:

config_get('backup_migrate.settings', 'file_private_path')

which always returns false, since this setting is not set by BM. Instead, those lines should be replaced by

config_get('system.core', 'file_private_path') to access the "real" private file path.