Discussion :

[Lekno]

Bonjour à tous, j'ai un petit soucis lors de l'insertion de données dans ma base MYSQL

J'ai ce code qui récupere mes données de formulaires et doit les inserer dans ma base.

Mon soucis est que je n'ai aucun message d'erreure, mais que mes données ne sont pas modifier. :s

J'ai essayé de faire un echo $upgrade; pour tester ma requete directement dans phpmyadmin, elle fonctionne.

Je ne comprend donc pas d'ou peut venir mon erreur !

Merci pour votre petit coup de main.

Cordialement

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?php
 
$id = htmlentities(mysql_escape_string($_POST['id']));
$titre = htmlentities(mysql_escape_string($_POST['titre']));
$content = htmlentities(mysql_escape_string($_POST['content']));
$date = htmlentities(mysql_escape_string($_POST['date']));
 
$update = 'UPDATE articles SET  id  ="' . $id . '", 
    titre ="' . $titre . '", content ="' . $content . '", date="' . $date . '"
        WHERE id=' . $id; 
mysql_query($update);
header('Location: index.php');
?>

[CinePhil]

Si id est un entier, inutile de l'entourer d'apostrophes dans la reqête et il serait plus judicieux d'utiliser intval($_POST['id']).

Et d'ailleurs, pourquoi mettre à jour id puisque c'est la ligne recherchée par le WHERE.

Quant à mysql_escape_string, c'est une fonction obsolète ; il vaut mieux utiliser mysql_real_escape_string.

Enfin, les valeurs textuelles s'entourent de guillemets, pas d'apostrophes. Je crois que MySQL accepte les deux mais j'aurais plutôt écrit ce code ainsi :

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
<?php
 
$id = htmlentities(intval($_POST['id']));
$titre = htmlentities(mysql_real_escape_string($_POST['titre']));
$content = htmlentities(mysql_real_escape_string($_POST['content']));
$date = htmlentities(mysql_real_escape_string($_POST['date']));
 
$update = "
	UPDATE articles 
	SET  titre = '".$titre."', 
		content = '".$content."', 
		date = '".$date."'
	WHERE id = ".$id
; 
mysql_query($update);
header('Location: index.php');
?>

Pour le reste, la requête semble correcte

[Lekno]

J'ai bien pris note de tes divers remarques.
Apres test, je n'ai toujours pas d'update effectif...la requete semble bien passer, mais l'UPDATE ne met pas à jour le contenu

[Celira]

Le plus simple, c'est sans doute de désactiver temporairement la redirection et d'afficher l'éventuelle erreur (sans oublier de revenir à la version normale lorsque la requête fonctionnera )

Code :

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

1
2
3
4
5
$result = mysql_query($update); 
if ($result === false) {
  die ('Erreur sur : '.$update.'<br/>'.mysql_error());
}
//header('Location: index.php');

[Lekno]

Je te remercie c'est bien le MYSQL_error qu'il me fallait ! Avec une jolie "No database selected"


Erreur stupide merci à toi