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

Incorrect retrieval of file_public_path #129

Closed argiepiano closed 9 months ago

argiepiano commented 9 months ago

In D7, class backup_migrate_files_destination_archivesource gets the file_public_path by using variable_get(). When backup_migrate was ported, this was adapted by using config_get('backup_migrate.settings','file_public_path'), which, of course, is wrong. In BAckdrop the counterpart of the variable file_public_path is obtained by doing config_get('system.core', 'file_public_path'). This should be changed.

argiepiano commented 9 months ago

PR #130

argiepiano commented 9 months ago

Will close this since this was addressed more comprehensively in #136