Discussion :

[DrVanhee]

Bonjour,

Pour migrer sur OVH, je passe en 7,2

J'ai un énorme problème avec mes scripts de formulaires.

Avant tout se passait sans problème. (magic quote)

Maintenant, il faut récupérer les données en $_POST dans lesquelles j'ai repris les variables d'origine
comme exemple:
$autres = $_POST[autres];
$horaires = $_POST[horaires];
$info = $_POST[info];

Puis j'ai changé le langage mysql vers mysqli

La la fonction UPDATE ne passe plus pour du texte avec des ' ou des ;

Le problème est que la variable du texte est intégrée en texte dans les données Mysql, et interprété comme une requète;

le ' finit prématurément la zone de texte
et le ; finit la requète

Bref, ma question est comment faites vous pour rentrer du texte maintenant.
Quel est le phrasé en PHP 7
J'ai fouillé partout, il y a un truc qui m'échappe.


$sql="UPDATE calendrier SET ...
autres = '$autres',
horaires = '$horaires',
info = '$info',
...
WHERE id = '$id' ;";

http://rongeur.com/calendrier/formulaire.php

Merci

[moimp]

Sans être vraiment spécialiste, tu as un risque d'injection SQL. Il n'est jamais bon d'insérer directement des variables dans une requête. Tu devrais utiliser des requêtes préparées et non pas mysqli mais plutôt la classe PDO.
Je te conseille aussi cet article.

[Celira]

PDO, c'est bien, mais quand il s'agit de migrer un existant PHP <5.5, je comprends qu'on puisse préférer mysqli pour rester avec un code similaire (et procédural)

Cela dit, si tu ne nous donnes pas de code, on ne va pas pouvoir faire de miracles. (et ton lien ne passe pas mon proxy ) Mais basiquement, il te manque les échappements qui étaient fait automatiquement par les magic_quotes (et mal faits, ce qui est la raison première de la disparition de la fonctionnalité )

Il faut maintenant que tu appliques mysqli_real_​escape_​string à tes valeurs avant de les injecter dans ta requête

Code :

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

1
2
3
4
5
6
$sql="UPDATE calendrier SET ...
autres = '".mysqli_real_escape_string($linkDb, $autres)."',
horaires = '".mysqli_real_escape_string($linkDb, $horaires)."',
info = '".mysqli_real_escape_string($linkDb, $info)."',
...
WHERE id = '".mysqli_real_escape_string($linkDb, $id)."' ;

[DrVanhee]

PDO, c'est bien, mais quand il s'agit de migrer un existant PHP <5.5, je comprends qu'on puisse préférer mysqli pour rester avec un code similaire (et procédural)

Merci,

ça marche... c'est super !

Oui, effectivement j'ai assez de taf, suite au passage du html à l'ancienne en tableau à du css3
puis upgrader php 5.3 vers 7.2

Je vais garder dans un premier temps le mysql, on changera plus tard...

De plus certains modules comme mon annuaire est totalement impossible à upgrader.

Et puis j'en ai des pages sur http://rongeurs.net