friendica/friendica-addons
7
2
Fork 13
Code Issues 3 Pull requests 2 Releases 16 Wiki Activity

[js_upload] Fixing missing extension index #827

Merged
nupplaphil merged 3 commits from issue/6338-php_notices into 2019.03-RC 2019-03-14 03:50:58 +01:00
Conversation 0 Commits 3 Files changed 1 +126 -122
1 changed files with 126 additions and 122 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
js_upload/js_upload.php
View file

@ -14,6 +14,7 @@
* Module Author: Chris Case
*
*/
use Friendica\Core\Config;
use Friendica\Core\Hook;
use Friendica\Core\L10n;
@ -217,7 +218,7 @@ class qqUploadedFileXhr {
fclose($input);
fclose($temp);
if ($realSize != $this->getSize()){
if ($realSize != $this->getSize()) {
return false;
}
return true;
@ -307,7 +308,7 @@ class qqFileUploader {
*/
function handleUpload(){
if (!$this->file){
if (!$this->file) {
return ['error' => L10n::t('No files were uploaded.')];
}
@ -333,7 +334,10 @@ class qqFileUploader {
$pathinfo = pathinfo($this->file->getName());
$filename = $pathinfo['filename'];
$ext = $pathinfo['extension'];
if (!isset($pathinfo['extension'])) {
Logger::warning('extension isn\'t set.', ['filename' => $filename]);
}
$ext = defaults($pathinfo, 'extension', '');
if($this->allowedExtensions && !in_array(strtolower($ext), $this->allowedExtensions)){
$these = implode(', ', $this->allowedExtensions);
@ -342,13 +346,13 @@ class qqFileUploader {
if ($this->file->save()){
return [
'success'=>true,
'success' => true,
'path' => $this->file->getPath(),
'filename' => $filename . '.' . $ext
];
} else {
return [
'error'=> L10n::t('Upload was cancelled, or server error encountered'),
'error' => L10n::t('Upload was cancelled, or server error encountered'),
'path' => $this->file->getPath(),
'filename' => $filename . '.' . $ext
];