Fonction UPDATE PHP+SQL

Version imprimable

:resolu:Bonjour à tous, je tente désepserement de faire fonctionner ma fonction d'update SQL :lol:

Ma requête est validée par phpmyadmin sans aucun soucis mais lors de l'execution de la fonction l'update n'est pas effectif 8O

J'appel :

Code:

updateEntreprise($updatenom, $updatelien, $updatemail, $updateinfo, $updatestatus, $updatedep);

La fonction :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 function updateEntreprise($updatenom, $updatelien, $updatemail, $updateinfo, $updatestatus, $updatedep) { $textSQL2 = "UPDATE entreprise SET nom = '" . $updatenom . "', lien = '" . $updatelien . "', mail = '" . $updatemail . "', info = '" . $updateinfo . "', status = '" . $updatestatus . "', departement = '" . $updatedep . "' WHERE id =$updateid "; return mysql_query($textSQL2); }
Merci pour votre aide :ccool:

02/05/2012, 09h11
Fench

Bonjour,

Et $updateid vient d'ou ????

(En plus tu as commencé avec de la concaténation de variable et rien pour cette partie de la requête)

En effet, il manque $updateid dans le libellé de la fonction.
Code:

1 2 3 4 5 6 7 function updateEntreprise($updateid, $updatenom, $updatelien, $updatemail, $updateinfo, $updatestatus, $updatedep) { $textSQL2 = "UPDATE entreprise SET // [...] WHERE id = '" . $updateid . "' "; return mysql_query($textSQL2); }
AS-tu aussi pensé à la PROTECTION contre les injections SQL ?

Bonjour,

J'effectu ceci avant un traitement dans la fonction

Code:

1
2
3
4
5
if (!isset($_POST['updatelien'])) {
    $updatelien = '';
} else {
    $updatelien = mysql_real_escape_string ($_POST['updatelien']);
}

J'ai bien vérifie via un echo que ma requête était bonne.

Mon nouvel appel de fonction :

Code:

1
2
 
updateEntreprise($updateid, $updatenom, $updatelien, $updatemail, $updateinfo, $updatestatus, $updatedep);

Mon traitement

Code:

1
2
3
4
5
6
7
8
9
10
11
function updateEntreprise($updateid, $updatenom, $updatelien, $updatemail, $updateinfo, $updatestatus, $updatedep) {
    $textSQL2 = "UPDATE entreprise SET
id = '".$updateid."',
nom = '" . $updatenom . "',
lien = '" . $updatelien . "',
mail = '" . $updatemail . "',
info = '" . $updateinfo . "',
status = '" . $updatestatus . "',
departement = '" . $updatedep . "' WHERE id ='" .$updateid. "' 
";
    return mysql_query($textSQL2);

Je n'ai helas toujours aucun update avec la fonction :?

03/05/2012, 08h51
RunCodePhp

Salut

Citation:

J'effectu ceci avant un traitement dans la fonction

Ce n'est théoriquement pas l'endroit le plus opportun.
L'appliquer dans la fonction serait (à mon sens) mieux, ceci pour 2 raisons.

1/ Si cette mise à jour se fait plusieurs fois (l'appel à cette fonction updateEntreprise) dans d'autres pages ou autre contexte (peu importe), tu vas inévitablement devoir répéter autant de fois ce code dans les traitements qu'il y aura d'appel à cette fonction.
Donc tu vas avoir du code répétitif.
A l'inverse, en les mettant dans la fonction tu n'auras plus ces codes répétitifs, mais surtout, tu vas te garantir que tous les appels à mysql_escape_string() seront effectués (moins risque d'oubli).
En somme il est en bonne partie là l'intérêt d'une fonction.

2/ Admettons que cette variable $updatelien sera non seulement exploitée dans la fonction updateEntreprise() mais aussi exploitée dans la page HTML pour être affichée (pour exemple).
Et bien si cette donnée contient des quotes (genre : c'est l'été), elles seront échappées (le but de mysql_real_escape_string), et par conséquent des caractères d'échappements seront visibles dans la page HTML.
Et bien le piège c'est se dire qu'il faudra alors les supprimer à grand renfort de striplashes(). :aie:
Le piège quoi.

Bref ... mysql_real_escape_string() est une fonction propre et dédiée à mysql, son usage doit être fait uniquement sur des données destinées à être exploitées dans une requête faite avec mysql_query().

C'était juste une (petite) parenthèse. ;)

Citation:

Je n'ai helas toujours aucun update avec la fonction

Prends tu bien soin de modifier les données ? (au moins une)
Est-ce que la valeur de $updateid correspond bien à une des données dans la Bdd ?

As tu pris soin d'afficher la chaine de requête (un echo sur $textSQL2) afin de vérifier que tout est cohérent/conforme ?

Ca se peu que la requête échoue (une erreur), et dans tel cas il est bon de savoir la nature de l'erreur éventuelle.
Pour cela :
- soit tu active l'affichage des erreurs de MySQL (mysql.trace_mode à On dans le php.ini ou faire un ini_set)
- Ou alors rajouter un : mysql_query($textSQL2) or die('ERREUR : '.mysql_error());
03/05/2012, 10h15
Invité

Pour l'update :

Citation:

$textSQL2 = "UPDATE entreprise SET
~~id = '".$updateid."',~~
nom = '" . $updatenom . "',
lien = '" . $updatelien . "',
mail = '" . $updatemail . "',
info = '" . $updateinfo . "',
status = '" . $updatestatus . "',
departement = '" . $updatedep . "'
WHERE id ='" .$updateid. "'
";
03/05/2012, 17h55
Lekno

C'est a présent résolu, après lecture et relecture, j'avais un input hidden qui ne me dirigeai pas vers le bon traitement dans un if plus lointain.... merci à tous pour votre aide :ccool: