From f721a028c8c9f123c78ea424918a6cdd1645e813 Mon Sep 17 00:00:00 2001 From: Emre KARTAL Date: Fri, 16 Dec 2022 22:31:13 +0100 Subject: [PATCH] Optimization of the code and respect of the coding standard, the project is finished :white_check_mark: --- src/.htaccess | 17 ++- src/config/Validation.php | 2 +- src/controller/CtrlUtilisateur.php | 29 +++-- src/controller/CtrlVisiteur.php | 5 +- src/controller/FrontControleur.php | 2 +- src/modele/MdlUtilisateur.php | 57 ++++++++- src/modele/MdlVisiteur.php | 35 ++--- src/modele/gateway/ListeGateway.php | 7 - src/modele/gateway/UtilisateurGateway.php | 2 +- src/vue/Images/trash2.png | Bin 247 -> 0 bytes src/vue/css/style.css | 1 - src/vue/html/PageListePrive.php | 12 +- src/vue/html/PageListePublic.php | 8 +- src/vue/html/erreur.php | 4 +- src/vue/html/inscription.php | 148 +++++++++++----------- src/vue/html/profil.php | 2 +- 16 files changed, 195 insertions(+), 136 deletions(-) delete mode 100644 src/vue/Images/trash2.png diff --git a/src/.htaccess b/src/.htaccess index c9fd233..9496916 100755 --- a/src/.htaccess +++ b/src/.htaccess @@ -10,14 +10,25 @@ RewriteEngine On # Règles de réécriture d'url #--------------------------------- +# Visiteur + RewriteRule ^login$ index.php?action=redirectionLogin [L] RewriteRule ^inscription$ index.php?action=redirectionInscription [L] RewriteRule ^VueListePublic$ index.php?action=redirectionListePublic [L] +RewriteRule ^previousPage$ index.php?action=pagePrécédente [L] +RewriteRule ^nextPage$ index.php?action=pageSuivante [L] +RewriteRule ^check$ index.php?action=check [L] + +# Utilisateur + RewriteRule ^logout$ index.php?action=logout [L] RewriteRule ^MonProfil$ index.php?action=redirectionProfil [L] RewriteRule ^supprimerCompte$ index.php?action=supprimerCompte [L] -RewriteRule ^previousPage$ index.php?action=pagePrécédente [L] -RewriteRule ^nextPage$ index.php?action=pageSuivante [L] -RewriteRule ^Users$ index.php?action=voirUser [L] RewriteRule ^listePrive$ index.php?action=listePrive [L] +RewriteRule ^previousPagePrive$ index.php?action=pagePrécédentePrive [L] +RewriteRule ^nextPagePrive$ index.php?action=pageSuivantePrive [L] +RewriteRule ^checkPrive$ index.php?action=checkPrive [L] + +# Admin + RewriteRule ^users$ index.php?action=redirectionVueUtilisateur [L] \ No newline at end of file diff --git a/src/config/Validation.php b/src/config/Validation.php index a0a06f1..739772d 100755 --- a/src/config/Validation.php +++ b/src/config/Validation.php @@ -91,7 +91,7 @@ class Validation { } static function cleanText(string &$txt) { - $txt = preg_replace('/[^A-Za-z\-\^0-9\ ]/', '', $txt); + $txt = preg_replace('/[^A-Za-z\-\^0-9\ \à\é\^]/', '', $txt); if($txt == null || $txt == '') { return null; diff --git a/src/controller/CtrlUtilisateur.php b/src/controller/CtrlUtilisateur.php index a7b6cf3..5343e31 100755 --- a/src/controller/CtrlUtilisateur.php +++ b/src/controller/CtrlUtilisateur.php @@ -16,7 +16,6 @@ class CtrlUtilisateur { switch($action) { - case NULL: $this->ConsulterListePrive($dVueErreur); break; @@ -64,6 +63,14 @@ class CtrlUtilisateur { case "SupprimerListePrive": $this->SupprimerListePrive($dVueErreur); break; + + case "pagePrécédentePrive": + $this->listePrécédente($dVueErreur); + break; + + case "pageSuivantePrive": + $this->listeSuivante($dVueErreur); + break; default: $dVueErreur[] = "Erreur d'appel php"; @@ -129,20 +136,20 @@ class CtrlUtilisateur { function ConsulterListePrive(array $dVueErreur) { global $rep,$vues; $listesPrive = MdlUtilisateur::RecupererListePrive(); - $taches = MdlVisiteur::RecupererTache(); + $taches = MdlUtilisateur::RecupererTache(); $action=NULL; require ($rep.$vues['listPrive']); } function SupprimerTachePrive(array $dVueErreur){ global $rep,$vues; - $tache = MdlVisiteur::SupprimerTache(); + $tache = MdlUtilisateur::SupprimerTache(); $this->ConsulterListePrive($dVueErreur); } function SupprimerListePrive(array $dVueErreur){ global $rep,$vues; - $liste = MdlVisiteur::SupprimerListe(); + $liste = MdlUtilisateur::SupprimerListe(); $this->ConsulterListePrive($dVueErreur); } @@ -150,14 +157,14 @@ class CtrlUtilisateur { public function AjouterTachePrive(array $dVueErreur){ global $rep,$vues; try{ - $tache = MdlVisiteur::AjouterTache($dVueErreur); + $tache = MdlUtilisateur::AjouterTache($dVueErreur); $this->ConsulterListePrive($dVueErreur); } catch (Exception $e) { $ErreurLog=$e->getMessage(); $listesPrive= MdlUtilisateur::RecupererListePrive(); - $taches = MdlVisiteur::RecupererTache(); + $taches = MdlUtilisateur::RecupererTache(); $action=NULL; require ($rep.$vues['listPrive']); } @@ -167,14 +174,14 @@ class CtrlUtilisateur { public function ModifierListePrive(array $dVueErreur){ global $rep,$vues; try{ - $tache = MdlVisiteur::ModifierListe($dVueErreur); + $tache = MdlUtilisateur::ModifierListe($dVueErreur); $this->ConsulterListePrive($dVueErreur); } catch (Exception $e) { $ErreurLog=$e->getMessage(); $listesPrive= MdlUtilisateur::RecupererListePrive(); - $taches = MdlVisiteur::RecupererTache(); + $taches = MdlUtilisateur::RecupererTache(); $action=NULL; require ($rep.$vues['listPublic']); } @@ -183,7 +190,7 @@ class CtrlUtilisateur { public function checkPrive(array $dVueErreur) { global $rep,$vues; - $tache = MdlVisiteur::check(); + $tache = MdlUtilisateur::check(); $this->ConsulterListePrive($dVueErreur); } @@ -198,13 +205,11 @@ class CtrlUtilisateur { { $ErreurLog=$e->getMessage(); $listes = MdlUtilisateur::RecupererListePrive(); - $taches = MdlVisiteur::RecupererTache(); + $taches = MdlUtilisateur::RecupererTache(); $action=NULL; require ($rep.$vues['listPrive']); } - } - public function listePrécédente(array $dVueErreur){ if($_COOKIE['pageUser'] > 1){ diff --git a/src/controller/CtrlVisiteur.php b/src/controller/CtrlVisiteur.php index 5854989..98117a6 100755 --- a/src/controller/CtrlVisiteur.php +++ b/src/controller/CtrlVisiteur.php @@ -160,10 +160,7 @@ class CtrlVisiteur { function SupprimerListe(array $dVueErreur){ global $rep,$vues; $liste = MdlVisiteur::SupprimerListe(); - $listes = MdlVisiteur::RecupererListePublic(); - $taches = MdlVisiteur::RecupererTache(); - $action=NULL; - require ($rep.$vues['listPublic']); + $this->ConsulterListePublic($dVueErreur); } public function AjouterTache(array $dVueErreur){ diff --git a/src/controller/FrontControleur.php b/src/controller/FrontControleur.php index a601ae8..5dec9d6 100755 --- a/src/controller/FrontControleur.php +++ b/src/controller/FrontControleur.php @@ -14,7 +14,7 @@ class FrontControleur { $string_actor=' '; $listeActions=array( - 'Utilisateur' => array('logout','redirectionProfil','supprimerCompte', 'listePrive','AjouterListePrive', 'checkPrive', 'SupprimerListePrive', 'AjouterTachePrive', 'ModifierListePrive', 'SupprimerTachePrive'), + 'Utilisateur' => array('logout','redirectionProfil','pageSuivantePrive','pagePrécédentePrive','supprimerCompte','modifMdp' ,'listePrive','AjouterListePrive', 'checkPrive', 'SupprimerListePrive', 'AjouterTachePrive', 'ModifierListePrive', 'SupprimerTachePrive'), 'Admin' => array('redirectionVueUtilisateur','SupprimerUtilisateur') ); diff --git a/src/modele/MdlUtilisateur.php b/src/modele/MdlUtilisateur.php index ccd7a39..2cba037 100755 --- a/src/modele/MdlUtilisateur.php +++ b/src/modele/MdlUtilisateur.php @@ -53,6 +53,26 @@ class MdlUtilisateur $id = Validation::cleanInt($_SESSION['id']); return $listeGtw->CountListe(intval($id)); } + + public function ModifierListe(&$dVueErreur){ + $userGtw = new ListeGateway(); + $id = $_POST['idListe']; + $nom = Validation::cleanText($_POST['nom-modif-liste']); + $description = Validation::cleanText($_POST['description-modif-liste']); + + if($nom == NULL && $description==null){ + return null; + } + if($description == NULL){ + $userGtw->EditerNom($id, $nom); + } + else if($nom == NULL){ + $userGtw->EditerDescription($id, $description); + } + else{ + $userGtw->Editer($id, $nom, $description); + } + } public static function suppressionUtilisateur(){ @@ -85,12 +105,12 @@ class MdlUtilisateur } public static function AjouterListePrive(&$dVueErreur){ - $taskGtw = new ListeGateway(); + $listGtw = new ListeGateway(); $nom=$_POST['nom-ajout-liste']; $description=$_POST['description-ajout-liste']; $idCreateur =Validation::cleanInt($_SESSION['id']); Validation::val_form_add($nom,$description,$dVueErreur); - $taskGtw->Ajouter($nom, $description,0, $idCreateur); + $listGtw->Ajouter($nom, $description,0, $idCreateur); } public static function RecupererListePrive(){ @@ -101,5 +121,38 @@ class MdlUtilisateur return $listeGtw->getListePrive(1,10); } + public static function RecupererTache(){ + $taskGtw = new TacheGateway(); + return $taskGtw->getTache(); + } + + public static function SupprimerListe(){ + $listeGtw = new ListeGateway(); + $taskGtw = new TacheGateway(); + $id = $_GET['idListe']; + $taskGtw->SupprimerViaListe($id); + $listeGtw->Supprimer($id); + } + + public static function check(){ + $taskGtw = new TacheGateway(); + $id = $_POST['idTache']; + $taskGtw->isDone($id); + } + + public static function SupprimerTache(){ + $taskGtw = new TacheGateway(); + $id = $_GET['idTache']; + $taskGtw->Supprimer($id); + } + + public function AjouterTache(&$dVueErreur){ + $taskGtw = new TacheGateway(); + $id = $_POST['idListe']; + $nom = $_POST['nom-ajout']; + $description = $_POST['description-ajout']; + Validation::val_form_add($nom,$description,$dVueErreur); + $taskGtw->AjouterTache($nom, $description,false,$id); + } } diff --git a/src/modele/MdlVisiteur.php b/src/modele/MdlVisiteur.php index 3e542c6..b1d68d4 100755 --- a/src/modele/MdlVisiteur.php +++ b/src/modele/MdlVisiteur.php @@ -16,31 +16,31 @@ class MdlVisiteur $userGtw->AjouterUtilisateur($_POST["nom-Form"],$_POST["prenom-Form"],$_POST["pseudo-Form"],$_POST["mail"],$hash); } -public static function RecupererListePublic(){ - $userGtw = new ListeGateway(); + public static function RecupererListePublic(){ + $listGtw = new ListeGateway(); if(isset($_COOKIE["page"])) - return $userGtw->getListePublic(($_COOKIE["page"]-1)*10,10); + return $listGtw->getListePublic(($_COOKIE["page"]-1)*10,10); else - return $userGtw->getListePublic(1,10); + return $listGtw->getListePublic(1,10); } public static function RecupererTache(){ - $userGtw = new TacheGateway(); - return $userGtw->getTache(); + $taskGtw = new TacheGateway(); + return $taskGtw->getTache(); } public static function SupprimerTache(){ - $userGtw = new TacheGateway(); + $taskGtw = new TacheGateway(); $id = $_GET['idTache']; - $userGtw->Supprimer($id); + $taskGtw->Supprimer($id); } public static function SupprimerListe(){ - $userGtw = new ListeGateway(); + $listGtw = new ListeGateway(); $taskGtw = new TacheGateway(); $id = $_GET['idListe']; $taskGtw->SupprimerViaListe($id); - $userGtw->Supprimer($id); + $listGtw->Supprimer($id); } public function AjouterTache(&$dVueErreur){ @@ -53,19 +53,22 @@ public static function RecupererListePublic(){ } public function ModifierListe(&$dVueErreur){ - $userGtw = new ListeGateway(); + $listGtw = new ListeGateway(); $id = $_POST['idListe']; $nom = Validation::cleanText($_POST['nom-modif-liste']); $description = Validation::cleanText($_POST['description-modif-liste']); + if($nom == NULL && $description==null){ + return null; + } if($description == NULL){ - $userGtw->EditerNom($id, $nom); + $listGtw->EditerNom($id, $nom); } else if($nom == NULL){ - $userGtw->EditerDescription($id, $description); + $listGtw->EditerDescription($id, $description); } else{ - $userGtw->Editer($id, $nom, $description); + $listGtw->Editer($id, $nom, $description); } } @@ -77,9 +80,9 @@ public static function RecupererListePublic(){ $listGtw->Ajouter($nom, $description,1, 1); } - public function check(){ + public static function check(){ $taskGtw = new TacheGateway(); - $id = $_POST['idTache']; + $id = Validation::cleanInt($_POST['idTache']); $taskGtw->isDone($id); } diff --git a/src/modele/gateway/ListeGateway.php b/src/modele/gateway/ListeGateway.php index 677fb49..319175a 100755 --- a/src/modele/gateway/ListeGateway.php +++ b/src/modele/gateway/ListeGateway.php @@ -47,13 +47,6 @@ class ListeGateway{ $this->con->executeQuery($query,array('id' => array($id, PDO::PARAM_INT))); } - public function getListe(int $offset, int $limit){ - $query = "SELECT * FROM ToDoList_Liste LIMITS $offset,$limit"; - $this->con->executeQuery($query); - $listes=$this->con->getResults(); - return $listes; - } - public function getListePublic($offset,$limit){ $query = "SELECT * FROM ToDoList_Liste WHERE estPublic LIMIT $offset, $limit"; $this->con->executeQuery($query); diff --git a/src/modele/gateway/UtilisateurGateway.php b/src/modele/gateway/UtilisateurGateway.php index 0dfdb89..bf00d5d 100755 --- a/src/modele/gateway/UtilisateurGateway.php +++ b/src/modele/gateway/UtilisateurGateway.php @@ -27,7 +27,7 @@ class UtilisateurGateway{ $query="SELECT * FROM ToDoList_Admin WHERE idAdmin=:id"; $this->con->executeQuery($query, array( 'id' => array($id, PDO::PARAM_INT))); - $results=$this->con->getResults(); + $results=$this->con->getResults(); if($results!=null){ return true; }else{ diff --git a/src/vue/Images/trash2.png b/src/vue/Images/trash2.png deleted file mode 100644 index c6e7b984ee6f0d951045fed6b9efd11a39ead45c..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 247 zcmeAS@N?(olHy`uVBq!ia0vp^4?&oN8Ax*GDtQ1Y#Q>iW*Z=?j1DVgBJ$v=)m7Sek zK|#Ud#f!gv`?h!Q-d($Ptyr;wxm9!yP{l+~7srr_Id8As=4~+Ga0sjkyIY?izM)0f;ch&u!e(yKU+u0ahdZAg@xM2Od$}P`Xx4oLyH!Z9pWO<29 zW@~t<*YT-g7XNsT*SA0ZztsLlyZw&Lq(8ewuo=q0@Sr=-sg*ORvflhUNP(xTpUXO@ GgeCypWmVPy diff --git a/src/vue/css/style.css b/src/vue/css/style.css index 20ff96d..a7fc215 100755 --- a/src/vue/css/style.css +++ b/src/vue/css/style.css @@ -26,7 +26,6 @@ body::before { z-index: -1; } - /* Model */ .model { diff --git a/src/vue/html/PageListePrive.php b/src/vue/html/PageListePrive.php index 8563d49..6a41777 100644 --- a/src/vue/html/PageListePrive.php +++ b/src/vue/html/PageListePrive.php @@ -200,7 +200,7 @@ $total = $total + 1; ?>
  • -
    + getEstValide() == 1) echo "checked" ?>> @@ -209,7 +209,7 @@
    • @@ -240,21 +240,21 @@ -