[Résolu] Changement d'alliance

[alechuga]

Salut c'est moi, NON NE PARTEZ PAS :triste:
Je suis entrain de faire mon script d'alliance mais lorsque quelqu'un veut s'inscrire dans une alliance le script plante avec cette merveilleuse erreur

Code PHP :

<?php 

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE ID='61')' at line 1 

[Valter]

Je suis pas sur, mais essaie :
1)

Code PHP :

<?php 

mysql_query('UPDATE alliance SET nombres = nombres + 1 WHERE NOM="'$nom'"') or die (mysql_error()); 

[Raoull]

Si $_SESSION[id] est un nombre, inutile de l'entourer de guillement dans ta requete SQL.
Mais c'est pas ça qui fait planter la requete. T'es sur te ta structure de bdd ?

PS : Tu récupères tes valeurs POST direct et tu les balances dans tes requetes sql, sans aucune verif, c'est très dangereux...

edit : pkoi c'est pas $_SESSION['id'] (avec les guillmets, id n'est pas une constante je pense)

[alechuga]

C'est ma faute j'ai pas été assez clair, la seule chose qui plante c'est la dernière ligne. les modifs sont faites en BDD pour toutes les autres mais pas celle-là.

Sinon Raoull tu peux m'expliquer le truc des guillemets ? Pour ce qui est de la sécurité je doute que beaucoup de monde s'intéressent à mon site.

[Eluox]

Voila ce que je ferais suivis de quelques conseil/expliquation

Code PHP :

<?php

session_start();

include("obligatoire.php");



$nom = htmlentities($_POST['nom'], ENT_QUOTES));

$descrit = htmlentities( $_POST['descrit'], ENT_QUOTES));



doquery("INSERT INTO alliance SET `nom` = {$nom}, `texte` = {$descrit}");

doquery("UPDATE alliance SET `nombres` = nombres+1 WHERE NOM={$nom}");

doquery("INSERT INTO membres SET `alliance` = {$nom} WHERE id= {$_SESSION['id']}");

header("Location: alliance.php");

?>

[Aro]

@Raoull: Je pense pas que l'absence d'apostrophes dans son tableau de session change quelque chose puisque dans le message d'erreur, il affiche bien "61"... Mais ce serait un truc à tenter

@alechuga:

Code PHP :

<?php 

mysql_query("INSERT INTO alliance (nom, nombres, texte) VALUES('".$nom."', '', '".$descrit."')") or die (mysql_error()); 

[Anthor]

Mais ca choque personne un INSERT TO couplé avec un WHERE ???

Soit tu UPDATE la ligne correspondant avec l'id soit tu utilise un REPLACE TO si tu n'es pas sur que la ligne existe.

[omsi668]

$nom = mysql_real_escape_string($_POST['nom']);


Peut être ??

[Raoull]

'ffectivement !
pas vu le insert/where, tu as l'oeil de taupe Anthor

[alechuga]

Dsl mais aucune de vos propositions ne marchent. Toujours la même erreur, elle ne se marque que si je demande un mysql_error sinon la requête n'est juste pas exécuter.