Insertion quand validation Form sous Firefox

Version imprimable

11/03/2014, 15h29
Kaelyscius

Insertion quand validation Form sous Firefox

Bonjour à tous et à toutes.

Je post ici mon souci, j'espère être au bon endroit. (J'ai faits des recherches, ici et sur le net bien sûr) et je n'ai rien trouvé se rapprochant de mon problème.

Jeune développeur web, je crée actuellement mon site pour "vendre" un peu mes bases de connaissances acquises et qui m'aiderait à trouver un premier emploi.

Bref!

Mon souci concerne en fait la validation de mon formulaire de contact côté PHP/Mysql en validant un formulaire sous Firefox.

En effet le code semble bon puis ce que je récupère les données et qu'elles s’inscrivent en BDD sous IE / Chrome / Opera. (donc dans la logique cela me semble bon puis ce que d'un coté ça fait le chemin en totalité)

Je suis bien conscient qu'un navigateur n'est pas censé intervenir sur un script côté serveur, mais ce que je comprends pas du coup, c'est qu'en POST, j'ai bien la récupération et l'envoient des données formulaires, mais elle n'arrive pas jusqu'à la base.

Alors je ne sais pas si j'ai raté un truc dans mon code (si mon controler renvoie pas ce que je demande, si c'est la méthode de ma classe que j'ai, n'a pas effectué un bon traitement , ou alors dans ma table MySQL).

Je peux évidemment mettre des bouts de code si besoin et je pense que cela sera le cas.

Je remercie d'avance celles et ceux qui prendront un peu de temps pour m'aider avant que je devienne chauve! ;)

Amicalement
11/03/2014, 15h46
sabotage

Ca devrait être assez simple à debuguer.
Avec Firefox :
- Que vaut $_POST ?
- Que vaut la requête ?
11/03/2014, 16h25
Kaelyscius

Hello,

sur Firefox via la console je vois en post :

email test@test.fr
firstname test
lastname test
message test
method addMsg
phone (j'ai laissé ce champ vide, il est optionnel)
website (j'ai laissé ce champ vide, il est optionnel)

Un print_r de mon $_post dans le controler = Array ( [firstname] => test [lastname] => test [email] => test@test.fr [website] => [phoneNumber] => [captcha] => 11 [message] => test )

en revanche je vois pas ce que tu souhaites au niveau de la requête.

Je te remercie de ton aide! ;)
11/03/2014, 18h27
sabotage

Tu executes une requête

Code:

$cnx->query('une requête');

affiche ce que vaux 'une requête'

Je suis désolé, je n'arrive pas à faire remonter l'info de ce que contient ma requête.:oops:

Ma méthode est celle ci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public function addMessage($array){
	$stmt = $this->_db->prepare('INSERT INTO messages(firstname,lastname,email,website,phone,message,datePosted)
		VALUES(:f,:l,:e,:w,:p,:m,:dP)');
	$stmt->execute(array(
		"f"			=> $array['firstname'],
		"l"			=> $array['lastname'],
		"e"			=> $array['email'],
		"w"			=> $array['website'],
		"p"			=> $array['phone'],
		"m"			=> $array['message'],
		"dP"		        => date('Y-m-d H:i:s')
	));
	return $this->_db->lastInsertId();
}

si tu ajoute print_r($array);:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
public function addMessage($array){
	print_r($array);
	$stmt = $this->_db->prepare('INSERT INTO messages(firstname,lastname,email,website,phone,message,datePosted)
		VALUES(:f,:l,:e,:w,:p,:m,:dP)');
	$stmt->execute(array(
		"f"			=> $array['firstname'],
		"l"			=> $array['lastname'],
		"e"			=> $array['email'],
		"w"			=> $array['website'],
		"p"			=> $array['phone'],
		"m"			=> $array['message'],
		"dP"		        => date('Y-m-d H:i:s')
	));
	return $this->_db->lastInsertId();
}

tu obtient quoi ?

11/03/2014, 19h19
Kaelyscius

Le print_r ne remonte pas, que je pointe l'action de mon formulaire sur mon fichier process.php, où messages.class.php ni sur firefox, ni sur chrome (avec lequel l'inscription en bdd se fait parfaitement.), rien ne s'affiche.

J'avais déja essayé un print_r sur la variable array, qui est censé tout contenir, mais je n'arrive pas à en obtenir le résultat
11/03/2014, 19h24
patrickbaras

il y a bien un endroit ou tu appelle addMessage(...) il faut vérifier la valeur qui est passée dans (...).

Dans mon process.php j'utilise ce code la :
Code:

1 2 3 4 5 require 'class/messages.class.php'; if ($_POST['method']=='addMsg') { $messages = new Messages(); $add = $messages->addMessage($_POST); }
Ce que contient le $_post est quelques post plus haut pour rappel au moment du test :
Array ( [firstname] => test [lastname] => test [email] => test@test.fr [website] => [phoneNumber] => [captcha] => 11 [message] => test ) 
les champs website et phoneNumber sont des champs optionnels non remplis au moment du test

11/03/2014, 21h35
sabotage

"method" a disparu de ton $_POST entre tes deux messages.
d'ou vient "method" ?

La methode vient de ma requête ajax :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
$("#submit").on("click",function(){
  var firstname = $("#firstname").val(),
  lastname      = $("#lastname").val(),
  email         = $("#email").val(),
  website       = $("#website").val(),
  phone         = $("#phonenumber").val(),
  message       = $("#message").val();
  if (firstname.length>0 && lastname.length>0 && email.length>0  && website.length>=0  && phone.length>=0 && message.length>0) {
    $.ajax({
      url: 'process.php',
      type: 'POST',
      data: {
        method: 'addMsg',
        firstname: firstname,
        lastname: lastname,
        email: email,
        website: website,
        phone: phone,
        message: message
      },
      success: function(result){
        }
    });
  } else {
    alert('Vous devez remplir tous les champs');
  }

L'apparition de la method apparait dans la console navigateur, mais n'est pas dans le $_POST

11/03/2014, 22h54
sabotage

Direction jQuery.

ne faudrait-il pas préciser que data contient du json !?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
...
    $.ajax({
      url: 'process.php',
      type: 'POST',
      dataType: "json",
      data: {
        method: 'addMsg',
        firstname: firstname,
        lastname: lastname,
        email: email,
        website: website,
        phone: phone,
        message: message
      },
...

11/03/2014, 23h18
Kaelyscius

Je vais essayer cela ;) je vous tiens au courant.
11/03/2014, 23h21
patrickbaras

certain navigateur sont plus pointilleux sur certain point !
11/03/2014, 23h31
Kaelyscius
Cela ne change malheureusement rien. :(

Cependant, la console me met en rouge une erreur :
http://localhost/Siteperso/process.php / query.js (ligne 9666)

Ce qui correspond à :
Code:

1 2 3 4 5 6 // Do send the request // This may raise an exception which is actually // handled in jQuery.ajax (so no try/catch here) xhr.send( ( options.hasContent && options.data ) || null );
Donc cela viendrait d'un problème avec l'utilisation d'ajax avec jquery et la connexion à la base de données? C'est bien ça?

Bonsoir

Avoir le courage d'indiquer explicitement que les propriétés d'un objet sont de type texte, évite bien des déboires. Le mot method n'est dans la liste des mots réservés, mais je suggère de tester :
Code:

1 2 3 4 5 6 7 8 9 data: { "method" : 'addMsg', "firstname" : firstname, "lastname" : lastname, "email" : email, "website" : website, "phone" : phone, "message" : message },

11/03/2014, 23h56
Kaelyscius

Bonsoir,

Je suis un développeur web débutant, donc je ne maitrise pas toute les bonnes pratiques encore.

Je ne comprend pas ce que peux signifier

Citation:

Avoir le courage d'indiquer explicitement que les propriétés d'un objet sont de type texte, évite bien des déboires.

C'est au niveau de mon explication, ou du code en lui même?

Sinon effectuer ces modifications n'ont rien changé. Même code erreur sur firefox, aucun soucis ailleurs. Merci vraiment d'essayer de me filer un coup de patte. Car j'avoue vraiment ne pas comprendre pourquoi ce que j'ai fais fonctionne d'un côté mais pas d'un autre.

bien je viens de faire le test dans une page test.php :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
<!DOCTYPE html>
<html>
<head>
<meta http-equiv="X-UA-Compatible" content="IE=Edge" />
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<script language="javascript" type="text/javascript" src="../javascript/jquery-1.7.2.min.js"></script>
</head>
<body>
	<input type='text' id='firstname' value='test'>
	<input type='text' id='lastname' value='test'>
	<input type='text' id='email' value='test@test.fr '>
	<input type='text' id='website' value=''>
	<input type='text' id='phonenumber' value=''>
	<input type='text' id='message' value='test'>
	<input type='button' id='submit'>
<script>
$("#submit").on("click",function(){
 
  var firstname = $("#firstname").val(),
  lastname      = $("#lastname").val(),
  email         = $("#email").val(),
  website       = $("#website").val(),
  phone         = $("#phonenumber").val(),
  message       = $("#message").val();
 
  if (firstname.length>0 && lastname.length>0 && email.length>0  && website.length>=0  && phone.length>=0 && message.length>0) 
  {
 
    $.ajax({
                url: 'page.php',
                type: 'POST',
                data: {
                        method: 'addMsg',
                        firstname: firstname,
                        lastname: lastname,
                        email: email,
                        website: website,
                        phone: phone,
                        message: message
                },
                success: function(msg){
                        alert(msg);
                }
                ,error : function(obj,msg,exep)
                {
                        alert(msg+" "+exep);
                }
    });
  } else {
    alert('Vous devez remplir tous les champs');
  } 
 });
</script>
</body>
</html>

et dans ma page page.php:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
$out='';
foreach($_POST as $key=>$value)
{
		$out.="POST[$key]=$value\n";
 
}
foreach($_GET as $key=>$value)
{
		$out.="GET[$key]=$value\n";
 
}
echo $out;
?>

J'ai le même resultat sur Chrome & Firefox
donc le soucis devrait venir du code de process.php ou de la class message

Hello,

Effectivement ton code ressort les données contenue en $_POST avec une boucle. j'en comprend bien l'utilité.

Quand je pointe l'action de mon formulaire sur process.php, je vois l'echo du $out, ou le print_r du $_POST.

J'ai également une notice qui apparait :

Citation:

Notice: Undefined index: method in C:\wamp\www\Siteperso\process.php on line 4

Ligne où mon code dans process.php ressemble à ça :

Code:

if ($_POST['method']=='addMsg'){

Une ligne bien récupéré par le POST quand je regarde dans la console de firefox:
email	=> test@test.fr
firstname => test
lastname => test
message => test
method => addMsg
phone => champ optionnel
website => champ optionnel
Quel casse-tête, je pensais m'en être sorti correctement, mais apparemment je vais devoir revoir des choses.

J'arrive pas à visualiser pourquoi tout les navigateurs sembles faire le bout du chemin sauf firefox.

Après je me suis dit, au vu de l'erreur cité plus haut, avec le paragraphe sur Jquery, je me demande si je ne devrait pas crée une Class PDO pour la connexion à la Bdd et faire des autres Class, des Class héritées. Mais après comment le faire sera une autre paire de manche.:P

pour rappel :
Code:

1 2 3 4 // Do send the request // This may raise an exception which is actually // handled in jQuery.ajax (so no try/catch here) xhr.send( ( options.hasContent && options.data ) || null );
Ma Class Messages :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 Class Messages{ private $_db; const HOST = "localhost"; const USER = "user"; const PASS = "mdp"; const DBNAME = "siteperso"; public function __construct(){ try { echo('la connexion est ok'); $this->_db = new PDO("mysql:host=".self::HOST.";dbname=".self::DBNAME,self::USER,self::PASS); } catch (PDOException $e) { echo $e->getMessage(); } } public function addMessage($array){ $stmt = $this->_db->prepare('INSERT INTO messages(firstname,lastname,email,website,phone,message,datePosted) VALUES(:f,:l,:e,:w,:p,:m,:dP)'); $stmt->execute(array( "f" => $array['firstname'], "l" => $array['lastname'], "e" => $array['email'], "w" => $array['website'], "p" => $array['phone'], "m" => $array['message'], "dP" => date('Y-m-d H:i:s') )); return $this->_db->lastInsertId(); } }

12/03/2014, 13h17
patrickbaras
juste avant t'a ligne

Code:

if ($_POST['method']=='addMsg'){

ajoute
Code:

1 2 var_dump($_POST); exit;
qu'a-tu comme resultat ?

array (size=7)
  'firstname' => string 'test' (length=4)
  'lastname' => string 'test' (length=4)
  'email' => string 'test@test.fr' (length=12)
  'website' => string '' (length=0)
  'phoneNumber' => string '' (length=0)
  'captcha' => string '65' (length=2)
  'message' => string 'test' (length=4)

12/03/2014, 13h54
patrickbaras
donc il te manque bien "method"
si tu deplace le code avant le require
Code:

1 2 3 4 5 6 7 var_dump($_POST); exit; require 'class/messages.class.php'; if ($_POST['method']=='addMsg') { $messages = new Messages(); $add = $messages->addMessage($_POST); }
a-tu "method" ?
si oui le problème vient de ta class message
si non c'est que le post n'envois pas la donnée "method"
essaye avec un autre nom "ma_methode" par exemple

tu est sur que l'appel ce fait bien par l'ajax que tu renseigne ici car il y a "captcha" alors que dans l'ajax cette donné n'est pas renseignée.
Code:

1 2 3 4 5 6 7 8 array (size=7) 'firstname' => string 'test' (length=4) 'lastname' => string 'test' (length=4) 'email' => string 'test@test.fr' (length=12) 'website' => string '' (length=0) 'phoneNumber' => string '' (length=0) 'captcha' => string '65' (length=2) 'message' => string 'test' (length=4)

12/03/2014, 14h08
Kaelyscius

Non effectivement j'ai toujours la même chose, le post n'envoie pas la method.

Il se voit dans la console, mais pas dans l'envoie.

Pour le captcha, j'ai souhaité ne pas le traiter, car c'est juste une validation de formulaire qui permet de cliquer ou non sur le bouton d'envoi, via un plugin jquery.

D'où le fait que j'ai crée un script en prenant pour variable que ce dont j'avais besoin et de les envoyer dans ma requête Ajax. Je n'ai pas trouvé cela bloquant lors de la création de mon script.
12/03/2014, 14h34
patrickbaras

vérifie également si l'input avec id="submit" est de type "submit","bouton" ou autre
12/03/2014, 14h44
Kaelyscius

Code:

<button type="submit" class="btn btn-primary" value="submit" id="submit">Envoyer mon message</button>
12/03/2014, 14h50
patrickbaras

remplace submit par button , je crois que cela fonctionnera mieux.

Code:

<button type="button" class="btn btn-primary" value="submit" id="submit">Envoyer mon message</button>
12/03/2014, 15h13
Kaelyscius
Dans la console firefox, ça ne met plus la ligne en rouge, mais en noir comme si cela fonctionnait bien:

POST http://localhost/Siteperso/process.php 200 OK 4ms

contenu du post dans la console
```
email	test@test.fr
firstname	test
lastname	test
message	test
method	addMsg
phone	
website	
```
cependant l'action ne marche plus car je ne tombe sur aucune autre page, et cela ne s'inscrit pas dans la base de données.

D'ailleurs pour le coup, mettre en type button, me bloque même sur les autres navigateurs.
12/03/2014, 15h38
patrickbaras
Je crois que tu mélange 2 manières de faire pour transmettre les informations:
soit tu utilise
Code:

1 2 3 4 5 <form...method="POST" action="pageXX.php"> <input type="text" name="nominput" id="idinput" valeur="valeurinput"> ... <input type="submit"...> </form>
sans AJAX
dans cette option seront "postée" nominput=valeurinput, si pas de name="..." dans input pas de valeur envoyée.
et le navigateur va a la page pageXX.php

soit tu utilise ajax avec <input type="button"...> les valeur postées seront celles contenu dans
data :
Code:

1 2 3 4 5 6 7 8 9 10 11 ... var varjsinput = $("#idinput").val() ... $.ajax({ url: 'process.php', type: 'POST', data: { unautrenom: varjsinput ... }, ...
dans cette option seront "postée" unautrenom=valeurinput
mais tu reste sur la même page, le but d'AJAX étant (en autre) d’exécuter des pages en restant sur la même page (sans la recharger)

A-tu-supprimé le code avec
Code:

1 2 var_dump($_POST); exit;
?
12/03/2014, 16h03
Kaelyscius

Oui j'ai supprimé le code dans le process.

Et ça marche niquel du coup avec le type=button, j'avais juste laissé une tite coquille. ! testé sous FireFox, Chrome, IE, Opéra!

Je te remercie vraiment pour ton aide, comme quoi au final, c'était juste un problème de type et d'incompréhension de transimision de données... Merci également pour cette précision sur ton post d'avant. Je tâcherais du coup de ne plus confondre les deux.

Plus qu'à faire en sorte que les champs s’efface ou que je change de page. Et je pourrais me recoller les cheveux que j'ai perdu ces derniers jours.

Merci vraiment !
12/03/2014, 16h09
patrickbaras

De rien , n'oublie pas de mettre le sujet en "résolu"