Discussion :

[danamaro]

Bonjour,
j'ai un problème de syntaxe pour exécuter une requête update dans un foreach :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
foreach ($tabFormCustomer as $entry => $key) {
 
//        var_dump($entry);
//        var_dump($key );
 
        $reponse_Updatecustomer = database::getDB()->fct_updatePrepare(
            "update customer_nfo
            left join stay_nfo on  idcustomer=post_id
            set
            '".$entry."' = '".$key."'
            where idstays_nfo = '".$get_CustomerID."' ");
    }

ça coince au niveau du :

Code :

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

'".$entry."' = '".$key."'

Si je rentre en dur comme par exemple :

Code :

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

ch_country = "zozo"

cela fonctionne parfaitement, la requête update s'exécute.

Pour les deux variables, le vardump donne cela (extrait..)
\src\BO\BO_view\updateCustomer_view.php:87:string 'ch_origin' (length=9)
\src\BO\BO_view\updateCustomer_view.php:88:string 'Airbnb' (length=6)
\src\BO\BO_view\updateCustomer_view.php:87:string 'ch_civility' (length=11)
\src\BO\BO_view\updateCustomer_view.php:88:string '2' (length=1)
\src\BO\BO_view\updateCustomer_view.php:87:string 'ch_first_name' (length=13)
\src\BO\BO_view\updateCustomer_view.php:88:string 'Alexandra' (length=9)


Merci d'avance

[vttman]

Bonjour danamaro,

En passant ...
Et en enlevant les simple quotes autour de $entry ça donne quoi ?
=>

Code :

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

1
2
3
4
5
6
$reponse_Updatecustomer = database::getDB()->fct_updatePrepare(
            "update customer_nfo
            left join stay_nfo on  idcustomer=post_id
            set
            ".$entry." = '".$key."'
            where idstays_nfo = '".$get_CustomerID."' ");

[Invité]

Bonjour,

A quoi ressemble la fonction fct_updatePrepare() ?

Parce que là, tu NE prépare RIEN du tout !

[danamaro]

Bonjour,

A quoi ressemble la fonction fct_updatePrepare() ?

Parce que là, tu NE prépare RIEN du tout !

Code :

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

1
2
3
4
5
public function fct_updatePrepare($statement) {
 
        $res = $this->getPDO()->prepare($statement);
        $res->execute();
    }

Cela fonctionne très bien si je rentre par exemple :

Code :

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

ch_country = "zozo"

mais avec les variables $entry et $key je n'arrive pas à trouver la bonne syntaxe

[danamaro]

J'ai fait quelques tests supplémentaires, et avec

Code :

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

 $tutu = "toto";

ça fonctionne !

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
foreach ($tabFormCustomer as $entry => $key) {
 
//        var_dump($entry);
//        var_dump($key );
        $tutu = "toto";
 
        $reponse_Updatecustomer = database::getDB()->fct_updatePrepare(
            "update customer_nfo
            left join stay_nfo on  idcustomer=post_id
            set
            ch_country = '".$tutu."'
            where idstays_nfo = '".$get_CustomerID."' ");
    }

[Invité]

Bonjour,

1- Le problème n'est pas que "ça fonctionne ou pas" : le problème c'est que ce N'est PAS une requête préparée !

2- Voici à quoi devrait ressembler une VRAI requête préparée :

Code :

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

1
2
3
4
5
public function fct_updatePrepare($statement, $params=[]) {
 
        $res = $this->getPDO()->prepare($statement);
        $res->execute($params);
}

Et l'appel :

Code :

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

1
2
3
4
5
6
7
8
$reponse_Updatecustomer = database::getDB()->fct_updatePrepare(
            "update customer_nfo
            left join stay_nfo on  idcustomer=post_id
            set
            ".$entry." = ?
            where idstays_nfo = ? ",
			[$key, $get_CustomerID]
			);

Encore faut-il que $entry n'amène pas de risque d'injection SQL !
-> il faudrait s'assurer qu'il correspond bien au nom d'une colonne de la table SQL

[danamaro]

Un grand merci, c'est tout à fait ça.
j'ai complètement zappé cette partie. Je vais creuser la question sur le problème de $entry.

Merci,

[danamaro]

Bonjour danamaro,

En passant ...
Et en enlevant les simple quotes autour de $entry ça donne quoi ?
=>

cela ne donne rien, désolé