|
hmm, maybe it might be also better to implement it as a file_storage method
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...
|
||||||||||||||||||||||||||||||||||||||||||||||||||||
Sub-task
Open
Minor
 |
|
All content on this web site is made available under the GNU General Public License, unless otherwise stated.
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)