Discussion :

[Anduriel]

Salut,

Je récupère dans une table une liste de chaines et je les place dans des options:

Code :

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

$cats .= "<option value='".htmlspecialchars($ones['text'])."'>".htmlspecialchars($ones['text'])."</option>";

Si je veux supprimer une chaine, je supprime grace à ce nom dans la clause WHERE (pas d'id):

Code :

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

$connection->query("DELETE FROM its_cat WHERE text = ".quote_smart($cat));

Mais dans le cas ou ma chaine est C'est beau, rien n'est supprimé. Après avoir fait un echo juste avant la requête de suppression j'ai C comme value de l'option (mais bien C'est beau à l'affichage de l'option).
Je ne comprends pas puisque je traite la chaine comme il faut avant non?

Comment faire?
Merci

[bkill]

Salut,

pour la fonction htmlspecialchars, il faut que tu lui spécifies que tu veux encoder les single quotes, avec le param ENT_QUOTES.

Si tu fais un view source de ta page où se trouve la select, à mon avis tu dois voir qqch du genre
<option value='C'est beau'>C'est beau</option>

[Kerod]

moi tout simple je modifierai le code en mettant comme ceci :

Code :

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

"<option value=\"".$valeur."\">$valeur</option>";

ou directement :

Code :

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

'<option value="'.$valeur.'">$valeur</option>'

[bkill]

Je modifierais aussi le code, pour etre W3C compliant, car les paramètres d'un attribut doivent être entourés de guillemets et non quotes.

Mais malgré cette modif je garderais tout de même un htmlentities() ou un htmlspecialchars(), histoire de ne pas être ennuyé avec les guillemets, suite à cette modif.

Et aussi, cette modif n'est pas valable:

Code :

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

1
2
 
'<option value="'.$valeur.'">$valeur</option>'

elle va afficher $valeur comme value de l'option

il faudra donc:

Code :

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

1
2
 
'<option value="'.$valeur.'">'.$valeur.'</option>'

[Kerod]

oui bien sur c'était sous entendu chef car le plus important est à la base le value.

Mais je m'excuse du manque de quote

[bkill]

Bon, allez, ca passe pour une fois