_backup_migrate_setup_database_defaults() includes five files from the current working directory.
backup_migrate_cron() includes './includes/common.inc' from the current working directory.

Should these functions use backup_migrate_include() or merely be corrected to use DRUPAL_ROOT?

Note: backup_migrate_include() uses DRUPAL_ROOT correctly.

This should really just be a case of replacing each:
'./' +
with:
DRUPAL_ROOT + '/' +

Thanks,
JP

CommentFileSizeAuthor
#1 backup_migrate-DRUPAL_ROOT-990954-2.patch2.64 KBgaelicWizard

Comments

gaelicWizard’s picture

gaelicWizard commented
Title: DRUPAL_ROOT constant missing from most require_once() calls » Use DRUPAL_ROOT constant
Status: Active » Needs review
StatusFileSize
new backup_migrate-DRUPAL_ROOT-990954-2.patch2.64 KB
ronan’s picture

ronan commented
Status: Needs review » Fixed

Patch committed. Thanks!

Status: Fixed » Closed (fixed)

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