Discussion :

[Stopher]

Bonjour à tous ,

voilà une insertion dans une BD qui me pose problème , pourtant , on ne peut pas plus simple ...

le but est d'enlever les espaces dans les numéros de tel et de fax .
j'utilise donc un simple preg_replace('/[[:space:]]/','',$_POST['telephone']);

mais cette fonction me fait planter la requête

par contre , si je ne met pas d'espace dans le num de tel , c'est ok ca passe ...

J'avoue ne plus trop comprendre d'ou peut venir le problème ...

voici le bout de code test :

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
18
19
20
21
22
23
 
 
 
$telephone = preg_replace('/[[:space:]]/','',$_POST['telephone']); 
$fax = preg_replace('/[[:space:]]/','',$_POST['fax']);
 
 
$requete = 'INSERT INTO entreprise
 (nom_ent,adresse,adresse2,cp,ville,pays,activite,telephone,fax,email,site_internet)
    VALUES (
    \''.$nom_ent.'\',
    \''.$adresse.'\',
    \''.$adresse2.'\',
    \''.$codepostal.'\',
    \''.$ville.'\',
    \''.$_POST['pays'].'\',
    \''.$activite.'\',
    \''.$telephone.'\',
    \''.$fax.'\',
    \''.$_POST['email'].'\',
    \''.$site.'\')';
 
    $res = mysql_query($requete)or die(mysql_error());

Merci d'avance pour votre aide ...

Cordialement ,
Christophe.

[julp]

mais cette fonction me fait planter la requête

Oui, c'est à dire ? Quelle est la requête exécutée ? Quel est le message d'erreur ?

Cela dit pourquoi ne pas avoir utilisé :

Code :

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

$fax = preg_replace('/[^\d]/', '', $_POST['fax']);

Pour éviter certaines problèmes (injections SQL dont vos autres champs ne sont pas vraisemblablement protégés).

[Stopher]

Voici la requete :

Code :

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

INSERT INTO entreprise (nom_ent,adresse,adresse2,cp,ville,pays,activite,telephone,fax,email,site_internet) VALUES ( 'ENTREPRISE', 'adresse1', 'adresse2', '59130', 'LAMBERSART', 'FRANCE', 'TESt', '0303040505', '1234123454', 'toto@titi.com', 'http://google.com')

Et le message d'erreur :

Code :

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

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

Et

Code :

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

$fax = preg_replace('/[^\d]/', '', $_POST['fax']);

donne la meme erreur ...

En effet rien n'est fait contre les injections sql , mais j'ai retiré tout traitement afin de trouver plus facilement d'ou vient l'erreur ...

[Stopher]

Ok ,
tout d'abord , désolé , car l'erreur venait d'ailleurs ,
en effet , une autre petite requete traînait un peut plus loin dans mon script .. de ce fait je ne regardais pas celle qui posait problème ...

Autant de temps perdu pour si peut ... au moins maintenant j'identifie ma requete dans les messages d'erreurs ... :-)

Merci à toi julp et désolé d'avoir fait perdre ton temps

Christophe.