I always get a warning when i delete backups. I took a look inside the code and found out that
in the function backup_db_history_delete is a typo.
current code
/**
* Delete history entry.
*/
function backup_db_history_delete($id) {
// Remove backup_db entry.
// @todo, use injected db service.
db_delete('backup_db')
->condition('fid', $id)
->execute();
// Remove file and managed entry.
return file_delete($fid);
}
should be
/**
* Delete history entry.
*/
function backup_db_history_delete($id) {
// Remove backup_db entry.
// @todo, use injected db service.
db_delete('backup_db')
->condition('fid', $id)
->execute();
// Remove file and managed entry.
return file_delete($id);
}
I created a patch to fix this
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 2830711-typo-in-delete-backup-history.patch | 389 bytes | Anonymous (not verified) |
Comments
Comment #1
Anonymous (not verified) commentedAlmare created an issue. See original summary.
Comment #2
Anonymous (not verified) commentedComment #4
swim commentedThank you!