Discussion :

[paintbox]

Bonjour,



une fois de plus, je fais appel à votre aide.

Voici le problème que je rencontre. J'ai 2 pages,la première est un simple formulaire à 1 champs qui permet de faire une recherche sur un numéro d'article. Sur cette même page apparaît le résultat. De cette page, on peut cliquer sur le numéro d'article pour éditer sa fiche. La seconde page est donc la fiche permettant de modifier les infos sur l'article. Le numéro d'article ($id_biens) entre la page 1° et la 2° se fait par un GET.

Chose étrange c'est que au début de mon code je récupère $id_biens sans problème mais lorsque je fais à nouveau appel à cette variable notamment pour ma requete sql, plus moyen d'avoir sa valeur ! Et donc,mes champs ne se préremplissent plus.

J'ai donc pensée à un problème de porté de variable, mais il ne me semble pas que ce soit le cas dans mon code. Je ne vois donc vraiment pas.



Peut-être l'un de vous aura une idée.



D'avance merci, voici mon code

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
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
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
 
 
 $id_biens=$_GET['id_biens'];//Récupération de l'id du bien à modifier
 
echo '<DIV class="container">';
 
echo '<p>Modification de la fiche du bien : '.$_GET['id_biens'].'</p>';
 
echo '<hr />';
 
/*DEBUT TRAITEMENT MODIFICATION BIEN*/
 
/*---------------------------------*/
 
if(isset($_GET['modifier_bien'])) //Si on demande de modifier un bien
 
{
 
//On protège la variable modifier_news pour éviter une faille SQL
 
$_GET['modifier_bien']=mysql_real_escape_string(htmlspecialchars($_GET['modifier_bien']));
 
 
//On récupère les infos de la news correspondante
 
$retour=mysql_query('SELECT * FROM biens INNER JOIN communes ON biens.id_commune=communes.id_commune WHERE id_biens=\''.$_GET['id_biens'].'\'');
 
$donnees=mysql_fetch_array($retour);
 
//Récupération des données
 
$bien=htmlentities($donnees['code_bien'], ENT_QUOTES,'UTF-8');
 
$statut=htmlentities($donnees['code_statut'], ENT_QUOTES,'UTF-8');
 
$adresse=htmlentities($donnees['adresse'], ENT_QUOTES,'UTF-8');
 
$commune=htmlentities($donnees['commune'], ENT_QUOTES,'UTF-8');
 
$pays=htmlentities($donnees['pays'], ENT_QUOTES,'UTF-8');
 
$prix=htmlentities($donnees['prix'], ENT_QUOTES,'UTF-8');
 
$description=htmlentities($donnees['description'], ENT_QUOTES,'UTF-8');
 
$id_biens=$donnees['id_biens']; //Cette variable va servir pour se souvenir que c'est une modification
 
}
 
else //C'est qu'on rédige une nouvelle news
 
{
 
//Les variables sont vides, puisque c'est un nouveau bien
 
$bien='';
 
$statut='';
 
$adresse='';
 
$commune='';
 
$pays='';
 
$prix='';
 
$descrition='';
 
$option[]='';
 
$id_biens=0;
 
}
 
 
 
/*FIN TRAITEMENT MODIFICATION BIEN*/
 
/*------------------------------*/
 
 
$request_commune=mysql_query('SELECT commune, id_commune, code_postal FROM communes');//Requete pour la liste des communes
 
 
 
/*DEBUT FORMULAIRE AJOUT DE BIENS*/
 
/*------------------------------*/
 
echo '<form action="gestion.php" method="POST" enctype="multipart/form-data">';
 
echo '<fieldset>';
 
echo '<legend>Vous êtes sur le point de modifier le bien : '.$id_biens.'</legend>';
 
echo '<br />';
 
echo '<p>Sélectionnez le type de bien</p>';
 
echo '<br />';
 
 
 
//Affichage type bien
 
if($donnees['code_bien']==10)
 
{
 
echo '<input type="radio" name="bien" value="Maison" checked="checked" ><label>Maison</label>';
 
echo '<input type="radio" name="bien" value="Appartement" ><label>Appartement</label>';
 
echo '<input type="radio" name="bien" value="Studio"><label>Studio</label>';
 
echo '<input type="radio" name="bien" value="Magasin"><label>Magasin</label>';
 
echo '<input type="radio" name="bien" value="Bureau"><label>Bureau</label>';
 
}
 
 
 
if($donnees['code_bien']==20)
 
{
 
echo '<input type="radio" name="bien" value="Maison" ><label>Maison</label>';
 
echo '<input type="radio" name="bien" value="Appartement" checked="checked" ><label>Appartement</label>';
 
echo '<input type="radio" name="bien" value="Studio"><label>Studio</label>';
 
echo '<input type="radio" name="bien" value="Magasin"><label>Magasin</label>';
 
echo '<input type="radio" name="bien" value="Bureau"><label>Bureau</label>';
 
}
 
if($donnees['code_bien']==30)
 
{
 
echo '<input type="radio" name="bien" value="Maison" ><label>Maison</label>';
 
echo '<input type="radio" name="bien" value="Appartement" ><label>Appartement</label>';
 
echo '<input type="radio" name="bien" value="Studio" checked="checked" ><label>Studio</label>';
 
echo '<input type="radio" name="bien" value="Magasin"><label>Magasin</label>';
 
echo '<input type="radio" name="bien" value="Bureau"><label>Bureau</label>';
 
}
 
if($donnees['code_bien']==40)
 
{
 
echo '<input type="radio" name="bien" value="Maison" ><label>Maison</label>';
 
echo '<input type="radio" name="bien" value="Appartement" ><label>Appartement</label>';
 
echo '<input type="radio" name="bien" value="Studio" ><label>Studio</label>';
 
echo '<input type="radio" name="bien" value="Magasin" checked="checked" ><label>Magasin</label>';
 
echo '<input type="radio" name="bien" value="Bureau"><label>Bureau</label>';
 
}
 
 
if($donnees['code_bien']==50)
 
{
 
echo '<input type="radio" name="bien" value="Maison" ><label>Maison</label>';
 
echo '<input type="radio" name="bien" value="Appartement" ><label>Appartement</label>';
 
echo '<input type="radio" name="bien" value="Studio" ><label>Studio</label>';
 
echo '<input type="radio" name="bien" value="Magasin" ><label>Magasin</label>';
 
echo '<input type="radio" name="bien" value="Bureau" checked="checked" ><label>Bureau</label>';
 
}
 
//FIN Affichage type bien
 
 
echo '<br />';
 
echo '<br />';
 
 
 
//Affichage Statut du bien
 
echo '<p>Sélectionnez le statut du bien</p>';
 
echo '<br />';
 
if($donnees['code_statut']==10)
 
{
 
echo '<input type="radio" name="statut" value="vendre" id="10" checked="checked" /><label>A vendre</label>';
 
echo '<input type="radio" name="statut" value="louer" id="20" /><label>A louer</label>';
 
}
 
if($donnees['code_statut']==20)
 
{
 
echo '<input type="radio" name="statut" value="vendre" id="10"  /><label>A vendre</label>';
 
echo '<input type="radio" name="statut" value="louer" id="20" checked="checked"/><label>A louer</label>';
 
}
 
//FIN Affichage Statut du bien
 
echo '<br />';
 
echo '<br />';
 
 
echo '<label>Adresse du bien : </label><input type="text" name="adresse" value="'.htmlentities($donnees['adresse']).'" size="75" />';
 
echo '<br />';
 
echo '<label>Sélectionnez une commune : '.$commune.'</label>';
 
echo '<SELECT name="commune" size="1" value='.$commune.'>';
 
echo '<OPTION>'.$commune.'</OPTION>';//Permet d'avoir un premier champs vide
 
while($donnees_commune=mysql_fetch_array($request_commune))
 
{
 
echo '<OPTION name="commune" value="'.$donnees_commune['commune'].'">'.$donnees_commune['commune'].'</OPTION>';
 
}
 
echo '</SELECT>';
 
echo '<label>Pays : </label>';
 
echo '<SELECT name="pays">';
 
echo '<OPTION>'.$pays.'</OPTION>';
 
echo '<OPTION name"Belgique">Belgique</OPTION>';
 
echo '<OPTION name"France">France</OPTION>';
 
echo '<OPTION name"Luxembourg">Luxembourg</OPTION>';
 
echo '<OPTION name"Pays-Bas">Pays-Bas</OPTION>';
 
echo '</SELECT>';
 
echo '<br />';
 
echo '<label>Prix : </label><input type="text" name="prix" size="30" value="'.$prix.'"/> Euros';
 
echo '<br />';
 
echo '<br />';
 
 
 
 
 
//Affichage des options
 
echo '<td>Garage : ';
 
if ($donnees['garage']==1)
 
{
 
echo '<input type="checkbox" name="option[]" value="garage" checked="true" >';	 }
 
else
 
{
 
echo '<input type="checkbox" name="option[]" value="garage" >';
 
}
 
echo '<td>';
 
echo '</tr>';
 
echo '<tr>';
 
echo '<td>Jardin : ';
 
 
 
if ($donnees['jardin']==1)
 
{
 
echo '<input type="checkbox" name="option[]" value="jardin" checked="true" >';
 
}
 
else
 
{
 
echo '<input type="checkbox" name="option[]" value="jardin" >'; 
 
}
 
echo '<td>';
 
echo '</tr>';
 
echo '<tr>';
 
echo '<td>Grenier : ';
 
 
 
if ($donnees['grenier']==1)
 
{
 
echo '<input type="checkbox" name="option[]" value="grenier" checked="true" >';
 
}
 
else
 
{
 
echo '<input type="checkbox" name="option[]" value="grenier">';
 
}
 
echo '<td>';
 
echo '</tr>';
 
echo '<tr>';
 
echo '<td>Cave : ';
 
 
 
if ($donnees['cave']==1)
 
{
 
echo '<input type="checkbox" name="option[]" value="cave" checked="true" >';
 
}
 
else
 
{
 
echo '<input type="checkbox" name="option[]" value="cave" >';
 
}
 
//FIN Affichage des options
 
echo '<br />';
 
echo '<br />';
 
 
 
//Affichage des images du bien
 
echo '<p>Images enregistrées pour ce bien :</p>';
 
echo '<br />';
 
 
 
echo vignettes($id_biens); //Appel de la fonction pour afficher les vignettes des images
 
echo '<br />';
 
 
 
 
 
//Description du bien
 
echo '<label>Description du bien : </label><br /><textarea name="description" rows="20" cols="80">'.htmlentities($donnees['description']).'</textarea>';
 
echo '<br />';
 
echo '<br />';
 
echo '<input type="hidden" name="id_biens" value="'.$id_biens.'" />';
 
echo '<input type="submit" name="modifier_bien" value="Modifier" id="'.$id_biens.'"/></a><input type="reset" value="Effacer"/>';
 
echo '</fieldset>';
 
echo '</form>';

[Vil'Coyote]

pour des raison pratique il serait un peu mieux d'utiliser ton "$id_biens" plutôt que ton "$_GET"

et petite question "gestion.php" c'est ta deuxième page de traitement?

[RunCodePhp]

Salut

Cette variable $id_bien est initialisée à plusieurs endroit dans ton code, et selon un cas elle est même à 0 : $id_biens=0;

Cette donnée devrait théoriquement avoir 2 valeur : Soit 0 (par défaut) car rien a été obtenu en GET, soit la valeur reçue en GET.

Donc ceci peu ce faire en 1 coup d'un seul, dès le départ :

Code :

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

1
2
 
$id_biens = (isset($_GET['id_biens']) && (int)$_GET['id_biens'] > 0) ? : 0;

Code :

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

$_GET['modifier_bien']=mysql_real_escape_string(htmlspecialchars($_GET['modifier_bien']));

Ceci est aussi à éviter, car modifier une valeur de $_GET n'est pas très zen.
Vaux mieux le mettre en 1er dans une variable ($id_bien) et ensuite la traiter si nécessaire.

Aussi, la fonction mysql_real_escape_string() est faite pour une donnée destinée à être insérée dans une BDD, donc vaut mieux l'appliquer juste à ce moment là.
Par contre, la fonction htmlspecialchars() est faite pour un affichage dans une page Web (HTML), donc pas pour insérer dans une Bdd.

Mais encore, et vu son nom (id_bien), je suppose que sa valeur doit être un identifiant (un entier), ça doit vraiment être le cas. Alors il y a un truc hyper simple dans ce cas là, c'est le "typage" (ou cast) :

Code :

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

1
2
3
$id_bien = (int)$_GET['id_bien'];
// Ou alors (c'est pareil)
$id_bien = intval($_GET['id_bien']);

Pour ce type de donnée (des entiers), c'est à mon sens le remède le plus imparable contre tout acte de piratage car dans les pires des cas ça sera 0 comme valeur.


Mise à part ça, une variable est accessible dans toute la page, sauf dans les fonctions ou classes (à moins de déclarer une portée plus global dans la fonction ou classe pour cette variable).

Quand tu ne sais pas ce qu'il se passe concernant une variable ou autre tableau, fait des echo ou print_r() sur celle ci, et cela à plusieurs endroits de ton script, ceci permet de constater visuellement sa valeur, on peu ensuite en déduire certaine choses.