Discussion :

[Zeus_Appolos]

Bonjour les amis, j'ai besoin de votre aide pour insérer les données mon tableau dans ma base de données .
je remplis un tableau html avec jQuery puis je récupère ses valeurs que j'insère ensuite dans ma base de données MySQL.
le tableau est bien remplis


Mon soucis se situe au niveau de l'insertion des données.
Lorsque je soumets le formulaire, il n'y a pas d'erreur. Dans le sous-onglet payload de Network, il affiche : [object Object]. les données ne s'enregistrent pas


script ajax

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
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
 
var table_data = [];
$("table#table tr").each(function(){
	hidden_fonction = parseInt($(this).find('.hidden_fonction').text());
	hidden_membre = parseInt($(this).find('.hidden_membre').text());
	hidden_dateentree = $(this).find('.hidden_dateentree').text();
 
	if($(this).find('td:eq(0)').text()==""){
 
	}else{
		table_data.push({
			'hidden_fonction' : $(this).find('td:eq(1)').text(),
			'hidden_membre' : $(this).find('td:eq(3)').text(),
			'hidden_dateentree' : $(this).find('td:eq(5)').text(),
		}
		);
	}
 
	//
});
 
 
$.ajax({  
	url:"ligne_add.php",  
	method:"POST",  
	data: {table_data : table_data},
	dataType: "json",
	contentType: "application/json; charset=utf-8",
	processData: false,
	contentType: false,
	success :function(data){
	// debut
		$("#message-bd.alert-success").remove();
		$("#message-bd").addClass("alert-success");
		$("#message-bd.alert-success").html(data.message);
		$("#message_field_add").children().remove();
		$('#bureau_form')[0].reset();
		$("#message_field_add").empty();
		$('#bureau #myModalAdd').css('transform', 'scale(0)');
		load_data_bureau();
		$('#message-bd.alert-success').fadeIn(250).delay(3000).fadeOut(250);
		window.setTimeout(function(){
			window.location.reload();     
		}, 3000);
	}	
});

php

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
24
25
26
27
 
if(!empty($_POST)){ 
       $tabDonnees=$_POST['table_data'];
        //var_dump($tabDonnees);
        foreach($tabDonnees as $table)
        {
            $membre_id = $table['hidden_membre'];
            $fonction_id = $table['hidden_fonction'];
            $dateentre = $table['hidden_dateentree'];
            echo $membre_id;
            $req=$bd->prepare("INSERT INTO  `composer_bureau`(`BureauId`,`MembreId`, `PosteId`, `BureauDateentree`, `BureauDatesortie`)  
                                                VALUES (:MembreId, :BureauId, :PosteId, :BureauDateentree, '9999-12-31')");
            $rep=$req->execute(array(
                ':BureauId' => $last_id,
                ':MembreId' => $membre_id,
                ':PosteId' => $fonction_id,
                ':BureauDateentree' => $dateentre
            ));
        }
        if($rep){
            $data['success_bd'] = true;
            $data['message'] = 'Enregistrement effectué avec succès';
        }else{
            $data['success_bd'] = false;
            $data['message'] = "impossible d'enregistrer";
        }
}

[escartefigue]

Bonjour,

Attention : l'ordre des colonnes n'est pas le même dans la liste des colonnes et dans la liste des host variables, l'identifiant du bureau et du membre sont inversés :

Code SQL :

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

1
2
3
INSERT INTO  `composer_bureau`
            (`BureauId`,`MembreId`, `PosteId`, `BureauDateentree`, `BureauDatesortie`)  
VALUES (:MembreId, :BureauId, :PosteId, :BureauDateentree, '9999-12-31')");

Ce faisant, soit la requête va planter pour valeur de clef étrangère inconnue dans la table où cette colonne est clef primaire, soit, pire encore, la requête va passer, mais les valeurs insérées seront incohérentes !

[Zeus_Appolos]

Merci escartefigue pour votre prompt réponse

Attention : l'ordre des colonnes n'est pas le même dans la liste des colonnes et dans la liste des host variables, l'identifiant du bureau et du membre sont inversés :

c'est une erreur d'inattention. je l'ai corrigé mais ça ne passe toujours pas.

[Zeus_Appolos]

Merci escartefigue,
Je veux savoir si ma méthode pour récupérer les données du tableau n'est la bonne côté jQuery comme côté PHP.
Portant lorsque je fais un console.log() de ma variable table_data, ça m'affiche bien les données dans la console.

[two3d]

Ce faisant, soit la requête va planter pour valeur de clef étrangère inconnue dans la table où cette colonne est clef primaire, soit, pire encore, la requête va passer, mais les valeurs insérées seront incohérentes !

Je penses pas qu'il y ai une incidence si on utilise des valeurs nommées dans le mauvais ordre, en revanche avec les valeurs positionnels ":?", oui. A confirmer, la doc ne le dit pas.

https://www.php.net/manual/fr/pdostatement.execute.php

C'est un tableau modifiable à la volée ?

[darkstar123456]

Bonjour,

Il y a de grandes chances que ça ne soit pas normal de voir [object Object] dans le Payload
Normalement vous devriez voir tout ce qui a été envoyé par la requête POST
D'ailleurs, dans votre PHP, avez-vous vérifiez que vous receviez quoi que ce soit ? J'imagine que vous n'avez rien du tout et donc la première condition (if(!empty($_POST)){) n'est jamais vraie et donc le code jamais exécuté.

Dans votre appel AJAX vous définissez plein d'attributs : je le déconseille fortement
Je trouve que de bien écrire sa réponse (avec un header JSON pour une réponse JSON par exemple) est plus propre que de forcer la requête AJAX

Si vous supprimez les propriétés contentType, processData et contentType de votre appel AJAX, il devrait s'exécuter normalement (et votre onglet "Payload" devrait afficher votre formulaire sous forme de tableau)

Code javascript :

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

1
2
3
4
5
6
7
8
9
$.ajax({  
	url:"ligne_add.php",  
	method:"POST",  
	data: {table_data : table_data},
	dataType: "json", // tant qu'à faire, faudrait supprimer dataType aussi :) c'est mieux d'envoyer le bon header avec la réponse voir plus bas
//	contentType: "application/json; charset=utf-8",
//	processData: false,
//	contentType: false,
	success :function(data){

Pour ce qui est de la réponse, vu que ça semble être une réponse JSON, voici une petite fonction PHP qui fait en sorte de l'afficher correctement et de retourner le header correct :

Code php :

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
<?php
/**
 * Output JSON response encoded with headers
 * 
 * @param array $response
 */
function json_response($response = null)
{
    // End AJAX return if no data.
    if ($response == null)
        exit;
    // Proper headers for json output.
    header('Content-Type: application/json');
    // Output our response.
    echo json_encode($response);
    exit;
}

Faut juste lui donner un array en PHP et il va le convertir et renvoyer le bon header
Exemple vu que vous utilisez $data dans votre code :

Code php :

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

1
2
3
4
5
6
7
8
9
 
        if($rep){
            $data['success_bd'] = true;
            $data['message'] = 'Enregistrement effectué avec succès';
        }else{
            $data['success_bd'] = false;
            $data['message'] = "impossible d'enregistrer";
        }
        json_response($data);

[two3d]

C'est ce que j'appelle ce compliquer la vie. Ta réponse est bonne Darkstar mais c'est ce compliquer pour rien.

Je résume :

$.ajax peut être directement utilisé en $.post :

Code :

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

1
2
3
4
5
6
$.post("ligne_add.php", {table_data : table_data},function(data){
 
  if(data.success_bd) // perso j'enverrais "true" (en string), pas true (booléen), puis je ferais "if(data.success_bd == "true")"
   alert("ok !");
 
}, "json");

Code :

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

1
2
if(!empty($_POST)){ // bof bof, rien ne vérifie si table_data existe
if(!empty($_POST['table_data']) && is_array($_POST['table_data'])){ //correct

Code :

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

header('Content-Type: application/json');

Pas besoin de se compliquer avec un header pour la réponse, sinon merci de confirmer pourquoi il est utile de l'utiliser

Je confirme que c'est inutile car une chaine JSON est décodée via l'appel JS $.post, donc inutile (je l'utilise tous les jours ainsi depuis longtemps..., jamais eu de souci)

[darkstar123456]

Hello @two3d,

Personnellement, je n'aime pas du tout l'écriture de $.post() car je trouve qu'on perd énormément en lisibilité
D'ailleurs, si tu trouves qu'il est inutile d'utiliser un header en PHP pour la réponse JSON, c'est pcq justement tu le forces dans ta requête AJAX avec le json en 4e argument.
Du coup, si tu omet ce 4e argument, la réponse sera interprétée comme étant un string...
Voici un test que je viens de mettre en place : https://tests.pierre-roels.com/ajax-headers.php

Suffit de changer la valeur du premier champ pour retourner ou non les headers dans la réponse
Les réponse sont visibles dans console et on voit bien qu'elles ne sont pas les mêmes :


Sinon, je rejoins l'avis de tester if(empty($_POST)) qui est pas terrible
Cependant, il est tout a fait possible d'utiliser des booléens dans la réponse JSON, sans aucun problème