|
Dongsheng Cai committed 3 files to 'Moodle CVS' - 17/Jul/08 05:34 PM
Mathieu Petit-Clair made changes - 08/Aug/08 10:22 AM
Mathieu Petit-Clair committed 2 files to 'Moodle CVS' - 27/Aug/08 03:57 PM
Mathieu Petit-Clair committed 1 file to 'Moodle CVS' - 27/Aug/08 04:36 PM
Mathieu Petit-Clair committed 1 file to 'Moodle CVS' - 29/Aug/08 03:51 PM
Mathieu Petit-Clair committed 1 file to 'Moodle CVS' - 01/Sep/08 02:15 PM
Mathieu Petit-Clair committed 1 file to 'Moodle CVS' - 01/Sep/08 06:33 PM
hmm, maybe it might be also better to implement it as a file_storage method
moodler committed 1 file to 'Moodle CVS' - 02/Sep/08 11:00 AM
Mathieu Petit-Clair committed 2 files to 'Moodle CVS' - 02/Sep/08 03:30 PM
why are the $itemid and $filepath optional? the uploaded file does not have any reasonable values there, right?
Well, there are, but no they're not right. Changing this...
Mathieu Petit-Clair committed 1 file to 'Moodle CVS' - 02/Sep/08 04:06 PM
moodler committed 1 file to 'Moodle CVS' - 03/Sep/08 02:07 PM
Mathieu Petit-Clair committed 2 files to 'Moodle CVS' - 04/Sep/08 09:56 AM
Mathieu Petit-Clair committed 1 file to 'Moodle CVS' - 04/Sep/08 10:07 AM
Mathieu Petit-Clair committed 1 file to 'Moodle CVS' - 05/Sep/08 05:23 PM
Mathieu Petit-Clair committed 2 files to 'Moodle CVS' - 05/Sep/08 07:02 PM
Mathieu Petit-Clair committed 1 file to 'Moodle CVS' - 08/Sep/08 02:09 PM
Mathieu Petit-Clair committed 1 file to 'Moodle CVS' - 09/Sep/08 05:08 PM
Eloy Lafuente (stronk7) made changes - 30/Mar/09 04:44 PM
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
I think it would be better to have
function move_file_to_final_destination($draftfileid, $context, $filearea, $itemid, $filename, $filepath)
file_browser should not be used at all, the file must always come from draft area of current $USER and it might be better to verify it (we do not want to let users steal files from elsewhere)