Discussion :

[dominos]

Bonjour,

Suite à une migration de PHP 5.6 vers PHP 7.2, la requête suivante ne fonctionne plus, impossible d'effectuer un Insert, j'ai regardé la documentation, tout semble correspondre, je ne comprends pas..., ça fait 2 jours que je suis entre autres choses dessus, j'ai aussi essayé d'isoler l'Insert et de passer tous les champs à "NULL", ça n'a rien donné... mais je persiste à penser que ça vient de la requête.

Code :

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

1
2
3
4
5
# Protection injections SQL
$cmt_pseudo = mysqli_real_escape_string($class->db_connect, $_POST['pseudo']);
$cmt_commentaire = mysqli_real_escape_string($class->db_connect, $_POST['commentaire']);
mysqli_query($class->db_connect, "INSERT INTO commentaires (cmt_id, cmt_reference, cmt_date, cmt_pseudo, cmt_commentaire, cmt_validate) values ('', '$article_reference', '$date_time', '$cmt_pseudo', '$cmt_commentaire', '0')");
		$last_id = mysqli_insert_id($class->db_connect); # Récupère le dernier id

Auriez-vous s'il vous plait une idée du problème, mes tests de migration sont effectués en amont en localhost avec : Wamp 3.1.4/64bits, PHP 7.2, MySQL 5.7.23

Merci d'avance pour vos réponses

[CinePhil]

Je suppose que cmt_id est un identifiant de type entier auto-incrémenté ?
Alors inutile de le mettre dans la requête et, surtout, lui donner en valeur une chaîne de caractères, fut-elle vide, est pour le moins... bizarre... pour rester poli !

À part cette bizarrerie, qu'obtenez-vous comme message d'erreur ?

[al1_24]

MySQL retourne certainement un message d'erreur qui pourrait te permettre d'identifier l'origine du problème.
Tu pourrais aussi demander l'affichage de la requête telle qu'elle est exécutée. Cela pourrait aussi t'aider ans ta recherche.

[dominos]

Merci pour vos réponses rapides,

@CinePhil : Comme vous l'expliquez très bien, l'erreur venait effectivement de l'identifiant auto-incrémenté qui était déclaré et que j'ai donc supprimé. Au passage, je vous remercie pour cette correction de syntaxe que je traine surement depuis des années, PHP 7.2 semble apparemment syntaxiquement plus rigoureux.

[CinePhil]

PHP 7.2 semble apparemment syntaxiquement plus rigoureux.

En l'occurrence, il ne s'agissait pas d'une plus grande rigueur de PHP mais de MySQL. Il y a un paramètre pour demander à MySQL d'être moins rigoureux mais je ne sais plus lequel et il vaut mieux être rigoureux si on veut conserver une BDD cohérente.

[Artemus24]

Salut dominos.

1) il n'est pas nécessaire de faire référence à votre colonne "cmt_id", dans l'insert, si celle-ci est du type auto incrémenté.
Laissez MySql se débrouiller tout seul !

2) je remarque que vous mettez partout des apostrophes.
Si vous avez une chaînes de caractères, de type littérale, (je parle de ceci 'blablabla') cela est nécessaire.
Dans le cas contraire, il est nécessaire d'encadrer votre chaîne par une apostrophe.
Sous mysql, c'est l'apostrophe et sous php, c'est le guillemet. A ne pas confondre !

Je constate que vous mettez aussi entre apostrophe sur le zéro.
Est-ce que le type de la colonne cmt_validate est integer ou char() ?
Si c'est de type numérique, ce n'est pas nécessaire.

3) les concaténations des chaines se font à l'aide d'un point en php.

4) ensuite la syntaxe de votre requête n'est pas, a priori, correcte :

Code :

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

1
2
3
4
$req = "INSERT INTO `commentaires` (`cmt_reference`,`cmt_date`,`cmt_pseudo`,`cmt_commentaire`,`cmt_validate`) values
 ('".$article_reference."','".$date_time."','".$cmt_pseudo."','".$cmt_commentaire."',0)";
 
mysqli_query($class->db_connect, $req);

5) au lieu d'utiliser l'extension "mysqli", vous feriez mieux de passer à "php pdo".
La gestion en serait plus simple à mettre en oeuvre !

@+