1 files changed, 104 insertions, 82 deletions

diff --git a/src/controller/ImageUploadController.php b/src/controller/ImageUploadController.php
index 77f0a47..7943116 100644
--- a/src/controller/ImageUploadController.php
+++ b/src/controller/ImageUploadController.php
@@ -5,17 +5,40 @@ declare(strict_types=1);
 
 class ImageUploadController
 {
-        static public function imagickCleanImage(string $image_data, string $local_path, string $format = 'jpeg'): bool // "string" parce que file_get_contents...
+        const ALLOWED_EXTENSIONS = ['jpg', 'jpeg', 'png', 'gif', 'webp', 'tiff', 'tif'];
+
+        static public function imagickCleanAndWriteImage(string $image_data, string $local_path): bool // "string" parce que file_get_contents...
         {
+                $format = strtolower(pathinfo($local_path)['extension']);
             try{
                 $imagick = new Imagick();
                 $imagick->readImageBlob($image_data);
                 $imagick->stripImage(); // nettoyage métadonnées
-                $imagick->setImageFormat($format);
-                if($format === 'jpeg'){
-                    $imagick->setImageCompression(Imagick::COMPRESSION_JPEG);
-                    $imagick->setImageCompressionQuality(85); // optionnel
+                //$imagick->setImageFormat($format); // inutile, writeImage force la conversion
+
+                // compression
+                switch($format){
+                    case 'jpeg': // particularité du switch, si 'jpeg' le test de 'jpg' est ignoré et on va jusqu'au break
+                    case 'jpg':
+                        $imagick->setImageCompression(Imagick::COMPRESSION_JPEG);
+                        $imagick->setImageCompressionQuality(85);
+                        break;
+                    case 'webp':
+                        $imagick->setImageCompression(Imagick::COMPRESSION_WEBP);
+                        $imagick->setImageCompressionQuality(85);
+                        break;
+                    case 'png':
+                        $imagick->setImageCompression(Imagick::COMPRESSION_ZIP);
+                        $imagick->setImageCompressionQuality(7); // 9 est sans perte
+                        break;
+                    case 'tiff':
+                        $imagick->setImageCompression(Imagick::COMPRESSION_LZW);  // LZW est sans perte
+                        break;
                 }
+
+                // enregistrement
+                // writeImage utilise l'extension du fichier et ignore le format détecté
+                // imagemagick est à l'origine une appli console, elle considère que l'extension montre l'intention de 
                 $imagick->writeImage($local_path); // enregistrement
                 $imagick->clear();
                 $imagick->destroy();
@@ -25,7 +48,7 @@ class ImageUploadController
                 return false;
             }
         }
-        static public function curlDownloadImage(string $url, $maxRetries = 3, $timeout = 10): string|false
+        static public function curlDownloadImage(string $url, int $maxRetries = 3, int $timeout = 10): string|false
         {
             $attempt = 0;
             $imageData = false;
@@ -43,8 +66,6 @@ class ImageUploadController
                 $httpCode = curl_getinfo($ch, CURLINFO_HTTP_CODE);
                 //$curlError = curl_error($ch);
 
-                curl_close($ch);
-
                 if($imageData !== false && $httpCode >= 200 && $httpCode < 300){
                     return $imageData;
                 }
@@ -59,42 +80,45 @@ class ImageUploadController
         // téléchargement par le plugin (bouton "insérer une image")
         static public function imageUploadTinyMce(): void
         {
-                if(isset($_FILES['file'])){
-                $file = $_FILES['file'];
-                $dest = 'user_data/images/';
-                $dest_mini = 'user_data/images-mini/';
-                
-                // Vérifier si les répertoires existent, sinon les créer
-                if(!is_dir($dest)){
-                    mkdir($dest, 0777, true);
-                }
-                if(!is_dir($dest_mini)){
-                    mkdir($dest_mini, 0777, true);
-                }
-                
-                $allowed_extensions = ['jpg', 'jpeg', 'png', 'gif', 'webp', 'tiff', 'tif'];
-                $name = Security::secureFileName(pathinfo($file['name'], PATHINFO_FILENAME));
-                $extension = strtolower(pathinfo($file['name'], PATHINFO_EXTENSION));
-                if(!in_array($extension, $allowed_extensions) || $extension === 'jpg'){
-                    $extension = 'jpeg';
-                }
-                $file_path = uniqid($dest . $name . '_') . '.' . $extension;
-
-                // créer une miniature de l'image
-                //
-
-                if(self::imagickCleanImage(file_get_contents($file['tmp_name']), $file_path, $extension)){ // recréer l’image pour la nettoyer
-                    echo json_encode(['location' => $file_path]); // renvoyer l'URL de l'image téléchargée
-                }
-                else{
-                    http_response_code(500);
-                    echo json_encode(['message' => 'Erreur image non valide', 'format' => $extension]);
-                }
-            }
-            else{
-                http_response_code(400);
+                if(!isset($_FILES['file'])){
+                        http_response_code(400);
                 echo json_encode(['message' => 'Erreur 400: Bad Request']);
             }
+            if(!is_uploaded_file($_FILES['file']['tmp_name'])) {
+            http_response_code(400);
+            echo json_encode(['message' => "Le fichier n'a pas été téléchargé correctement."]);
+            die;
+        }
+
+        $dest = 'user_data/images/';
+        $dest_mini = 'user_data/images-mini/';
+        
+        // Vérifier si les répertoires existent, sinon les créer
+        if(!is_dir($dest)){
+            mkdir($dest, 0755, true);
+        }
+        if(!is_dir($dest_mini)){
+            mkdir($dest_mini, 0755, true);
+        }
+        
+        $name = Security::secureFileName(pathinfo($_FILES['file']['name'], PATHINFO_FILENAME));
+        $extension = strtolower(pathinfo($_FILES['file']['name'], PATHINFO_EXTENSION));
+        $image_data = file_get_contents($_FILES['file']['tmp_name']);
+        if(!in_array($extension, self::ALLOWED_EXTENSIONS)){
+            $extension = 'jpeg';
+        }
+        $local_path = uniqid($dest . $name . '_') . '.' . $extension;
+
+        // créer une miniature de l'image
+        //
+
+        if(self::imagickCleanAndWriteImage($image_data, $local_path)){ // recréer l’image pour la nettoyer
+            echo json_encode(['location' => $local_path]); // renvoyer l'URL de l'image téléchargée
+        }
+        else{
+            http_response_code(500);
+            echo json_encode(['message' => 'Erreur image non valide']);
+        }
             die;
         }
 
@@ -103,40 +127,39 @@ class ImageUploadController
         {
                 $json = json_decode(file_get_contents('php://input'), true);
             
-            if(isset($json['image_url'])){
-                $image_data = self::curlDownloadImage($json['image_url']); // téléchargement de l’image par le serveur avec cURL au lieu de file_get_contents
-                $dest = 'user_data/images/';
-                
-                if(!is_dir($dest)) { // Vérifier si le répertoire existe, sinon le créer
-                    mkdir($dest, 0777, true);
-                }
-
-                if($image_data === false){
-                    http_response_code(400);
-                    echo json_encode(['message' => "Erreur, le serveur n'a pas réussi à télécharger l'image."]);
-                    die;
-                }
-                
-                $allowed_extensions = ['jpg', 'jpeg', 'png', 'gif', 'webp', 'tiff', 'tif'];
-                $url_path = parse_url($json['image_url'], PHP_URL_PATH);
-                $name = Security::secureFileName(pathinfo($url_path, PATHINFO_FILENAME));
-                $extension = strtolower(pathinfo($url_path, PATHINFO_EXTENSION));
-                if(!in_array($extension, $allowed_extensions) || $extension === 'jpg'){
-                    $extension = 'jpeg';
-                }
-                $local_path = uniqid($dest . $name . '_') . '.' . $extension;
-                
-                if(self::imagickCleanImage($image_data, $local_path, $extension)){ // recréer l’image pour la nettoyer
-                    echo json_encode(['location' => $local_path]); // nouvelle adresse
-                }
-                else{
-                    http_response_code(500);
-                    echo json_encode(['message' => 'Erreur image non valide', 'format' => $extension]);
-                }
-            }
-            else{
-                echo json_encode(['message' => 'Erreur 400: Bad Request']);
+            if(!isset($json['image_url'])){
+                http_response_code(400);
+            echo json_encode(['message' => "Erreur 400: Bad Request"]);
+            die;
             }
+
+        $image_data = self::curlDownloadImage($json['image_url']); // téléchargement de l’image par le serveur avec cURL au lieu de file_get_contents
+        if(!$image_data){
+            http_response_code(400);
+            echo json_encode(['message' => "Erreur, le serveur n'a pas réussi à télécharger l'image."]);
+            die;
+        }
+
+        $dest = 'user_data/images/';
+        if(!is_dir($dest)) { // Vérifier si le répertoire existe, sinon le créer
+            mkdir($dest, 0755, true);
+        }
+        
+        $url_path = parse_url($json['image_url'], PHP_URL_PATH);
+        $name = Security::secureFileName(pathinfo($url_path, PATHINFO_FILENAME));
+        $extension = strtolower(pathinfo($url_path, PATHINFO_EXTENSION));
+        if(!in_array($extension, self::ALLOWED_EXTENSIONS) || $extension === 'jpg'){
+            $extension = 'jpeg';
+        }
+        $local_path = uniqid($dest . $name . '_') . '.' . $extension;
+        
+        if(self::imagickCleanAndWriteImage($image_data, $local_path)){ // recréer l’image pour la nettoyer
+            echo json_encode(['location' => $local_path]); // nouvelle adresse
+        }
+        else{
+            http_response_code(500);
+            echo json_encode(['message' => 'Erreur image non valide', 'format' => $extension]);
+        }
             die;
         }
 
@@ -147,7 +170,7 @@ class ImageUploadController
             $dest = 'user_data/images/';
 
             if(!is_dir($dest)){
-                mkdir($dest, 0777, true);
+                mkdir($dest, 0755, true);
             }
 
             // détection de data:image/ et de ;base64, et capture du format dans $type
@@ -157,22 +180,21 @@ class ImageUploadController
                 die;
             }
 
-            $allowed_extensions = ['jpg', 'jpeg', 'png', 'gif', 'webp', 'tiff', 'tif'];
-            $extension = strtolower($type[1]);
-            if(!in_array($extension, $allowed_extensions) || $extension === 'jpg'){
+            $extension = strtolower($type[1]); // dans (\w+)
+            if(!in_array($extension, self::ALLOWED_EXTENSIONS)){
                 $extension = 'jpeg';
             }
 
+            $name = 'pasted_image'; 
             $image_data = base64_decode(substr($json['image_base64'], strpos($json['image_base64'], ',') + 1)); // découpe la chaine à la virgule puis convertit en binaire
             if($image_data === false){
                 http_response_code(400);
                 echo json_encode(['message' => 'Décodage base64 invalide']);
                 die;
             }
-            
-            $local_path = uniqid($dest . 'pasted_image_') . '.' . $extension;
+            $local_path = uniqid($dest . $name . '_') . '.' . $extension;
 
-            if(self::imagickCleanImage($image_data, $local_path)){
+            if(self::imagickCleanAndWriteImage($image_data, $local_path)){
                 echo json_encode(['location' => $local_path]);
             }
             else{