|
now in 1.9stable and HEAD - not sure about 1.8stable though.... that function is new in 1.9 and I can't seem to find the string mentioned in the forum posting anywhere in 1.8 .... is this an issue in 1.8stable?
Dan Hi Dan,
if I'm not wrong... under 1.8 that is executed by backup_execute.html (in 1.9 it was repackaged to the backup_execute() function you've changed). Also... backup_scheduled.php and restorelib.php seem to be using that function with similar behaviour: stop the process. Ciao thanks for the details about 1.8 - will look at patching that too sometime soon (I thought those other files did something similar too, but was playing with cvs a bit and wanted to test some small commits!)
...looking at the way those errors are presented - the silent backup errors don't go anywhere! - I'm keen to insert them into the backup_logs table, and can't see much in scheduled backups that relys on the data in backup_logs except for this call to see if it hasn't completed - which we might be able to alter slightly.... $numofrec = count_records_select ("backup_log","time > $timeafter") In fact I'm thinking that we could add a new field to backup_log that defines what process/function wrote to the log eg (manual backup, Scheduled backup, Incremental backup, Silient backup - or individual module backup scripts?) does that sound ok?
Dan Ah, yes! I read this yesterday and forgot to comment.
Sounds ok for me (HEAD only) + potential mini add_to_backup_log() function to control all them together. Ciao yes - I was thinking Head only - (can't make db changes to Stable branches)
Dan attaching patches for restorelib and backup_scheduled.php - we really need somewhere to store all those $errorstr when calling restore_silently, backup_silently!
Dan Adding 2.0 as fixfor version (partially) and closing... thanks Dan!
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Bug
Closed
Minor
issue deleting temp dirs in backup
 |
|
that shouldn't be an stop-error. debugging sounds ok here... although perhaps I also will notify() the user about that. Note that the BACKUP_SILENTLY part (some lines below...) is also returning false and perhaps should allow to continue. And same could be happening in scheduled_backups and restore (both use that function).
Can you handle this? TIA!