Skip to content
Snippets Groups Projects
Verified Commit 9d88fbd5 authored by Florian Azizian's avatar Florian Azizian
Browse files

FIX #11875 TIME 0:05 rm control size + change return in scan context

parent 3183c68f
No related branches found
No related tags found
No related merge requests found
...@@ -164,11 +164,11 @@ class ConvertPdfController ...@@ -164,11 +164,11 @@ class ConvertPdfController
$resource = file_get_contents("{$tmpPath}{$tmpFilename}.pdf"); $resource = file_get_contents("{$tmpPath}{$tmpFilename}.pdf");
$aReturn = []; $aReturn = [];
$aReturn["encodedResource"] = base64_encode($resource);
if ($aArgs['context'] == 'scan') { if ($aArgs['context'] == 'scan') {
$aReturn["tmpFilename"] = $tmpFilename.'.pdf'; $aReturn["tmpFilename"] = $tmpFilename.'.pdf';
} else { } else {
$aReturn["encodedResource"] = base64_encode($resource);
unlink("{$tmpPath}{$tmpFilename}.pdf"); unlink("{$tmpPath}{$tmpFilename}.pdf");
} }
return $aReturn; return $aReturn;
...@@ -245,24 +245,22 @@ class ConvertPdfController ...@@ -245,24 +245,22 @@ class ConvertPdfController
$size = strlen($file); $size = strlen($file);
if (strtolower($ext) == 'pdf' && strtolower($mimeType) == 'application/pdf') { if (strtolower($ext) == 'pdf' && strtolower($mimeType) == 'application/pdf') {
$return['encodedResource'] = $body['base64'];
if ($body['context'] == 'scan') { if ($body['context'] == 'scan') {
$tmpPath = CoreConfigModel::getTmpPath(); $tmpPath = CoreConfigModel::getTmpPath();
$tmpFilename = 'scan_converting' . rand() . '.pdf'; $tmpFilename = 'scan_converting' . rand() . '.pdf';
file_put_contents($tmpPath . $tmpFilename, $file); file_put_contents($tmpPath . $tmpFilename, $file);
$return['tmpFilename'] = $tmpFilename; $return['tmpFilename'] = $tmpFilename;
} else {
$return['encodedResource'] = $body['base64'];
} }
return $response->withJson($return); return $response->withJson($return);
} else { } else {
$fileAccepted = StoreController::isFileAllowed(['extension' => $ext, 'type' => $mimeType]); $fileAccepted = StoreController::isFileAllowed(['extension' => $ext, 'type' => $mimeType]);
$maxFilesizeMo = ini_get('upload_max_filesize');
$canConvert = ConvertPdfController::canConvert(['extension' => $ext]); $canConvert = ConvertPdfController::canConvert(['extension' => $ext]);
if (!$fileAccepted) { if (!$fileAccepted) {
return $response->withStatus(400)->withJson(['errors' => 'File type not allowed. Extension : ' . $ext . '. Mime Type : ' . $mimeType . '.']); return $response->withStatus(400)->withJson(['errors' => 'File type not allowed. Extension : ' . $ext . '. Mime Type : ' . $mimeType . '.']);
} elseif ($size/1024 > $maxFilesizeMo*1024) {
return $response->withStatus(400)->withJson(['errors' => 'File maximum size is exceeded ('.$maxFilesizeMo.' Mo)']);
} elseif (!$canConvert) { } elseif (!$canConvert) {
return $response->withStatus(400)->withJson(['errors' => 'File accepted but can not be converted in pdf']); return $response->withStatus(400)->withJson(['errors' => 'File accepted but can not be converted in pdf']);
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment