[Résolu] Erreur de syntaxe SQL

[TiliN]

Bonjour, ça fait un moment que je ne vous ai pas rendu visite ceci pour le fait que je ne codais plus par manque de temps et d'envie je dois vous avouer

Bref j'ai un petit problème de synthaxe sql :

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 'font-size: 10px;">Edité le 28-04-08 à 20:46:47 par test.
" WHERE id = " ' at line 1

voici le message en question que j'affiche par un echo :

Non programmée pour le moment xD

développement en local jusqu'à la sortie de la v0.1b

héhé


edit


Edité le 28-04-08 à 20:46:47 par test.

et le code incriminée :

Code PHP :

<?php 

$time=time();

    $suplement = '<div style = "font-size: 10px;">Edité le '.date('d-m-y \à H\:i\:s\ ', $time).' par '.$_SESSION['pseudo'].'.</div>'; 

    echo $suplement;

    $message= ''.$_POST['new'].'<br /><br /><br />'.$suplement.'';

    $message = nl2br($message);

    echo '<br />'.$message.'';

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

    query('UPDATE news SET objet = "'.$sujet.'", message = "'.$message.'" WHERE id = "'.$_POST['id'].'" LIMIT 1') or die (mysql_error());

    echo 'News corrigée.<br />';

    

    echo $message ; 

[Ruz]

le " du style sur le div... je parierais ^^

$time=time();
$suplement = '<div style = "font-size: 10px;">Edité le '.date('d-m-y \à H\:i\\ ', $time).' par '.$_SESSION['pseudo'].'.</div>';
echo $suplement;
$message= ''.$_POST['new'].'<br /><br /><br />'.$suplement.'';
$message = nl2br($message);
echo '<br />'.$message.'';
$sujet = mysql_real_escape_string($_POST['sujet']);
query('UPDATE news SET objet = \''.$sujet.'\', message = \''.$message.'\' WHERE id = \''.$_POST['id'].'\' LIMIT 1') or die (mysql_error());
echo 'News corrigée.<br />';

echo $message ;

ca irait pas mieux?

remarque perso: les balises HTML n'ont rien à faire dans ta base de données.
1. ca prend de la place pour rien
2. c'est source d'erreurs
=> prévois deux champs supplémentaires pour ces infos (genre edition_pseudo et edition_time) si t'y tiens vraiment, ou n'importe quel autre système...

et supplement prends 2 "p" ^^

[TiliN]

je prend bonne note de tes remarques

cependant snouf :

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 'à la sortie de la v0.1b

héhé


edit
<' at line 1

[TiliN]

et ce que je ne comprend pas c'est que si je modifie

Code PHP :

<?php 

$message = nl2br($message); 

[TiliN]

[Ruz]

$time=time();
$upmessage= ''.$_POST['new'].'<br /><br /><br /><div style = "font-size: 10px;">Edité le '.date('d-m-y \à H\:i\\ ', $time).' par '.$_SESSION['pseudo'].'.</div>';
$upmessage = nl2br($upmessage);
echo $upmessage ;
$sujet = $_POST['sujet'];
$queryrelatenews = 'UPDATE news SET objet = "'.$sujet.'", message = \''.$upmessage.'\' WHERE id = "'.$_POST['id'].'" LIMIT 1';
query($queryrelatenews) or die (mysql_error());
echo 'News corrigée.<br /><br /><br />';

je le fais comme si c'était pour moi:

$time=time();
$upmessage= $_POST['new'].'<br /><br /><br /><div style = "font-size: 10px;">Edité le '.date('d-m-y \à H\:i\\ ', $time).' par '.$_SESSION['pseudo'].'.</div>';
echo nl2br($upmessage) ;
$sujet = $_POST['sujet'];
$queryrelatenews = 'UPDATE `news` SET `objet` = \''.addslashes($sujet).'\', `message` = \''.addslashes($upmessage).'\' WHERE `id` = \''.$_POST['id'].'\' LIMIT 1';
query($queryrelatenews) or die ('<p class="error">[erreur] : '.mysql_error().'<br />Requete : '.$queryrelatenews.'</p>');
echo '<p>News corrigée.</p>';

essaye avec ca, juste pour voir ^^

[TiliN]

idem, rien n'est modifié .... :/
je ne comprend pas

edit j'ajoute la structure de la table :

Code PHP :

<?php 

--

-- Structure de la table `news`

--



CREATE TABLE `news` (

  `id` int(11) NOT NULL auto_increment,

  `pseudoe` varchar(20) NOT NULL default '',

  `ide` int(11) NOT NULL default '0',

  `objet` text NOT NULL,

  `message` text NOT NULL,

  `time` int(11) NOT NULL default '0',

  PRIMARY KEY  (`id`)

) ENGINE=MyISAM  DEFAULT CHARSET=latin1 AUTO_INCREMENT=18 ;



--

-- Contenu de la table `news`

--



INSERT INTO `news` (`id`, `pseudoe`, `ide`, `objet`, `message`, `time`) VALUES

(17, 'test', 1, 'OUVERTURE !!!', 'Ouverture du jeu non prévu, cependant, il sera développé et testé complètement en local.<br />\r\nSur ce ....', 1209412190); 

[Ruz]

rajoute un peu ca à la fin:

echo '<p>'.$queryrelatenews.'</p>';

qu'on voit la requete ^^

ou mieux:

$time=time();
$upmessage= $_POST['new'].'<br /><br /><br /><div style = "font-size: 10px;">Edité le '.date('d-m-y \à H\:i\\ ', $time).' par '.$_SESSION['pseudo'].'.</div>';
echo nl2br($upmessage) ;
$sujet = $_POST['sujet'];
$queryrelatenews = 'UPDATE `news` SET `objet` = \''.addslashes($sujet).'\', `message` = \''.addslashes($upmessage).'\' WHERE `id` = \''.$_POST['id'].'\' LIMIT 1';
query($queryrelatenews) or die ('<p class="error">[erreur] : '.mysql_error().'<br />Requete : '.$queryrelatenews.'</p>');
if(mysql_affected_rows()==1){echo '<p>News corrigée.</p>';}
else{echo '<p>'.$queryrelatenews.'</p>'; }

[Belsion]

Quand je vois les erreurs et ton code, il y a plusieurs choses qui me choquent:

1° faire un échappement des apostrophes et guillemets à la fois dans ton objet et dans ton message (cf les addslashes de Ruz, même si je préfères htmlspecialchars(), mais c'est parce que je ne stocke pas de code html)

2° dans ton premier message d'erreur, il y avait " WHERE id='' ". Est-ce que dans $_POST['id'] tu renvoies bien l'id de ta news à modifié? A priori non.

[Ruz]

le $_POST["id"] existe bien (input hidden), maintenant, est-ce le bon ID???

sinon, l'échappement des apostrophes est tout à fait l'inverse de mon habitude, d'où mes tatonnements ^^

M'enfin, si il me poste la requete, on verra deja bcp mieux ^^