catch exceptions while uploading and pass on the error message
This commit is contained in:
parent
730c80ff9c
commit
7c6ed6ab33
|
@ -110,30 +110,35 @@ if (strpos($dir, '..') === false) {
|
||||||
|| (isset($_POST['resolution']) && $_POST['resolution']==='replace')
|
|| (isset($_POST['resolution']) && $_POST['resolution']==='replace')
|
||||||
) {
|
) {
|
||||||
// upload and overwrite file
|
// upload and overwrite file
|
||||||
if (is_uploaded_file($files['tmp_name'][$i]) and \OC\Files\Filesystem::fromTmpFile($files['tmp_name'][$i], $target)) {
|
try
|
||||||
|
{
|
||||||
|
if (is_uploaded_file($files['tmp_name'][$i]) and \OC\Files\Filesystem::fromTmpFile($files['tmp_name'][$i], $target)) {
|
||||||
|
|
||||||
// updated max file size after upload
|
// updated max file size after upload
|
||||||
$storageStats = \OCA\Files\Helper::buildFileStorageStatistics($dir);
|
$storageStats = \OCA\Files\Helper::buildFileStorageStatistics($dir);
|
||||||
|
|
||||||
|
$meta = \OC\Files\Filesystem::getFileInfo($target);
|
||||||
|
if ($meta === false) {
|
||||||
|
$error = $l->t('Upload failed. Could not get file info.');
|
||||||
|
} else {
|
||||||
|
$result[] = array('status' => 'success',
|
||||||
|
'mime' => $meta['mimetype'],
|
||||||
|
'mtime' => $meta['mtime'],
|
||||||
|
'size' => $meta['size'],
|
||||||
|
'id' => $meta['fileid'],
|
||||||
|
'name' => basename($target),
|
||||||
|
'originalname' => $files['tmp_name'][$i],
|
||||||
|
'uploadMaxFilesize' => $maxUploadFileSize,
|
||||||
|
'maxHumanFilesize' => $maxHumanFileSize,
|
||||||
|
'permissions' => $meta['permissions'],
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
$meta = \OC\Files\Filesystem::getFileInfo($target);
|
|
||||||
if ($meta === false) {
|
|
||||||
$error = $l->t('Upload failed. Could not get file info.');
|
|
||||||
} else {
|
} else {
|
||||||
$result[] = array('status' => 'success',
|
$error = $l->t('Upload failed. Could not find uploaded file');
|
||||||
'mime' => $meta['mimetype'],
|
|
||||||
'mtime' => $meta['mtime'],
|
|
||||||
'size' => $meta['size'],
|
|
||||||
'id' => $meta['fileid'],
|
|
||||||
'name' => basename($target),
|
|
||||||
'originalname' => $files['tmp_name'][$i],
|
|
||||||
'uploadMaxFilesize' => $maxUploadFileSize,
|
|
||||||
'maxHumanFilesize' => $maxHumanFileSize,
|
|
||||||
'permissions' => $meta['permissions'],
|
|
||||||
);
|
|
||||||
}
|
}
|
||||||
|
} catch(Exception $ex) {
|
||||||
} else {
|
$error = $ex->getMessage();
|
||||||
$error = $l->t('Upload failed. Could not find uploaded file');
|
|
||||||
}
|
}
|
||||||
|
|
||||||
} else {
|
} else {
|
||||||
|
@ -164,5 +169,5 @@ if ($error === false) {
|
||||||
OCP\JSON::encodedPrint($result);
|
OCP\JSON::encodedPrint($result);
|
||||||
exit();
|
exit();
|
||||||
} else {
|
} else {
|
||||||
OCP\JSON::error(array('data' => array_merge(array('message' => $error), $storageStats)));
|
OCP\JSON::error(array(array('data' => array_merge(array('message' => $error), $storageStats))));
|
||||||
}
|
}
|
||||||
|
|
|
@ -365,7 +365,7 @@ $(document).ready(function() {
|
||||||
} else if (result[0].status !== 'success') {
|
} else if (result[0].status !== 'success') {
|
||||||
//delete data.jqXHR;
|
//delete data.jqXHR;
|
||||||
data.textStatus = 'servererror';
|
data.textStatus = 'servererror';
|
||||||
data.errorThrown = result.data.message; // error message has been translated on server
|
data.errorThrown = result[0].data.message; // error message has been translated on server
|
||||||
var fu = $(this).data('blueimp-fileupload') || $(this).data('fileupload');
|
var fu = $(this).data('blueimp-fileupload') || $(this).data('fileupload');
|
||||||
fu._trigger('fail', e, data);
|
fu._trigger('fail', e, data);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue