From (penny at catalyst.net.nz) Thursday, 16 June 2005, 06:54 AM:
we could change it to only log if clam is enabled...
I'm also wondering what the best way to differentiate between copying temp files around during backup process and actually restoring uploaded files into a new course. The latter is really only where we want the log happening.
From Eloy Lafuente (stronk7 at moodle.org) Thursday, 16 June 2005, 08:11 AM:
uhm. To differentiate backup and restore you'll need to pass an extra argument to the copy function. Perhaps one clam_log=false at the end could be a good solution (you will be able to decide when to set it to true outside, isn't it...
Talking about the clam_log_upload(), a clever spanish moodler, Jose Luis D?az, has detected that exists a documented php bug in the html_entity_decode($var) (version 4.3.1) producing segmentation faults if $var is empty. He has changed line 2712 from:
$url = html_entity_decode($url);
if ($url) $url = html_entity_decode($url);
and his problems have disappeared. Perhaps we should add this check too...
From (penny at catalyst.net.nz) Thursday, 16 June 2005, 08:29 AM:
new option in the backup_copy_file function definition sounds good. I'll do it today.
as far as html_entity_decode - I'm found it in datalib in the add_to_log function - is that where you mean? in any case, yup, sounds like a good idea
From (penny at catalyst.net.nz) Thursday, 16 June 2005, 11:00 AM:
ok, fixes to both these problems in head & 1.5
From Eloy Lafuente (stronk7 at moodle.org) Friday, 17 June 2005, 12:02 AM:
So, this can be closed isn't it?