filespec::move_file() from functions_upload.php does not error correctly

If $overwrite is false and the the destination file exists, no error message is given, file_moved is set to true and the return value is true.
All 3 of these results are wrong.
It should add an error message to ->error, file_moved should not be set to true and return should be false.