UPDATE en POO - MVC

**Demirdiouff** · 01/06/2019, 23h41

Bonjour,

Je me permets de faire appel à vous car je rencontre un soucis d'UPDATE dans le cadre de POO MVC avec lequel je m'exerce.

Je n'arrive pas a faire fonctionner ce bout de code, et je ne rentre même pas dans mes conditions, le résultat reste toujours la même (redirection dans le 'else')
Voici mes bouts de code qui parleront mieux que moi :

Routeur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
            switch ($action) {
                case 'pageModifierChapitre':
                    $this->ctrlPageModifierChapitre = new ControleurEspaceAdmin();
                    $this->ctrlPageModifierChapitre->pageModifierChapitre();
                    break;
                case 'modifierChapitre':
                    $this->ctrlModifierChapitre = new ControleurEspaceAdmin();
                    $this->ctrlModifierChapitre->modifierChapitre();
                    break;

Contrôleur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
    public function pageModifierChapitre($msgErreurModifChapitre = '', $msgValideModifChapitre = '') {
        if (!isset($_SESSION['isAdmin'])) {
            header("Location:index.php?");
        }
        if (isset($_REQUEST['idPost'])) {
            $idPost = $_REQUEST['idPost'];
            $titrePost = $_REQUEST['titrePost'];
        }
        $post = $this->postManager->getPost($idPost);
        $vue = new Vue("PageModifierChapitre", "Modification : " . $titrePost);
        $vue->genererPageEspaceAdmin(array(
            'msgErreurModifChapitre' => $msgErreurModifChapitre,
            'msgValideModifRoman' => $msgValideModifChapitre,
            'post' => $post
        ));
    }
 
    public function modifierChapitre() {
        $msgErreurModifChapitre = '';
        $msgValideModifChapitre = '';
        if (isset($_POST['modifierChapitre']) && isset($_POST['auteurPost']) && isset($_POST['titrePost']) && isset($_POST['contenuPost']) && ($_GET['idPost'])) {
            $post = new Post([
                'auteurPost' => $_POST['auteurPost'],
                'titrePost' => $_POST['titrePost'],
                'contenuPost' => $_POST['contenuPost'],
                'idPost' => $_GET['idPost']
            ]);
            if (!$post->auteurValide()) {
                echo 'erreur auteur';
                $msgErreurModifChapitre = '<p style="color:red">Erreur : Le nom de l\'auteur ne peut être vide</p>';
                $this->espaceAdmin($msgErreurModifChapitre, '');
            } elseif (!$post->titreValide()) {
                echo 'erreur titre';
                $msgErreurModifChapitre = '<p style="color:red">Erreur : Le titre du chapitre ne peut être vide</p>';
                $this->espaceAdmin($msgErreurModifChapitre, '');
            } elseif (!$post->contenuValide()) {
                echo 'erreur contenu';
                $msgErreurModifChapitre = '<p style="color:red">Erreur : Le contenu du chapitre ne peut être vide</p>';
                $this->espaceAdmin($msgErreurModifChapitre, '');
            } else {
                echo 'modif réussi';
               $this->postManager->update($post);
               $msgValideModifChapitre = '<p style="color:green">Votre chapitre a bien été mis à jour</p>';
               $this->espaceAdmin('', $msgValideModifChapitre);
            }
        } else {
            echo 'rien passé';
            $this->espaceAdmin();
        }
    }

Lorsque je tente un essai, je rentre tout le temps dans le cas 'rien passé'

Modele :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
    public function update(Post $post) {
        // Connexion à la BDD
        // Prépare une requête de type UPDATE
        // Assignation des valeurs à la requête
        // Exécution de la requête
        $bdd = $this->getBdd();
        $update = $bdd->prepare('UPDATE posts SET date_post = :date_post, titre_post = :titre_post, contenu_post = :contenu_post WHERE id_post = :id_post');
        $update->bindValue(':date_post', $post->datePost());
        $update->bindValue(':titre_post', $post->titrePost());
        $update->bindValue(':contenu_post', $post->contenuPost());
        $update->execute();
    }

Vue :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
			<h2 class="sub-header">Modification "<strong><?= $post->titrePost()?></strong>"</h2>
			<form method="post" action="index.php?action=modifierChapitre">
			<div class="table-responsive">
				<div class="form-group">
					<label for="usr">Auteur :</label> <input type="text"
						class="form-control" id="usr" value="<?= $post->auteurPost()?>">
				</div>
				<div class="form-group">
					<label for="pwd">Titre du chapitre :</label> <input type="text"
						class="form-control" id="pwd" value="<?= $post->titrePost()?>">
				</div>
				<div class="form-group">
					<label for="comment">Contenu :</label>
					<textarea rows="" cols=""><?= $post->contenuPost()?></textarea>
					<!-- <textarea>Next, get a free Tiny Cloud API key!</textarea> -->
				</div>
			</div>
			<a href="index.php?action=modifierChapitre&idPost=<?= $post->id()?>" class="btn btn-warning">
			<input type="submit" name="modifierChapitre" value="Modifier"><span class="glyphicon glyphicon-download-alt"></span></a>
			</form>
<?php
if (!empty($msgErreurModifChapitre)) {
echo $msgErreurModifChapitre;
} ?>

Merci d'avance pour tout vos retours

**rawsrc** · 02/06/2019, 09h44

Salut,

plusieurs remarques :
après une redirection, tu dois toujours sortir du script donc header("Location:index.php?"); doit être :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
header("Location:index.php?");
exit;

ensuite si tu sors toujours par le else c'est qu'une de tes condition n'est pas remplie. Note que ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if (isset($_POST['modifierChapitre']) && isset($_POST['auteurPost']) && isset($_POST['titrePost']) && isset($_POST['contenuPost']) && ($_GET['idPost']))

peut être remplacé par cela :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if (isset($_POST['modifierChapitre'], $_POST['auteurPost'], $_POST['titrePost'], $_POST['contenuPost'], $_GET['idPost']) && is_int($_GET['idPost']))

enfin si tu avais testé tes conditions une par une tu aurais vu qu'il te manque le nom des contrôles sur ton formulaire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
<input type="text" class="form-control" id="usr" value="<?= $post->auteurPost()?>">
// par 
<input type="text" name="auteurPost" class="form-control" id="usr" value="<?= $post->auteurPost()?>">

et pour finir tu dois TOUJOURS échapper les valeurs que tu envoies au navigateur avec htmlspecialchars() et pas faire simplement : value="<?= $post->auteurPost()?>" à moins que $post->auteurPost() soit déjà sécurisé

**Demirdiouff** · 02/06/2019, 11h40

Salut,

Je te remercie pour toutes tes remarques, ça m'aide beaucoup !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
if (isset($_POST['modifierChapitre'], $_POST['auteurPost'], $_POST['titrePost'], $_POST['contenuPost'], $_GET['idPost']) && is_int($_GET['idPost']))

un seul "isset" pour tout vérifier ? J'avais d'abord rédigé mon code de cette manière (sans la vérification is_int à la fin), mais on m'avait corrigé en ajoutant un "isset" sur chaque "$_POST" donc je me retrouve un peu confus

Le fait de l'écrire avec un && et "," doit être différent si je comprend bien ? Avec un && je suis obligé de revérifier tandis qu'avec "," ça reste dans la même vérification ?

Erreur d'inattention de ma part pour les name dans le formulaire... je ne suis pas encore très à l'aise avec le debug.
Par contre, malgré les "name" ajouté, mon script part toujours dans le else...

**rawsrc** · 02/06/2019, 12h36

Poste voir ton code corrigé stp

**Demirdiouff** · 02/06/2019, 19h25

Contrôleur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
 
    public function pageModifierChapitre($msgErreurModifChapitre = '', $msgValideModifChapitre = '') {
        if (!isset($_SESSION['isAdmin'])) {
            header("Location:index.php?");
            exit();
        }
        if (isset($_REQUEST['idPost'])) {
            $idPost = $_REQUEST['idPost'];
            $titrePost = $_REQUEST['titrePost'];
        }
        $post = $this->postManager->getPost($idPost);
        $vue = new Vue("PageModifierChapitre", "Modification : " . $titrePost);
        $vue->genererPageEspaceAdmin(array(
            'msgErreurModifChapitre' => $msgErreurModifChapitre,
            'msgValideModifRoman' => $msgValideModifChapitre,
            'post' => $post
        ));
    }
 
    public function modifierChapitre() {
        $msgErreurModifChapitre = '';
        $msgValideModifChapitre = '';
        if (isset($_POST['modifierChapitre'], $_POST['auteurPost'], $_POST['titrePost'], $_POST['contenuPost'], $_GET['idPost']) && is_int($_GET['idPost'])){
            $post = new Post([
                'auteurPost' => $_POST['auteurPost'],
                'titrePost' => $_POST['titrePost'],
                'contenuPost' => $_POST['contenuPost'],
                'idPost' => $_GET['idPost']
            ]);
            if (!$post->auteurValide()) {
                echo 'erreur auteur';
                $msgErreurModifChapitre = '<p style="color:red">Erreur : Le nom de l\'auteur ne peut être vide</p>';
                $this->espaceAdmin($msgErreurModifChapitre, '');
            } elseif (!$post->titreValide()) {
                echo 'erreur titre';
                $msgErreurModifChapitre = '<p style="color:red">Erreur : Le titre du chapitre ne peut être vide</p>';
                $this->espaceAdmin($msgErreurModifChapitre, '');
            } elseif (!$post->contenuValide()) {
                echo 'erreur contenu';
                $msgErreurModifChapitre = '<p style="color:red">Erreur : Le contenu du chapitre ne peut être vide</p>';
                $this->espaceAdmin($msgErreurModifChapitre, '');
            } else {
                echo 'ajout réussi';
               $this->postManager->update($post);
               $msgValideModifChapitre = '<p style="color:green">Votre chapitre a bien été mis à jour</p>';
               $this->espaceAdmin('', $msgValideModifChapitre);
            }
        } else {
            echo 'rien passé';
            $this->espaceAdmin();
        }
    }

Vue :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
			<form method="post" action="index.php?action=modifierChapitre">
			<div class="table-responsive">
				<div class="form-group">
					<label for="usr">Auteur :</label> 
					<input type="text" class="form-control" id="usr" name="auteurPost" value="<?= htmlspecialchars($post->auteurPost()) ?>">
				</div>
				<div class="form-group">
					<label for="pwd">Titre du chapitre :</label> 
					<input type="text" class="form-control" id="pwd" name="titrePost" value="<?= htmlspecialchars($post->titrePost()) ?>">
				</div>
				<div class="form-group">
					<label for="comment">Contenu :</label>
					<textarea name="contenuPost"><?= htmlspecialchars($post->contenuPost()) ?></textarea>
					<!-- <textarea>Next, get a free Tiny Cloud API key!</textarea> -->
				</div>
			</div>
			<a href="index.php?action=modifierChapitre&idPost=<?= $post->id() ?>" class="btn btn-warning">
			<input type="submit" name="modifierChapitre" value="Modifier"><span class="glyphicon glyphicon-download-alt"></span></a>
			</form>
<?php
if (!empty($msgErreurModifChapitre)) {
echo $msgErreurModifChapitre;
} ?>

Le Modèle et le Routeur n'ont pas bougé

**rawsrc** · 02/06/2019, 21h04

euh action="index.php?action=modifierChapitre", il ne manquerait pas idPost comme tu testes $_GET['idPost'], je dis ça par hasard...
si tu ne fais un poil de débogage, je pense que ce fil va s'étirer mais s'étirer...

**Demirdiouff** · 02/06/2019, 21h24

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

<form method="post" action="index.php?action=modifierChapitre&idPost=<?= $post->id() ?>">

Même sous ce format la, j'ai le résultat "rien passé", pourtant l'idPost arrive bien à se balader de page en page

**rawsrc** · 02/06/2019, 21h32

donc il semblerait que le souci se balade du côté de $_GET['idPost'].
Tu n'as pas d'idée ?
remplace && is_int($_GET['idPost']) par && ctype_digit($_GET['idPost']) && ((int)$_GET['idPost'] > 0)

**Demirdiouff** · 02/06/2019, 21h43

J'aimerais bien te dire "problème résolu ! merci énormement" mais malheureusement toujours pas...

En revanche j'ai trouvé une petite variante; quand je teste simplement :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
        if (isset($_POST['modifierChapitre'], $_POST['auteurPost'], $_POST['titrePost'], $_POST['contenuPost'], $_GET['idPost'])){

J'arrive a rentrer dans la condition "ajout réussi", et l'étape suivante était le message d'erreur :

SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens

Erreur qui par la suite a été rectifiée en modifiant le Modele, car 'date_post' n'existe pas dans ce contexte et qu'il manquait également le bind 'id_post'.
A l'heure actuelle, l'UPDATE fonctionne, mais du coup, il ne contient pas la vérification que tu souhaites y ajouter...

UPDATE en POO - MVC

PHP & Base de données

Discussions similaires

Partager

Partager