Notice: Undefined index: exclude_tables in backup_migrate_destination_db->backup_settings_form() (line 122 of destinations.db.inc).
Notice: Undefined index: nodata_tables in backup_migrate_destination_db->backup_settings_form() (line 130 of destinations.db.inc).

This PHP notice appears on the advance backup form when you set multiple database connection details in settings.php. Only the settings for the default database seem to have the index exclude_tables and nodata_tables.

This is just a PHP notice and is not causing any disruption that I can see. I will supply a patch to not set the default values in the form if these values are undefined.

CommentFileSizeAuthor
#1 backup_migrate-undefined_exclude_tables-1600532-1.patch1.26 KBnlisgo

Comments

nlisgo’s picture

nlisgo commented
StatusFileSize
new backup_migrate-undefined_exclude_tables-1600532-1.patch1.26 KB

Here is the patch:

spessex’s picture

spessex commented

Using Drupal 7.18

I have the same problem as I have a second database due to connecting a Lime Survey Database (Lime Sync). However I have applied the suggested patch but appear to have an error as demonstrated below:

Stephens-MacBook-Pro:backup_migrate spessex$ patch < backup_migrate-undefined_exclude_tables-1600532-1.patch
can't find file to patch at input line 5
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/includes/destinations.db.inc b/includes/destinations.db.inc
|index 27d947b..8aeccd5 100644
|--- a/includes/destinations.db.inc
|+++ b/includes/destinations.db.inc
--------------------------
File to patch:

ronan’s picture

ronan commented
Status: Needs review » Closed (fixed)

I think this got fixed in a different issue, but reopen if I'm mistaken.

dkmn’s picture

dkmn commented

I am seeing this issue with Core 7.24 and Backup and Migrate 7.x-2.8. It's either re-occured or perhaps wasn't fixed initially... If I understand the significance, it's not a blocking issue, but I wanted to note this for visibility.

dkmn’s picture

dkmn commented
Issue summary: View changes
Status: Closed (fixed) » Active
couturier’s picture

couturier commented
Status: Active » Closed (outdated)

Please re-open if this is still an issue in the latest 7.x-3.2 version. Note that the 2.x branch is being deprecated.