upload de fichiers (JS + contrôleur + htaccess), collage de fichiers dans l'éditeur (JS), amélioration utilisation de Imagick (ImageUploadController)

7 files changed, 392 insertions, 150 deletions

diff --git a/public/js/tinymce.js b/public/js/tinymce.js
index a187189..7b59506 100644
--- a/public/js/tinymce.js
+++ b/public/js/tinymce.js
@@ -72,6 +72,9 @@ function deleteArticle(id){
 
 class Editor
 {
+    extensions_white_list = ['pdf', 'rtf', 'doc', 'docx', 'xls', 'xlsx', 'ppt', 'pptx', 'odt', 'ods', 'odp'];
+    // = $extensions_white_list côté PHP
+
     constructor(id){
         this.id = id;
         this.article = document.getElementById(this.id);
@@ -108,7 +111,7 @@ class Editor
         tinymce.init({
             selector: `[id="${this.id}"]`, // écrire [id="246"] au lieu de #246 parce que l'id commence par un chiffre
             language: 'fr_FR',
-            language_url: 'js/tinymce-langs/fr_FR.js', // téléchargement ici: https://cdn.jsdelivr.net/npm/tinymce-lang/langs/fr_FR.min.js
+            language_url: 'js/tinymce-langs/fr_FR.js',
             license_key: 'gpl',
             branding: false,
             plugins: 'lists link autolink table image media autoresize help',
@@ -116,6 +119,13 @@ class Editor
             menubar: false,
             toolbar_mode: 'wrap',
             statusbar: false,
+            link_title: false, // supprime le champ compliqué "titre" (apparaît au survol du lien) dans la fenêtre "link"
+            /*link_attributes_postprocess: (attrs) => { // modifier les attributs des liens créés
+                console.log(attrs);
+                if (attrs.rel) {
+                    attrs.rel += 'noreferrer'; // cacher la page d'où on vient
+                }
+            },*/
             // les fonctions fléchées permettent de garder le contexte (= this)
             setup: (editor) => {
                 editor.on('init', () => {
@@ -141,55 +151,61 @@ class Editor
                     }
                 });
                 let skipPastePreProcess = false;
-                editor.on('Paste', function (e){ // déclenchement AVANT PastePreProcess et quelque que soit le contenu collé
-                    const clipboardData = (e.clipboardData || e.originalEvent.clipboardData);
-                    if(!clipboardData){
+                editor.on('Paste', (e) => { // déclenchement AVANT PastePreProcess et quelque que soit le contenu collé
+                    if(!e.clipboardData){ // e.clipboardData: DataTransfer
                         return;
                     }
-                    const items = clipboardData.items;
-                    let foundImage = false;
-
-                    for(let i = 0; i < items.length; i++){
-                        let item = items[i];
-
-                        if(item.type.indexOf('image') !== -1){ // test type MIME contenant image
-                            foundImage = true;
-
-                            const file = item.getAsFile(); // presse-papier => fichier lisible
-                            const reader = new FileReader();
-
-                            reader.onload = function (event){ // fonction exécutée lorsque reader.readAsDataURL(file) est terminée
-                                const base64Data = event.target.result; // données de l'image
-
-                                fetch('index.php?action=upload_image_base64', {
-                                    method: 'POST',
-                                    headers: { 'Content-Type': 'application/json' },
-                                    body: JSON.stringify({ image_base64: base64Data })
-                                })
-                                .then(response => response.json())
-                                .then(data => {
-                                    if(data.location){
-                                        editor.insertContent('<img src="' + data.location + '">');
-                                    }
-                                })
-                                .catch(error => {
-                                    console.error('Erreur lors de l’upload de l’image base64 :', error);
-                                });
-                            };
-                            reader.readAsDataURL(file); // lecture asynchrone du fichier
+                    const items = e.clipboardData.items; // base64
+                    const files = e.clipboardData.files; // explorateur de fichiers
+                    let found_file = false;
+
+                    // données dans files
+                    if(files && files.length > 0){ // noter que files peut être vide, alors que items non
+                        for(let i = 0; i < files.length; i++){
+                            let file = files[i];
+                            
+                            if(this.extensions_white_list.includes(file.name.split('.').pop()?.toLowerCase() || '')){
+                                found_file = true;
+                                this.uploadDocument(file, editor);
+                            }
+                            else if(file.type.indexOf('image') !== -1){ 
+                                found_file = true;
+                                this.uploadImageBase64(file, editor);
+                            }
+                        }
+                    }
+                    // données dans items
+                    else{ // les images collées depuis l'explorateur sont aussi dans items, or elles sont déjà gérées plus haut
+                        for(let i = 0; i < items.length; i++){
+                            let item = items[i];
+
+                            if(item.type.indexOf('image') !== -1){ // test type MIME contenant image
+                                found_file = true;
+                                const file = item.getAsFile(); // presse-papier => fichier lisible
+                                if(file){
+                                    this.uploadImageBase64(file, editor);
+                                }
+                                else{
+                                    console.error('fichier invalide');
+                                }
+                            }
                         }
                     }
 
-                    if(foundImage){
+                    if(found_file){
                         e.preventDefault(); // supprime le collage automatiue
                         skipPastePreProcess = true; // désactiver le PastePreProcess pour ce collage
                     }
                 });
                 editor.on('PastePreProcess', function (e){ // déclenchement au collage AVANT insertion dans l'éditeur
+                    if(skipPastePreProcess){
+                        skipPastePreProcess = false; // réinitialiser pour la prochaine fois
+                        return; // ignorer ce traitement
+                    }
+
                     const parser = new DOMParser();
                     const doc = parser.parseFromString(e.content, 'text/html');
                     const images = doc.querySelectorAll('img');
-                    
                     let downloads_in_progress = [];
                     
                     images.forEach(img => {
@@ -221,7 +237,6 @@ class Editor
                     // une image web ou plus: différer l'insertion dans l'éditeur le temps que le serveur télécharge les images
                     if(downloads_in_progress.length > 0){
                         e.preventDefault();
-
                         Promise.all(downloads_in_progress).then(() => {
                             e.content = doc.body.innerHTML; // remplacement du HTML dans l'éditeur par la copie modifiée (doc)
                             editor.insertContent(e.content);
@@ -230,30 +245,58 @@ class Editor
                     else{
                         e.content = doc.body.innerHTML; // remplacement du HTML dans l'éditeur par la copie modifiée (doc)
                     }
-                }); // fin editor.on('PastePreProcess'...
-            },
-            // upload d'image natif de tinymce avec le bouton "Insérer une image"
-            images_upload_handler: (blobInfo, progress) => new Promise((resolve, reject) => {
-                const formData = new FormData();
-                formData.append("file", blobInfo.blob());
-
-                fetch("index.php?action=upload_image_tinymce", {
-                    method: "POST",
-                    body: formData
-                })
-                .then(response => response.json())
-                .then(data => {
-                    if(data.location) {
-                        resolve(data.location);
+                });
+                // glisser-déposer de fichiers (sauf images qui sont déjà gérées nativement)
+                editor.on('drop', (e) => {
+                    const data = e.dataTransfer;
+                    if(!data || !data.files || data.files.length === 0){
+                        return; // Laisser TinyMCE gérer (texte, images déjà supportées, etc.)
                     }
-                    else {
-                        reject("Erreur: Chemin d'image invalide");
+                    const files = data.files;
+
+                    let has_documents = false;
+                    for(let i = 0; i < files.length; i++){
+                        if(this.extensions_white_list.includes(files[i].name.split('.').pop()?.toLowerCase() || '')){
+                            has_documents = true;
+                            break;
+                        }
                     }
-                })
-                .catch(error => {
-                    reject("Erreur lors de l'upload");
+                    
+                    if(has_documents){
+                        e.preventDefault();
+                        e.stopPropagation();
+                        
+                        for(let i = 0; i < files.length; i++){
+                            let file = files[i];
+                            
+                            if(this.extensions_white_list.includes(file.name.split('.').pop()?.toLowerCase() || '')){
+                                this.uploadDocument(file, editor);
+                            }
+                            else if(file.type.indexOf('image') !== -1){
+                                this.uploadImageBase64(file, editor);
+                            }
+                        }
+                    }
+                    // autres cas: tinymce gère tout seul
                 });
-            }),
+            },
+            // upload d'image avec le bouton "Insérer une image"
+            images_upload_handler: this.images_upload_handler, // = fonction fléchée
+            // upload de documents avec le bouton "insérer un lien"
+            files_upload_handler: this.files_upload_handler, // = fonction fléchée
+            documents_file_types: [ // files_upload_handler a besoin qu'on lui donne tous les types mime
+                { mimeType: 'application/pdf', extensions: [ 'pdf' ] },
+                { mimeType: 'application/rtf', extensions: [ 'rtf' ] },
+                { mimeType: 'application/msword', extensions: [ 'doc' ] },
+                { mimeType: 'application/vnd.openxmlformats-officedocument.wordprocessingml.document', extensions: [ 'docx' ] },
+                { mimeType: 'application/vnd.ms-excel', extensions: [ 'xls' ] },
+                { mimeType: 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet', extensions: [ 'xlsx' ] },
+                { mimeType: 'application/vnd.ms-powerpoint', extensions: [ 'ppt' ] },
+                { mimeType: 'application/vnd.openxmlformats-officedocument.presentationml.presentation', extensions: [ 'pptx' ] },
+                { mimeType: 'application/vnd.oasis.opendocument.text', extensions: [ 'odt' ] },
+                { mimeType: 'application/vnd.oasis.opendocument.spreadsheet', extensions: [ 'ods' ] },
+                { mimeType: 'application/vnd.oasis.opendocument.presentation', extensions: [ 'odp' ] }
+            ],
             image_caption: true
         });
     }
@@ -385,9 +428,107 @@ class Editor
         delete editors[this.id];
         console.log(`Editor ${this.id} détruit.`);
     }*/
-}
 
+    images_upload_handler = (blobInfo, progress) => new Promise((resolve, reject) => {
+        const formData = new FormData();
+        formData.append("file", blobInfo.blob());
+
+        fetch('index.php?action=upload_image_tinymce', {
+            method: 'POST',
+            body: formData
+        })
+        .then(response => response.json())
+        .then(data => {
+            if(data.location){
+                resolve(data.location);
+            }
+            else{
+                reject("Erreur: Chemin d'image invalide");
+            }
+        })
+        .catch(error => {
+            reject("Erreur lors de l'upload");
+        });
+    });
+    files_upload_handler = (blobInfo, progress) => new Promise((resolve, reject) => { // utilisation = bouton "link" (OU drag & drop, et oui)
+        const formData = new FormData();
+        formData.append("file", blobInfo.blob());
+
+        fetch('index.php?action=upload_file_tinymce', {
+            method: 'POST',
+            body: formData
+        })
+        .then(response => response.json())
+        .then(data => {
+            if(data.location){
+                // resolve et reject fonctionne avec Promise => type de retour standardisé et évite l'utilistion de callbacks
+                resolve({
+                    url: data.location,
+                    fileName: blobInfo.filename(),
+                });
+            }
+            else{
+                reject("Erreur: Chemin du fichier invalide");
+            }
+        })
+        .catch(error => {
+            reject("Erreur lors de l'upload");
+        });
+    });
+
+    uploadImageBase64(file, editor){
+        const reader = new FileReader();
 
+        reader.onload = function (event){ // fonction exécutée lorsque reader.readAsDataURL(file) est terminée
+            const base64_target = event.target;
+            if(!base64_target || !base64_target.result){
+                console.error("erreur de lecture du fichier");
+                return;
+            }
+
+            fetch('index.php?action=upload_image_base64', {
+                method: 'POST',
+                headers: { 'Content-Type': 'application/json' },
+                body: JSON.stringify({ image_base64: base64_target.result })
+            })
+            .then(response => response.json())
+            .then(data => {
+                if(data.location){
+                    editor.insertContent('<img src="' + data.location + '">');
+                }
+            })
+            .catch(error => {
+                console.error('Erreur lors de l’upload de l’image base64 :', error);
+            });
+        };
+        reader.readAsDataURL(file); // lecture asynchrone du fichier
+    }
+    uploadDocument(file, editor){ // utilisation = copier-coller de l'explorateur de fichiers
+        const formData = new FormData();
+        formData.append("file", file);
+        
+        fetch('index.php?action=upload_file_tinymce', {
+            method: 'POST',
+            body: formData
+        })
+        .then(response => response.json())
+        .then(data => {
+            if(data.location){
+                // créer le lien <a>
+                const file_name = file.name;
+                const extension = file_name.split('.').pop()?.toLowerCase() || '';
+                const target = extension === 'pdf' ? 'target="_blank"' : ''; // PDF = page
+                editor.insertContent(`<a href="${data.location}" ${target} title="${file_name}">[${extension}] ${file_name}</a>`);
+            }
+            else {
+                console.error("Erreur: Chemin du fichier invalide");
+            }
+        })
+        .catch(error => {
+            console.error("Erreur lors de l'upload du document :", error);
+        });
+    }
+}
 
 
 
diff --git a/public/user_data/media/.htaccess b/public/user_data/media/.htaccess
new file mode 100644
index 0000000..4494297
--- /dev/null
+++ b/public/user_data/media/.htaccess
@@ -0,0 +1,10 @@
+# désactiver l'interpréteur PHP
+php_flag engine off
+
+# exploration des dossiers
+Options -Indexes
+
+# interdire l'accès directaux fichiers .htaccess et .htpasswd
+<FilesMatch "^\.(htaccess|htpasswd)$">
+    Require all denied
+</FilesMatch>
diff --git a/src/Captcha.php b/src/Captcha.php
index f60031a..a0c7a54 100644
--- a/src/Captcha.php
+++ b/src/Captcha.php
@@ -10,12 +10,10 @@ class Captcha
 {
         private int $a;
         private int $b;
-        private int $solution;
 
         public function __construct(){
                 $this->a = rand(2, 9);
                 $this->b = rand(2, 9);
-                $this->solution = $this->a * $this->b;
         }
 
         public function getA(): string
@@ -28,7 +26,7 @@ class Captcha
         }
         public function getSolution(): int
         {
-                return $this->solution;
+                return ($this->a * $this->b);
         }
 
         private function toLettersFrench(int $number): string
@@ -45,6 +43,8 @@ class Captcha
                         default => '', // erreur
                 };
         }
+        
+        // (à déplacer dans FormValidation?)
         static public function controlInput(string $input = '0'): int
         {
             // un POST est une chaîne qu'on doit convertir en nombre dans deux conditions:
diff --git a/src/controller/FileUploadController.php b/src/controller/FileUploadController.php
new file mode 100644
index 0000000..f53f5c2
--- /dev/null
+++ b/src/controller/FileUploadController.php
@@ -0,0 +1,67 @@
+<?php
+// src/controller/FileUploadController.php
+
+declare(strict_types=1);
+
+class FileUploadController
+{
+        static public function checkFileDownload(array $file): bool
+        {
+                $extensions_white_list = ['pdf', 'rtf', 'doc', 'docx', 'xls', 'xlsx', 'ppt', 'pptx', 'odt', 'ods', 'odp']; // = extensions_white_list côté javascript
+                $mime_type_white_list = ['application/pdf', 'application/rtf', 'text/rtf', 'application/msword', 'application/vnd.openxmlformats-officedocument.wordprocessingml.document', 'application/vnd.ms-excel', 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet', 'application/vnd.ms-powerpoint', 'application/vnd.openxmlformats-officedocument.presentationml.presentation', 'application/vnd.oasis.opendocument.text', 'application/vnd.oasis.opendocument.spreadsheet', 'application/vnd.oasis.opendocument.presentation'];
+                
+                // 1/ extension
+                $extension = strtolower(pathinfo($file['name'], PATHINFO_EXTENSION));
+                if(!in_array($extension, $extensions_white_list, true)){
+                        return false;
+            }
+                
+                // 2/ fichier obtenu par HTTP POST (théoriquement inutile si le routeur est solide, mais ça ne mange pas de pain)
+                if(!is_uploaded_file($file['tmp_name'])){
+                        return false;
+                }
+
+                // 3/ objet $finfo valide (dépend du paramètre FILEINFO_MIME_TYPE)
+            $finfo = new finfo(FILEINFO_MIME_TYPE);
+                if($finfo === false){
+                return false;
+            }
+                
+                // 4/ contrôle du "vrai" type mime (finfo_file lit les 1ers octets des fichiers pour y trouver des "signatures", très fiable sauf avec les conteneurs: doc, zip...)
+            $real_type = finfo_file($finfo, $file['tmp_name']);
+            return in_array($real_type, $mime_type_white_list, true);
+        }
+
+        static public function fileUploadTinyMce(): void
+        {
+                if(isset($_FILES['file'])){
+                $dest = 'user_data/media/';
+                if(!is_dir($dest)){ // Vérifier si le répertoire existe, sinon le créer
+                    mkdir($dest, 0755, true);
+                }
+                
+                $name = Security::secureFileName(pathinfo($_FILES['file']['name'], PATHINFO_FILENAME)); // retirer caractères spéciaux et changer espaces en underscores
+                $extension = strtolower(pathinfo($_FILES['file']['name'], PATHINFO_EXTENSION)); 
+                $file_path = $dest . $name . '_' . uniqid() . '.' . $extension; // nom unique
+                
+                if(self::checkFileDownload($_FILES['file'])){
+                        if(move_uploaded_file($_FILES['file']['tmp_name'], $file_path)){
+                                        echo json_encode(['location' => $file_path]);
+                                }
+                                else{
+                                        http_response_code(500);
+                                        echo json_encode(['message' => 'Erreur enregistrement du fichier.']);
+                                }
+                }
+                else{
+                        http_response_code(400);
+                                echo json_encode(['message' => 'Erreur 400: fichier non valide.']);
+                        }
+            }
+            else{
+                http_response_code(400);
+                echo json_encode(['message' => 'Erreur 400: Bad Request']);
+            }
+            die;
+        }
+}
\ No newline at end of file
diff --git a/src/controller/HeadFootController.php b/src/controller/HeadFootController.php
index de7ae53..83e27a8 100644
--- a/src/controller/HeadFootController.php
+++ b/src/controller/HeadFootController.php
@@ -4,7 +4,6 @@
 declare(strict_types=1);
 
 use App\Entity\NodeData;
-use App\Entity\NodeDataAsset;
 use App\Entity\Asset;
 use App\Entity\AssetEmployment;
 use Doctrine\ORM\EntityManager;
@@ -82,7 +81,7 @@ class HeadFootController
                         }
 
                         /* -- écriture du fichier sur le disque -- */
-                        if(!ImageUploadController::imagickCleanImage(file_get_contents($file['tmp_name']), Asset::USER_PATH . $name, $extension)){ // recréer l’image pour la nettoyer
+                        if(!ImageUploadController::imagickCleanAndWriteImage(file_get_contents($file['tmp_name']), Asset::USER_PATH . $name, $extension)){ // recréer l’image pour la nettoyer
                                 http_response_code(500);
                     echo json_encode(['success' => false, 'message' => 'Erreur image non valide.', 'format' => $extension]);
                         }
@@ -146,6 +145,4 @@ class HeadFootController
                 }
                 die;
         }
-
-        //static public function uploadImage(EntityManager $entityManager, array $request_params): void
 }
\ No newline at end of file
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{
diff --git a/src/router.php b/src/router.php
index 7348f2b..ff219da 100644
--- a/src/router.php
+++ b/src/router.php
@@ -231,12 +231,17 @@ elseif($request->getMethod() === 'POST'){
             }
         }
 
-        // upload d'image dans tinymce avec le plugin (bouton "insérer une image" de l'éditeur)
+        // upload avec FormData
         elseif(strpos($_SERVER['CONTENT_TYPE'], 'multipart/form-data') !== false)
         {
+            // dans tinymce avec le plugin (bouton "insérer une image" de l'éditeur ou glisser-déposer)
             if($request->query->has('action') && $request->query->get('action') === 'upload_image_tinymce'){
                 ImageUploadController::imageUploadTinyMce();
             }
+            // dans tinymce, des quatre méthodes: bouton "link", drag & drop, html, base64
+            elseif($request->query->has('action') && $request->query->get('action') === 'upload_file_tinymce'){
+                FileUploadController::fileUploadTinyMce();
+            }
             elseif($request->query->has('head_foot_image')){
                 HeadFootController::uploadAsset($entityManager, $request->query->get('head_foot_image'));
             }