From eaf46444427944bf170735d1500c8bd0a126b9f4 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Joshua=20R=C3=BCsweg?= Date: Thu, 31 Jan 2019 15:23:29 +0100 Subject: [PATCH] Simplify code See #2825 --- .../install/files/lib/action/AJAXFileUploadAction.class.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/wcfsetup/install/files/lib/action/AJAXFileUploadAction.class.php b/wcfsetup/install/files/lib/action/AJAXFileUploadAction.class.php index 44f5e7080c..98a4070d02 100644 --- a/wcfsetup/install/files/lib/action/AJAXFileUploadAction.class.php +++ b/wcfsetup/install/files/lib/action/AJAXFileUploadAction.class.php @@ -89,10 +89,10 @@ class AJAXFileUploadAction extends AbstractSecureAction { foreach ($_FILES['__files']['tmp_name'] as $id => $tmpName) { if ($field->isImageOnly()) { if (@getimagesize($tmpName) === false) { - if (!($field->svgImagesAllowed() && in_array(FileUtil::getMimeType($tmpName), [ + if (!$field->svgImagesAllowed() || !in_array(FileUtil::getMimeType($tmpName), [ 'image/svg', 'image/svg+xml' - ]))) { + ])) { $response['error'][$i++] = [ 'filename' => $_FILES['__files']['name'][$id], 'errorMessage' => WCF::getLanguage()->get('wcf.upload.error.noImage') -- 2.20.1