optimisation UserController::existUsers, gestion des $response par le routeur, erreur article déjà supprimé

8 files changed, 114 insertions, 86 deletions

diff --git a/public/index.php b/public/index.php
index 89bb203..6149f7a 100644
--- a/public/index.php
+++ b/public/index.php
@@ -1,15 +1,20 @@
 <?php
 // public/index/php
 
-/* plan d'action pour symfonyfier le site
-1/ intégrer les classes Request et Response sans changer modifier les liens
-2/ méthodes HTTP: GET, HEAD, POST, PUT, PATCH, DELETE, etc, pour un pré-routage (légères modifications des liens)
-3/ passer à des chemins modernes "ciblant des ressources" genre /chemin/de/la/page
-    le mode modification de page doit thérioquement être appelé comme ça: /chemin/de/la/page/modif_page
-    apparemment, le from=nom_page pour les formulaires ne se fait pas...
-4/ utiliser le routeur de symfony: nécéssite que tous les contrôleurs soient des classes avec un namespace
-5/ http-foundation possède aussi une classe Session. intéressant!
-*/
+/* plan d'action pour "symfonyfier" le site
+A - 
+    1/ de vrais contrôleurs: classes et méthodes prenant une requête en entrée et retournant une réponse
+    (début de séparation contrôleurs et classes métier, exemple: ViewController/Director)
+    2/ routeur structuré: méthodes GET et POST, content-type, admin
+    3/ routeur amélioré: pré-routage avec méthodes HTTP: GET, HEAD, POST, PUT, PATCH, DELETE, etc
+    4/ réécriture avec les classes Request et Response sans toucher les liens
+B - 
+    1/ passer à des chemins modernes "ciblant des ressources" genre /chemin/de/la/page
+        le mode modification de page doit thérioquement être appelé comme ça: /chemin/de/la/page/modif_page
+        apparemment, le from=nom_page pour les formulaires ne se fait pas...
+    2/ utiliser le routeur de symfony: nécéssite que tous les contrôleurs soient des classes avec un namespace */
+
+// http-foundation possède aussi une classe Session. intéressant!
 
 declare(strict_types=1);
 
diff --git a/public/js/tinymce.js b/public/js/tinymce.js
index 29b82bf..f153246 100644
--- a/public/js/tinymce.js
+++ b/public/js/tinymce.js
@@ -145,7 +145,7 @@ function openEditor(id, page = '') {
             const formData = new FormData();
             formData.append("file", blobInfo.blob());
 
-            fetch("index.php?action=upload_image", {
+            fetch("index.php?action=upload_image_tinymce", {
                 method: "POST",
                 body: formData
             })
diff --git a/src/controller/ArticleController.php b/src/controller/ArticleController.php
index a982d8c..71bfdc3 100644
--- a/src/controller/ArticleController.php
+++ b/src/controller/ArticleController.php
@@ -101,7 +101,11 @@ class ArticleController
         static public function deleteArticle(EntityManager $entityManager, array $json): Response
         {
                 $director = new Director($entityManager);
-                $director->makeArticleNode($json['id'], true);
+                if(!$director->makeArticleNode($json['id'], true)){
+                        return new Response(
+                        '{"success": false, "message": "Erreur: pas d\'article à supprimer"}',
+                Response::HTTP_INTERNAL_SERVER_ERROR); // 500
+                }
             $article = $director->getArticleNode();
                 $section = $director->getNode();
 
diff --git a/src/controller/CalendarController.php b/src/controller/CalendarController.php
index cc37d0f..b60ff11 100644
--- a/src/controller/CalendarController.php
+++ b/src/controller/CalendarController.php
@@ -52,6 +52,7 @@ class CalendarController
             $entityManager->flush();
                 
                 echo json_encode(['success' => true, 'id' => $event->getId()]);
+                die;
         }
         static public function updateEvent(array $json, EntityManager $entityManager):void
         {
@@ -67,6 +68,7 @@ class CalendarController
             $entityManager->flush();
 
                 echo json_encode(['success' => true]);
+                die;
         }
         static public function removeEvent(array $json, EntityManager $entityManager):void
         {
@@ -75,5 +77,6 @@ class CalendarController
             $entityManager->flush();
 
                 echo json_encode(['success' => true]);
+                die;
         }
 }
\ No newline at end of file
diff --git a/src/controller/UserController.php b/src/controller/UserController.php
index 1acb4d5..a35b09e 100644
--- a/src/controller/UserController.php
+++ b/src/controller/UserController.php
@@ -22,14 +22,14 @@ class UserController
         // account
         static public function existUsers(EntityManager $entityManager): bool
         {
-            // optimiser ça si possible, on veut juste savoir si la table est vide ou non
-                $users = $entityManager->getRepository(User::class)->findAll();
-                
-                if(count($users) === 0) // table vide
+                $nb_users = $entityManager
+                        ->createQuery('SELECT COUNT(u.id_user) FROM App\Entity\User u')
+                        ->getSingleScalarResult();
+
+                if($nb_users === 0) // table vide
                 {
                         $_SESSION['user'] = '';
                         $_SESSION['admin'] = false;
-
                         return false;
                 }
                 else{
diff --git a/src/router.php b/src/router.php
index 8aa0aa9..92dccb6 100644
--- a/src/router.php
+++ b/src/router.php
@@ -39,7 +39,7 @@ if($_SERVER['REQUEST_METHOD'] === 'GET'){
     }
 
     // construction d'une page
-    $response = (new ViewController)->buildView($entityManager, $request);
+    $response = (new ViewController)->buildView($entityManager, $request); // utilise Director
 }
 
 
@@ -87,25 +87,19 @@ elseif($_SERVER['REQUEST_METHOD'] === 'POST'){
             $data = file_get_contents('php://input');
             $json = json_decode($data, true);
 
-            if(isset($_GET['action']))
+            if($request->query->has('action'))
             {
                 /* -- manipulation des articles -- */
-                if($_GET['action'] === 'editor_submit' && isset($json['id']) && isset($json['content']))
-                {
+                if($_GET['action'] === 'editor_submit' && isset($json['id']) && isset($json['content'])){
                     ArticleController::editorSubmit($entityManager, $json);
                 }
                 elseif($_GET['action'] === 'delete_article' && isset($json['id'])){
                     $response = ArticleController::deleteArticle($entityManager, $json); // version AJAX
-                    $response->send();
-                    die;
                 }
-                // inversion de la position de deux noeuds
-                elseif($_GET['action'] === 'switch_positions' && isset($json['id1']) && isset($json['id2']))
-                {
+                elseif($_GET['action'] === 'switch_positions' && isset($json['id1']) && isset($json['id2'])){
                     ArticleController::switchPositions($entityManager, $json);
                 }
-                elseif($_GET['action'] === 'date_submit' && isset($json['id']) && isset($json['date']))
-                {
+                elseif($_GET['action'] === 'date_submit' && isset($json['id']) && isset($json['date'])){
                     ArticleController::dateSubmit($entityManager, $json);
                 }
 
@@ -116,29 +110,54 @@ elseif($_SERVER['REQUEST_METHOD'] === 'POST'){
                 elseif($_GET['action'] === 'test_email'){
                     ContactFormController::sendTestEmail($entityManager, $json);
                 }
+
+
+                /* -- upload d'image dans tinymce par copier-coller -- */
+                // collage de HTML contenant une ou plusieurs balises <img>
+                elseif($request->query->get('action') === 'upload_image_url'){
+                    ImageUploadController::uploadImageHtml();
+                }
+                // collage d'une image (code base64 dans le presse-papier) non encapsulée dans du HTML
+                elseif($request->query->get('action') === 'upload_image_base64'){
+                    ImageUploadController::uploadImageBase64();
+                }
+
+
+                /* -- requêtes spécifiques au calendrier -- */
+                elseif($request->query->get('action') === 'new_event'){
+                    CalendarController::newEvent($json, $entityManager);
+                }
+                elseif($request->query->get('action') === 'update_event'){
+                    CalendarController::updateEvent($json, $entityManager);
+                }
+                elseif($request->query->get('action') === 'remove_event'){
+                    CalendarController::removeEvent($json, $entityManager);
+                }
+                else{
+                    echo json_encode(['success' => false]);
+                    die;
+                }
             }
 
+
             /* -- page Menu et chemins -- */
             elseif(isset($_GET['menu_edit']))
             {
-                // récupération des données (serait peut-être mieux dans la classe)
-                Director::$menu_data = new Menu($entityManager);
+                // ne suit pas la règle, faire ça dans un contrôleur
+                Director::$menu_data = new Menu($entityManager); // récupération des données
 
                 // flèche gauche <=: position = position du parent + 1, parent = grand-parent, recalculer les positions
                 if($_GET['menu_edit'] === 'move_one_level_up' && isset($json['id'])){
                     MenuAndPathsController::MoveOneLevelUp($entityManager, $json);
                 }
-
                 // flèche droite =>: position = nombre d'éléments de la fraterie + 1, l'élément précédent devient le parent
-                if($_GET['menu_edit'] === 'move_one_level_down' && isset($json['id'])){
+                elseif($_GET['menu_edit'] === 'move_one_level_down' && isset($json['id'])){
                     MenuAndPathsController::MoveOneLevelDown($entityManager, $json);
                 }
-
-                if($_GET['menu_edit'] === 'switch_positions' && isset($json['id1']) && isset($json['id2'])){
+                elseif($_GET['menu_edit'] === 'switch_positions' && isset($json['id1']) && isset($json['id2'])){
                     MenuAndPathsController::switchPositions($entityManager, $json);
                 }
-
-                if($_GET['menu_edit'] === 'displayInMenu' && isset($json['id']) && isset($json['checked'])){
+                elseif($_GET['menu_edit'] === 'displayInMenu' && isset($json['id']) && isset($json['checked'])){
                     MenuAndPathsController::displayInMenu($entityManager, $json);
                 }
             }
@@ -161,45 +180,18 @@ elseif($_SERVER['REQUEST_METHOD'] === 'POST'){
             elseif($request->query->has('bloc_edit'))
             {
                 // renommage d'un bloc
-                if($request->query->get('bloc_edit') === 'rename_page_bloc')
-                {
+                if($request->query->get('bloc_edit') === 'rename_page_bloc'){
                     PageManagementController::renameBloc($entityManager, $json);
                 }
                 // inversion des positions de deux blocs
-                elseif($request->query->get('bloc_edit') === 'switch_blocs_positions')
-                {
+                elseif($request->query->get('bloc_edit') === 'switch_blocs_positions'){
                     PageManagementController::SwitchBlocsPositions($entityManager, $json);
                 }
             }
-
-            /* -- upload d'image dans tinymce par copier-coller -- */
-            // collage de HTML contenant une ou plusieurs balises <img>
-            if($request->query->has('action') && $request->query->get('action') == 'upload_image_html'){
-                ImageUploadController::uploadImageHtml();
-            }
-            // collage d'une image (code base64 dans le presse-papier) non encapsulée dans du HTML
-            elseif($request->query->has('action') && $request->query->get('action') == 'upload_image_base64'){
-                ImageUploadController::uploadImageBase64();
-            }
-            
-            /* -- requêtes spécifiques au calendrier -- */
-            if($request->query->get('action') === 'new_event'){
-                CalendarController::newEvent($json, $entityManager);
-            }
-            elseif($request->query->get('action') === 'update_event'){
-                CalendarController::updateEvent($json, $entityManager);
-            }
-            elseif($request->query->get('action') === 'remove_event'){
-                CalendarController::removeEvent($json, $entityManager);
-            }
-            else{
-                echo json_encode(['success' => false]);
-            }
-            die;
         }
 
         // upload d'image dans tinymce avec le plugin (bouton "insérer une image" de l'éditeur)
-        elseif(strpos($_SERVER['CONTENT_TYPE'], 'multipart/form-data') !== false && $request->query->has('action') && $request->query->get('action') === 'upload_image')
+        elseif(strpos($_SERVER['CONTENT_TYPE'], 'multipart/form-data') !== false && $request->query->has('action') && $request->query->get('action') === 'upload_image_tinymce')
         {
             ImageUploadController::imageUploadTinyMce();
         }
@@ -207,7 +199,7 @@ elseif($_SERVER['REQUEST_METHOD'] === 'POST'){
         elseif(isset($_SERVER['HTTP_X_REQUESTED_WITH']) && $_SERVER['HTTP_X_REQUESTED_WITH'] == 'XMLHttpRequest')
         {
             //echo "requête XMLHttpRequest reçue par le serveur";
-            echo json_encode(['success' => false]); // ça marche mais ça marche pas...
+            echo json_encode(['success' => false]); // noyer le poisson en laissant penser que le site gère les requêtes XHR
             die;
         }
 
@@ -215,14 +207,7 @@ elseif($_SERVER['REQUEST_METHOD'] === 'POST'){
         elseif($_SERVER['CONTENT_TYPE'] === 'application/x-www-form-urlencoded')
         {
             if($_GET['action'] === 'delete_article' && isset($_GET['id'])){
-                $response = json_decode(ArticleController::deleteArticle($entityManager, $_GET)->getContent(), true); // version formulaire
-                $url = new URL;
-                if(isset($_GET['from'])){
-                    $url->addParams(['page' => $_GET['from']]);
-                }
-                $url->addParams(['success' => $response['success'], 'message' => $response['message']]);
-                header('Location: ' . $url);
-                die;
+                $response = ArticleController::deleteArticle($entityManager, $_GET); // version formulaire
             }
 
             /* -- nouvelle page -- */
@@ -242,6 +227,7 @@ elseif($_SERVER['REQUEST_METHOD'] === 'POST'){
                 PageManagementController::deletePage($entityManager);
             }
 
+
             /* -- mode Modification d'une page -- */
 
             // modification du chemins en snake_case
@@ -294,20 +280,51 @@ elseif($_SERVER['REQUEST_METHOD'] === 'POST'){
                 die;
             }
         }
+        // POST admin ne matchant pas
+        else{
+            echo json_encode(['success' => false]);
+            die;
+        }
+    }
+    // POST non admin ne matchant pas
+    else{
+        echo json_encode(['success' => false]);
+        die;
     }
-
-    // rien ne match
-    header("Location: " . new URL);
-    die;
 }
 
-
+// méthode inconnue
 else{
     header("Location: " . new URL(['error' => 'tu fais quoi là mec?']));
     die;
 }
 
-// enlever le test isset à terme
+
+
+/* -- utilisation de la réponse -- */
 if(isset($response)){
-    $response->send();
-}
\ No newline at end of file
+    if($_SERVER['REQUEST_METHOD'] === 'POST' && $_SERVER['CONTENT_TYPE'] === 'application/x-www-form-urlencoded')
+    {
+        $response_data = json_decode(($response)->getContent(), true);
+        $url = new URL;
+        if(isset($_GET['from'])){
+            $url->addParams(['page' => $_GET['from']]);
+        }
+        $url->addParams(['success' => $response_data['success'], 'message' => $response_data['message']]);
+        header('Location: ' . $url);
+    }
+    else{
+        $response->send();
+    }
+}
+else{
+    if($_SERVER['REQUEST_METHOD'] === 'POST' && $_SERVER['CONTENT_TYPE'] === 'application/x-www-form-urlencoded')
+    {
+        header("Location: " . new URL(['error' => 'erreur côté serveur']));
+    }
+    else{
+        http_response_code(500);
+        echo "erreur côté serveur";
+    }
+}
+//die; // inutile normalement
\ No newline at end of file
diff --git a/src/view/NewsBlockBuilder.php b/src/view/NewsBlockBuilder.php
index e449c7a..6b54196 100644
--- a/src/view/NewsBlockBuilder.php
+++ b/src/view/NewsBlockBuilder.php
@@ -23,7 +23,7 @@ class NewsBlockBuilder extends AbstractBuilder
 
             $presentation = $node->getNodeData()->getPresentation()->getName(); // affichage list ou grid
 
-            // exécution de la stratégie (utilisation d'une méthode ou d'une classe)
+            // exécution de la stratégie (utilisation de méthodes ou de classe List, Grid, CarouselPresentation)
             $section_class = $presentation;
             $section_child_class = $presentation === 'grid' ? 'grid_columns' : '';
 
diff --git a/src/view/PostBlockBuilder.php b/src/view/PostBlockBuilder.php
index fadae31..6ec91f7 100644
--- a/src/view/PostBlockBuilder.php
+++ b/src/view/PostBlockBuilder.php
@@ -12,8 +12,7 @@ class PostBlockBuilder extends AbstractBuilder
         parent::__construct($node);
 
         // à remplacer par list.php/grid.php (une vue par stratégie) le jour ou ou a besoin de les différencier
-        //$viewFile = self::VIEWS_PATH . $node->getName() . '.php';
-        $viewFile = self::VIEWS_PATH . 'post_block.php'; // actuellement identique à news_block.php
+        $viewFile = self::VIEWS_PATH . $node->getName() . '.php'; // = post_block.php, actuellement identique à news_block.php
         
         if(file_exists($viewFile))
         {
@@ -24,7 +23,7 @@ class PostBlockBuilder extends AbstractBuilder
 
             $presentation = $node->getNodeData()->getPresentation()->getName(); // affichage list ou grid
 
-            // exécution de la stratégie (utilisation d'une méthode ou d'une classe)
+            // exécution de la stratégie (utilisation de méthodes ou de classe List, Grid, CarouselPresentation)
             $section_class = $presentation;
             $section_child_class = $presentation === 'grid' ? 'grid_columns' : '';