Choix suivant condition if/else

**JovaanC** · 13/11/2018, 08h32

Bonjour à tous,
je suis bloqué depuis hier,
l'appui sur mon bouton "enregistrer" ne fonctionne plus pour le INSERT INTO mais seulement pour le UPDATE, je ne comprend pas, je suis débutant.
Voici mon code :

Code php :

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
<?php
include '../lib/includes.php';
 
/**
* La sauvegarde
**/
if(isset($_POST['name']) && isset($_POST['slug'])){
    checkCsrf();
    $slug = $_POST['slug'];
    if(preg_match('/^[a-z\-0-9]+$/', $slug)){
        $name = $db->quote($_POST['name']);
        $slug = $db->quote($_POST['slug']);
        $category_id = $db->quote($_POST['category_id']);
        $content = $db->quote($_POST['content']);
 
        /**
        * SAUVEGARDE de la réalisation
        **/
        if(isset($_GET['id'])){
          $id = $db->quote($_GET['id']);
            $db->query("UPDATE works SET name=$name, slug=$slug, content=$content, category_id=$category_id WHERE id=$id");
        }else{
      		 $db->query("INSERT INTO works SET name=$name, slug=$slug, content=$content, category_id=$category_id");
            $_GET['id'] = $db->lastInsertId();
 
        }
        setFlash('La réalisation a bien été ajoutée');
 
        /**
        * ENVOIS DES IMAGES
        **/
        $work_id = $db->quote($_GET['id']);
        $files = $_FILES['images'];
        $images = array();
        foreach($files['tmp_name'] as $k => $v){
            $image = array(
                'name' => $files['name'][$k],
                'tmp_name' => $files['tmp_name'][$k]
            );
            $extension = pathinfo($image['name'], PATHINFO_EXTENSION);
            if(in_array($extension, array('jpg','png'))){
                $db->query("INSERT INTO images SET work_id=$work_id");
                $image_id = $db->lastInsertId();
                $image_name = $image_id . '.' . $extension;
                move_uploaded_file($image['tmp_name'], IMAGES . '/works/' . $image_name);
                resizeImage(IMAGES . '/works/' . $image_name, 150,150);
                $image_name = $db->quote($image_name);
                $db->query("UPDATE images SET name=$image_name WHERE id = $image_id");
            }
 
        }
 
		header('Location:work.php');
        die();
 
    }else{
        setFlash('Le slug n\'est pas valide', 'danger');
    }
}
 
/**
* On récupère une réalisation
**/
if(isset($_GET['id'])){
    $id = $db->quote($_GET['id']);
    $select = $db->query("SELECT * FROM works WHERE id=$id");
    if($select->rowCount() == 0){
        setFlash("Il n'y a pas de réalisation avec cet ID", 'danger');
        header('Location:work.php');
        die();
    }
    $_POST = $select->fetch();
 
}
 
/**
* Suppression d'une image
**/
if(isset($_GET['delete_image'])){
    checkCsrf();
    $id = $db->quote($_GET['delete_image']);
    $select = $db->query("SELECT name, work_id FROM images WHERE id=$id");
    $image = $select->fetch();
    $images=glob(IMAGES . '/works/' . pathinfo($image['name'], PATHINFO_FILENAME) . '_*x*.*');
    if(is_array($images)){
        foreach($images as $v){
            unlink($v);
        }
    }
    unlink(IMAGES . '/works/' . $image['name']);
    $db->query("DELETE FROM images WHERE id=$id");
    setFlash("L'image a bien été supprimée");
    header('Location:work_edit.php?id=' . $image['work_id']);
    die();
}
 
 
 
/**
* On récup la liste des catégories
**/
$select = $db->query('SELECT id, name FROM categories ORDER BY name ASC');
$categories = $select->fetchAll();
$categories_list = array();
foreach($categories as $category){
    $categories_list[$category['id']] = $category['name'];
}
 
/**
* On récup la liste des images
**/
if(isset($_GET['id'])){
    $work_id = $db->quote($_GET['id']);
    $select = $db->query("SELECT id, name FROM images WHERE work_id=$work_id");
    $images = $select->fetchAll();
}else{
    $images = array();
}
 
include '../partials/admin_header.php';
?>
 
 
 
 
<h1>Editer une réalisation</h1>
 
<div class="row">
    <form action="#" method="post" enctype="multipart/form-data">
    <div class="col-sm-8">
            <div class="form-group">
                <label for="name">Nom de la réalisation</label>
                <?= input('name'); ?>
            </div>
            <div class="form-group">
                <label for="slug">URL de la réalisation</label>
                <?= input('slug'); ?>
            </div>
            <div class="form-group">
                <label for="content">Contenu de la réalisation</label>
                <?= textarea('content'); ?>
            </div>
            <div class="form-group">
                <label for="category_id">Catégorie</label>
                <?= select('category_id', $categories_list); ?>
            </div>
            <?= csrfInput(); ?>
            <button type="submit" class="btn btn-default">Enregistrer</button>
    </div>
 
    <div class="col-sm-4">
        <?php foreach ($images as $k => $image): ?>
            <p>
                <a href="?delete_image=<?= $image['id']; ?>&<?= csrf(); ?>" onclick="return confirm('Sur ?');"> <img src="<?= WEBROOT; ?>/img/works/<?= $image['name']; ?>" width="100"></a>
            </p>
        <?php endforeach ?>
 
        <div class="form-group">
            <input type="file" name="images[]">
            <input type="file" name="images[]" class="hidden" id="duplicate">
        </div>
        <p>
            <a href="#" class="btn btn-success" id="duplicatebtn">Ajouter une image</a>
        </p>
    </div>
 
    </form>
 
</div>
 
 
 
<?php ob_start(); ?>
<script src="<?= WEBROOT; ?>/js/tinymce/tinymce.min.js"></script>
<script>
(function($){
 
    $('#duplicatebtn').click(function(e){
        e.preventDefault();
        var $clone = $('#duplicate').clone().attr('id', '').removeClass('hidden');
        $('#duplicate').before($clone);
    })
 
 
})(jQuery);
tinyMCE.init({
        // General options
        mode : "textareas",
 
});
</script>
<?php $script = ob_get_clean(); ?>
 
<?php include '../partials/footer.php'; ?>

Merci d'avance pour votre aide.

**Dendrite** · 14/11/2018, 00h31

Bonsoir Jovaanc, sur le strict problème qui te préoccupe

Code :

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

1
2
3
4
5
6
7
8
  if(isset($_GET['id'])){
          $id = $db->quote($_GET['id']);
            $db->query("UPDATE works SET name=$name, slug=$slug, content=$content, category_id=$category_id WHERE id=$id");
        }else{
      		 $db->query("INSERT INTO works SET name=$name, slug=$slug, content=$content, category_id=$category_id");
            $_GET['id'] = $db->lastInsertId();
 
        }

Si l'insert ne fonctionne pas, 2 possibilités à vérifier
1) tu as toujours un $_GET['id'] dans l'url... donc il passe toujours en mode update...
2) Tu as dans ta table une clé d'unicité qui te bloque... donc il ne peut pas insérer un truc que tu as déjà inséré si tu l'as défini comme un doublon dont tu ne veux pas...
Si c'est cela, tu peux contourner le problème facilement :
Imaginons que tu aies une clé d'unicité sur name et slug (au pif)... ça donnerait ça :

Code :

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

1
2
3
4
5
6
7
8
  if(isset($_GET['id'])){
          $id = $db->quote($_GET['id']);
            $db->query("UPDATE works SET name=$name, slug=$slug, content=$content, category_id=$category_id WHERE id=$id");
        }else{
      		 $db->query("INSERT INTO works SET name=$name, slug=$slug, content=$content, category_id=$category_id on duplicate key update content=$content, category_id=$category_id;");
            $_GET['id'] = $db->lastInsertId();
 
        }

A part ça, tu ne dois plus utiliser comme ça tes requpete PDO, mais ainsi, avec des requêtes préparées, car ce sont des données utilisateurs.

Voir ici
PDO, une soupe et au lit
Chapitre 7 et 8 en ce qui te concerne...

Bonne chance ! Sylvie

**JovaanC** · 14/11/2018, 07h47

Bonjour Sylvie,

Je n'ai pas de GET dans l'url mais j'ai quand même l'impression que je reste du côté if...
J'ai transformé en requête préparée, j'ai essayé avec "on duplicate key update"

Toujours aucune réaction au clic sur enregistrer

**Maryline Lilly** · 14/11/2018, 09h19

Bonjour,

La syntaxe classique pour un insert en SQL est INSERT INTO table (column1, column2, ... column_n ) VALUES (expression1, expression2, ... expression_n )

Code php :

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

"INSERT INTO works (name,slug,content,category_id) values('".$name."','".$slug."','".$content."',".$category_id.")"

avec les cotes en fonction des types de colonnes bien sur

**JovaanC** · 14/11/2018, 09h53

J'ai tout bien mis, mais ça ne change rien vu que je n'entre pas dans le else...

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
       if(isset($_GET['id'])){
          $id = $_GET['id'];
           $req = $db->prepare("UPDATE works SET name=?, slug=?, content=?, category_id=? WHERE id=?");
           $req->execute(array(
               $_POST['name'],
               $_POST['slug'],
               $_POST['content'],
               $_POST['category_id'],
               $_GET['id']
           ));
            setFlash('La réalisation a bien été modifiée');
        }
        else {
              $req = $db->prepare("INSERT INTO works (name, slug, content, category_id) VALUES (?, ?, ?, ?)");
             $req->execute(array(
                    $_POST['name,'],
                    $_POST['slug'],
                    $_POST['content'],
                    $_POST['category_id']
                ));
            $_GET['id'] = $db->lastInsertId();
            setFlash('La réalisation a bien été ajoutée');
            }

**MaitrePylos** · 14/11/2018, 10h44

Envoyé par JovaanC

Bonjour Sylvie,

Je n'ai pas de GET dans l'url mais j'ai quand même l'impression que je reste du côté if...
(

et que donne

Code :

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

var_dump($_GET)

comme ça on es fixé

**JovaanC** · 14/11/2018, 11h01

Envoyé par MaitrePylos

et que donne

Code :

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

var_dump($_GET)

comme ça on es fixé

J'ai :
C:\wamp64\www\Portfolio\admin\work_edit.php:7:
array (size=0)
empty

et quand je veux modifier j'ai :
C:\wamp64\www\Portfolio\admin\work_edit.php:7:
array (size=1)
'id' => string '3' (length=1)

Donc j'ai ce qu'il faut...

**MaitrePylos** · 14/11/2018, 11h11

Ok, déboguons ligne par ligne alors :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
  if(isset($_GET['id'])){
         // $id = $db->quote($_GET['id']);
          //  $db->query("UPDATE works SET name=$name, slug=$slug, content=$content, category_id=$category_id WHERE id=$id");
            echo "isset"; die();
        }else{
      		// $db->query("INSERT INTO works SET name=$name, slug=$slug, content=$content, category_id=$category_id on duplicate key update content=$content, category_id=$category_id;");
            //$_GET['id'] = $db->lastInsertId();
 
                  echo "pas isset";
                  die();
 
        }

Dite nous si quelque soit la situation, vous avez bien le message isset qui s'affiche

Choix suivant condition if/else

Langage PHP

Vue hybride

Discussions similaires

Partager

Partager