cleansing my watchdog table i found this. now i saw there's a 2.x branch going to switch now :)

CommentFileSizeAuthor
backup_migrate-fix-undefined-variables.patch886 byteseMPee584

Comments

ronan’s picture

ronan commented

Thanks for the patch, nice find. I'll get this applied as soon as i have a chance.

Be careful with the 2.x branch, it's still very much pre-alpha, but I'd love to hear your experiences with it (bugs etc).

Ronan

ronan’s picture

ronan commented
Status: Needs review » Fixed

Patch is now in both the 6.x and 5.x branches, thanks again for the patch.

eMPee584’s picture

eMPee584 commented
Version: 6.x-1.x-dev » 6.x-2.x-dev

Oh come on i am fed up fixing php errors in all modules i try and run on my site it has friggin stopped being funny. Would all drupal module developers PLEASE review their code with the coder module AND E_ALL enabled??! THANK YOU!!!!

Message: Undefined index: type in /var/www/drupal-hfopi/sites/all/modules/backup_migrate/includes/destinations.inc on line 85.
Severity: error
Message: Undefined index: location in /var/www/drupal-hfopi/sites/all/modules/backup_migrate/includes/destinations.inc on line 89.
Severity: error
Message: Undefined variable: op in /var/www/drupal-hfopi/sites/all/modules/backup_migrate/includes/destinations.inc on line 153.
Severity: error
Message: Undefined index: password in /var/www/drupal-hfopi/sites/all/modules/backup_migrate/includes/destinations.inc on line 636.
Severity: error
Message: Undefined variable: filename in /var/www/drupal-hfopi/sites/all/modules/backup_migrate/includes/destinations.file.inc on line 149.
Severity: error
Message: Undefined variable: contents in /var/www/drupal-hfopi/sites/all/modules/backup_migrate/includes/destinations.file.inc on line 149.
Severity: error
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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