Script PHP - INSERT INTO

Version imprimable

Bonjour,

je fais face à un problème assez particulier aujourd'hui, je m'explique: sur une page permettant la gestion des salariés de mon entreprise, un bouton "Nouveau" ouvre une modale contenant un formulaire à remplir. Quand tous les champs sont remplis, l'insertion réussie. Mais quand il manque un seul champ, l'insertion ne réussie pas et qu'importe le champ non remplis. Je remplis evidemment les champs obligatoires.

Voici la table:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
CREATE TABLE IF NOT EXISTS salarie
 (
   numSalarie INTEGER NOT NULL AUTO_INCREMENT ,
   numNiveau INTEGER NOT NULL  ,
   numPoste INTEGER NOT NULL  ,
   nomSalarie CHAR(32) NOT NULL  ,
   prenomSalarie VARCHAR(128) NOT NULL  ,
   dateFIMO DATE NULL  ,
   dateFCO DATE NULL  ,
   dateValiditeCarteConducteur DATE NULL  ,
   dateValiditeQualificationConducteur DATE NULL  ,
   dateVisiteMedicale DATE NULL  ,
   AIPR VARCHAR(32) NULL  ,
   dateValiditeAIPR DATE NULL  
   , PRIMARY KEY (numSalarie) 
 )

Le formulaire :

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
 
<FORM method="POST" action="Scripts/salarie_create.php">
							Nom: <input type="text" name="nomSalarie"> <br>
							Prénom: <input type="te" name="prenomSalarie"> <br>
							* Format des dates: aaaa-mm-jj * <br>
							Prochaine visite médicale: <input type="text" name="dateVisiteMedicale"> <br>
 
							<!--Liste déroulante -->
							Niveau de droit:<select name="droit" style="width: 230px">
												<option value="0"> - - - - - - - - </option>
												<?php
												$req = "SELECT * FROM niveauSalarie ORDER BY numNiveau";
												try{
													$stmt = $pdo->prepare($req);
													$stmt->execute();
													while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
														echo "<option value='".$row["numNiveau"]."'>".$row["nomNiveau"]."</option>";
													}
												}
												catch(Exception $e){
													print "Erreur ! ".$e->getMessage(). "<br>";
												}
												?>
												</select> <br>
 
							<!-- Liste déroulante -->					
							Poste :	<select name="poste" style="width: 230px">
										<option value="0"> - - - - - - - - </option>
										<?php
										$req = "SELECT * FROM posteSalarie ORDER BY nomPoste";
										try{
											$stmt = $pdo->prepare($req);
											$stmt->execute();
											while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
												echo "<option value='".$row["numPoste"]."'>".$row["nomPoste"]."</option>";
											}
										}
										catch(Exception $e){
											print "Erreur !".$e->getMessage()."<br>";										}
										?>
									</select> <br> <br>
							< - - - Zone conducteur / chauffeur - - - > <br>
							Date précédente FCO: <input type="text" name="fimo"> <br>
							Date prochaine FCO: <input type="tex" name="fco"> <br>
							Date fin de validité de <br> la carte conducteur: <input type="text" name="carte"> <br>
							Date fin de validité de <br> la qualification conducteur: <input type="text" name="qualification"> <br>
							AIPR: 	<select name="aipr">
										<option value="0"> - - - - - - - - </option>
										<option value="Concepteur">Concepteur</option>
										<option value="Encadrant">Encadrant</option>
										<option value="Operateur">Opérateur</option>
									</select> <br>
							Date fin de validité AIPR: <input type="text" name="dateAIPR"> <br>
							<input type="submit" value="Confirmer">
						</FORM>

Le script PHP appelé:

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
56
57
58
59
 
<?php 
include "../include/connexion.php";
//Si le nom du salarié est bien indiqué, ainsi que son prénom, son poste et son niveau de droit sur l'application
if (isset($_POST['nomSalarie']) && !empty($_POST['prenomSalarie']) && !empty($_POST['poste']) && !empty($_POST['droit'])) {
	$nom = $_POST['nomSalarie'];
	$prenom = $_POST['prenomSalarie'];
	$poste = $_POST['poste'];
	$niveau = $_POST['droit'];
	//On crée un nouveau salarié
	include "../Classe/salarie.php";
	$unSalarie = new Salarie ($nom, $prenom, $poste, $niveau);
 
	//On le met à jour en fonction des informations rentrées avant de l'enregistrer dans la BDD
	//SI *propriété* est mise, alors appelle de la fonction Setter avec param, SINON on vide l'attribut pour éviter que le nouveau salarié récupère la propriété d'un précédent salarié
	$vide = ""; //Variable pour vider
	if (isset($_POST['dateVisiteMedicale'])) {
		$unSalarie->setDateVisiteMedicale($_POST['dateVisiteMedicale']);
	} else {
		$unSalarie->setDateVisiteMedicale($vide);
	}
	if (isset($_POST['fimo'])) {
		$unSalarie->setDateFIMO($_POST['fimo']);
	} else {
		$unSalarie->setDateFIMO($vide);
	}
	if (isset($_POST['fco'])) {
		$unSalarie->setDateFCO($_POST['fco']);
	} else{
		$unSalarie->setDateFCO($vide);
	}
	if (isset($_POST['carte'])) {
		$unSalarie->setDateValiditeCarteConducteur($_POST['carte']);
	} else {
		$unSalarie->setDateValiditeCarteConducteur($vide);
	}
	if (isset($_POST['qualification'])) {
		$unSalarie->setDateQualificationConducteur($_POST['qualification']);
	} else {
		$unSalarie->setDateQualificationConducteur($vide);
	}
	if (isset($_POST['aipr'])) {
		$unSalarie->setAIPR($_POST['aipr']);
	} else {
		$unSalarie->setAIPR($vide);
	}
	if (isset($_POST['dateAIPR'])) {
		$unSalarie->setDateValiditeAIPR($_POST['dateAIPR']);
	} else {
		$unSalarie->setDateValiditeAIPR($vide);
	}
 
	//Enregistrement dans la BDD
	$unSalarie->create();
 
	//retour à la page salarie
	header("Location: ../page_salarie.php");
} 
?>

ainsi que ma fonction create():

Code:

1
2
3
4
5
6
7
8
9
10
11
 
	public function create(){
		include "../include/connexion.php";
		try{
			$req = "INSERT INTO salarie VALUES (NULL,'".$this->_niveauSalarie."','".$this->_posteSalarie."','".$this->_nomSalarie."','".$this->_prenomSalarie."','".$this->_dateFIMO."','".$this->_dateFCO."','".$this->_dateValiditeCarteConducteur."','".$this->_dateValiditeQualificationConducteur."','".$this->_dateVisiteMedicale."','".$this->_AIPR."','".$this->_dateValiditeAIPR."')";
			$pdo->exec($req);
		}
		catch(Exception $e){
			print "Erreur ! ".$e->getMessage(). "<br/>";
		}
	}

Je n'arrive pas à définir ce qui cause l'empêchement de l'insertion dans ma bdd puisque celle ci fonctionne lorsque tous les champs sont remplis et non quand il en manque des non obligatoires...

Merci d'avance pour toutes aides apportées !
Cordialement,
Skunka.

Bonjour

Une cause du probleme

Code:

1
2
 
$req = "INSERT INTO salarie (colonnes) VALUES (NULL,'".$this->_niveauSalarie."','".$this->_posteSalarie."','".$this->_nomSalarie."','".$this->_prenomSalarie."','".$this->_dateFIMO."','".$this->_dateFCO."','".$this->_dateValiditeCarteConducteur."','".$this->_dateValiditeQualificationConducteur."','".$this->_dateVisiteMedicale."','".$this->_AIPR."','".$this->_dateValiditeAIPR."')";

mettre les colonnes désignées apres salarie

Il y a un message d'erreur?

11/06/2018, 17h09
Celira

Déjà, utilise plutôt une requête préparée, vu que tu insères des données extérieures.

Quand tu dis "il manque un champ", je suppose que ça veut dire "quand un champ n'est pas rempli dans le formulaire". Le problème se pose avec n'importe quel champ ? Tu as un message d'erreur à nous montrer ?
11/06/2018, 17h18
Invité

Bonjour,

1- que donne : echo $req; ?

2- Pourquoi toute la gestion des $unSalarie->set...... n'est pas directement dans la fonction create (en passant le $_POST en paramètre) ?
Je ne maitrise pas la POO, mais ça me semble étrange...

Citation:
Envoyé par bonjourajax
Bonjour

Une cause du probleme
Code:

1 2 $req = "INSERT INTO salarie (colonnes) VALUES (NULL,'".$this->_niveauSalarie."','".$this->_posteSalarie."','".$this->_nomSalarie."','".$this->_prenomSalarie."','".$this->_dateFIMO."','".$this->_dateFCO."','".$this->_dateValiditeCarteConducteur."','".$this->_dateValiditeQualificationConducteur."','".$this->_dateVisiteMedicale."','".$this->_AIPR."','".$this->_dateValiditeAIPR."')";
mettre les colonnes désignées apres salarie

Il y a un message d'erreur?
Il me semble inutile d'indiquer les colonnes puisque la requête les traite toutes dans l'ordre présent dans la BDD (c'est en tout cas ce que m'ont appris mes professeurs :mouarf: )
C'est bien pour ça que je cherche de l'aide, je n'ai aucun message d'erreur qui s'affiche (il se peut que j'ai oublié de traiter cela dans le code...) et le script appelé me ramène bien sur ma page.
Tu as vu juste Celira, je parlais bien d'un champ du formulaire non remplis.

Si vous avez la solution me permettant d'afficher le message d'erreur, j'en serais ravis, cela m'aidera sûrement!
Sinon, toutes autres propositions sont la bienvenue!

Edit pour jreaux62 qui m'écris pendant que j'écrivais:
où place tu ton echo $req? Dans la fonction create() j'imagine ?
pour les "set" ton explication est légère mais je pense comprendre, j'essaye cela dès demain!

Cordialement,
Skunka.

PS: ma journée se termine dans 5 minutes, je suis de retour demain matin ! :p

11/06/2018, 17h58
Celira
Il n'est pas nécessaire d'écrire les colonnes dans une requête INSERT si on les alimente toutes, mais il est utile d'écrire les colonnes dans une requête INSERT. C'est comme le SELECT * : il vaut toujours mieux utiliser des requêtes explicites.Dans le cas d'une requête INSERT, ça permet d'être sûr de l'ordre dans lequel elles sont alimentées sans avoir à se référer à la base toutes les 3 minutes, ça évite (entre autres) d'avoir à repasser sur le code si tu ajoutes une colonne facultative à ta table et surtout ça évite les mauvaises surprises du genre "Hop ! On réinitialise la base et oh ! dans le nouveau script les colonnes sont par ordre alphabétique... Oups..."

Si tu veux des messages d'erreur avec PDO, il faut penser à activer les exceptions au niveau de la connexion :
Code:

1 2 $pdo = new PDO('...'); $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
@jreaux : Concernant la POO, le fait qu'il crée à partir de l'objet me parait valable : On construit un Salarie (comme on construit un tableau de données) et ensuite on le sauvegarde en base.
En revanche, ce qui m'embête, c'est que la méthode create est apparemment dans la classe Salarie, alors que Salarie est visiblement un objet métier (la représentation d'un salarié). Pour moi, il faut 2 classes : Salarie qui représente le salarié, avec nom, prénom, etc...) et SalarieDao (ou SalarieManager ou quelque chose qui genre) qui s'occupe de la communication avec la base.

1 pièce(s) jointe(s)

J'ai renseigné les colonnes à remplir avec une syntaxe normalement correcte:
Code:

1 2 $req = "INSERT INTO salarie (numSalarie, numNiveau, numPoste, nomSalarie, prenomSalarie, dateFIMO, dateFCO, dateValiditeCarteConducteur, dateValiditeQualificationConducteur, dateVisiteMedicale, AIPR, dateValiditeAIPR) VALUES (NULL,'".$this->_niveauSalarie."','".$this->_posteSalarie."','".$this->_nomSalarie."','".$this->_prenomSalarie."','".$this->_dateFIMO."','".$this->_dateFCO."','".$this->_dateValiditeCarteConducteur."','".$this->_dateValiditeQualificationConducteur."','".$this->_dateVisiteMedicale."','".$this->_AIPR."','".$this->_dateValiditeAIPR."')";
Egalement modifié un peu le code au niveau de la connexion pour activer les exceptions:
Code:

1 2 3 4 5 6 7 8 9 10 <?php try{ $pdo = new PDO('mysql:host=localhost;dbname=ttpm;charset=utf8', 'root', ''); $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); } catch(Exception $e){ die('Erreur :'.$e->getMessage()); } ?>
J'avais déjà un catch(Exception), j'ai ensuite ré effectué plusieurs essaies d'insertion et le résultat reste le même: l'insertion fonctionne quand tous les champs du formulaire sont renseignés mais ne fonctionne pas quand au moins un champ du formulaire n'est pas renseigné. J'ai même essayé de laisser un champ obligatoire en blanc mais je ne récupère toujours pas de message d'erreur ...
Dans mon script salarie_create.php , j'en suis venu à modifier le contenu de ma variable $vide par $vide = "NULL" pour que, lors de l'insertion, mon SGBD reçoivent une valeur NULL plutôt que vide mais cela n'a pas aidé :/

Ensuite j'ai entrepris cette modification du script salarie_create.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 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 <?php include "../include/connexion.php"; //Si le nom du salarié est bien indiqué, ainsi que son prénom, son poste et son niveau de droit sur l'application if (isset($_POST['nomSalarie']) && !empty($_POST['prenomSalarie']) && !empty($_POST['poste']) && !empty($_POST['droit'])) { $nom = $_POST['nomSalarie']; $prenom = $_POST['prenomSalarie']; $poste = $_POST['poste']; $niveau = $_POST['droit']; //On crée un nouveau salarié include "../Classe/salarie.php"; $unSalarie = new Salarie ($nom, $prenom, $poste, $niveau); //Enregistrement dans la BDD $unSalarie->create(); //On récupère le numéro du salarié créer pour le mettre à jour $req = "SELECT numSalarie FROM salarie WHERE nomSalarie = ".$nom." AND prenomSalarie = ".$prenom; try{ $stmt = $pdo->prepare($req); $stmt->execute(); } catch(Exception $e){ print "Erreur ! ".$e->getMessage(). "<br/>"; } //On le met à jour en fonction des informations rentrées avant de l'enregistrer dans la BDD //SI *propriété* est mise, alors appelle de la fonction Setter avec param, SINON on vide l'attribut pour éviter que le nouveau salarié récupère la propriété d'un précédent salarié $vide = ""; //Variable pour indiquer un champ non renseigné if (isset($_POST['dateVisiteMedicale'])) { $unSalarie->setDateVisiteMedicale($_POST['dateVisiteMedicale']); } else { $unSalarie->setDateVisiteMedicale($vide); } if (isset($_POST['fimo'])) { $unSalarie->setDateFIMO($_POST['fimo']); } else { $unSalarie->setDateFIMO($vide); } if (isset($_POST['fco'])) { $unSalarie->setDateFCO($_POST['fco']); } else{ $unSalarie->setDateFCO($vide); } if (isset($_POST['carte'])) { $unSalarie->setDateValiditeCarteConducteur($_POST['carte']); } else { $unSalarie->setDateValiditeCarteConducteur($vide); } if (isset($_POST['qualification'])) { $unSalarie->setDateQualificationConducteur($_POST['qualification']); } else { $unSalarie->setDateQualificationConducteur($vide); } if (isset($_POST['aipr'])) { $unSalarie->setAIPR($_POST['aipr']); } else { $unSalarie->setAIPR($vide); } if (isset($_POST['dateAIPR'])) { $unSalarie->setDateValiditeAIPR($_POST['dateAIPR']); } else { $unSalarie->setDateValiditeAIPR($vide); } $unSalarie->update($stmt); } //retour à la page salarie header("Location: ../page_salarie.php"); ?>
Je créer plus tôt le salarié puis je fait appelle à sa méthode Update pour mettre à jour le salarié fraîchement crée pour obtenir ce message d'erreur:

Pièce jointe 388001

Je constate que le problème initial est donc la syntaxe de la requête INSERT. Ferais-je mieux de partir sur deux méthodes "create()"? Car, dans mon contexte, il existe deux types de salariés: les chauffeurs qui possède les propriétés de date et les autres. Avoir un createChauffeur() qui prend en compte une syntaxe approprié pour les dates et un create() qui renseigne uniquement les propriétés non daté, valables pour tous les salariés ?

Citation:

Pour moi, il faut 2 classes : Salarie qui représente le salarié, avec nom, prénom, etc...) et SalarieDao (ou SalarieManager ou quelque chose qui genre) qui s'occupe de la communication avec la base.

Je ne vois pas très bien l'utilité de cette seconde classe ? SalarieDao serait la classe qui permet l’interaction avec la BDD de la classe salarie ?

12/06/2018, 09h58
CinePhil

Dans votre dernier code, il n'y a qu'une requête SELECT alors que l'erreur intervient sur une requête INSERT. :weird:

Donnez-nous le code de $unSalarie->create(); SVP.

Le code était présent dans le premier message mais le revoici en clair:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
	public function create(){
		include "../include/connexion.php";
		try{
			$req = "INSERT INTO salarie (numSalarie, numNiveau, numPoste, nomSalarie, prenomSalarie, dateFIMO, dateFCO, dateValiditeCarteConducteur, dateValiditeQualificationConducteur, dateVisiteMedicale, AIPR, dateValiditeAIPR) VALUES (NULL,'".$this->_niveauSalarie."','".$this->_posteSalarie."','".$this->_nomSalarie."','".$this->_prenomSalarie."','".$this->_dateFIMO."','".$this->_dateFCO."','".$this->_dateValiditeCarteConducteur."','".$this->_dateValiditeQualificationConducteur."','".$this->_dateVisiteMedicale."','".$this->_AIPR."','".$this->_dateValiditeAIPR."')";
			echo($req);
			$pdo->exec($req);
		}
		catch(Exception $e){
			print "Erreur ! ".$e->getMessage(). "<br/>";
		}
	}

12/06/2018, 10h28
CinePhil

OK alors, d'après les messages d'erreur reçus, le souci vient de la tentative d'insertion d'une date invalide.

C'est à cause du fait que les paramètres donnés à la requête ne sont pas préalablement contrôlés et probablement parce que les colonnes de date n'ont pas de valeur pas défaut.

Pouvez-vous nous donner le script SQL de création de la table, à l'aide de la requête ci-dessous ?

Code:

SHOW CREATE TABLE salarie
12/06/2018, 10h29
Invité
Bonjour,

1-

Citation:

....J'ai renseigné les colonnes à remplir avec une syntaxe normalement correcte:...

Non.
:alerte: Il faut imperativement une requête préparée

2- "NULL" / 'NULL' est une chaîne de caractères. Ce n'est pas NULL.
Remets $vide = "";.

3- optimisation
Remplace tous les
Code:

1 2 3 4 5 if (isset($_POST['dateVisiteMedicale'])) { $unSalarie->setDateVisiteMedicale($_POST['dateVisiteMedicale']); } else { $unSalarie->setDateVisiteMedicale($vide); }
Par des
Code:

1 2 $dateVisiteMedicale = ( isset($_POST['dateVisiteMedicale']) ) ? $_POST['dateVisiteMedicale'] : $vide; // (condition ternaire) $unSalarie->setDateVisiteMedicale( $dateVisiteMedicale );
4- Quant au message d'erreur, il indique la ligne 169.
Quel est le code correspondant (cette ligne 169 et précédentes) ?
12/06/2018, 10h34
MaitrePylos

Pièce jointe 388001

En 6ième position il attend une date, hors vous lui passez un champ vide !
Le message dit : Invalid Datetime format

Traitons un problème à la fois. On va d'abord s’occuper de ton insertion et on verra le select et l'update ensuite.
Pour l'insertion, je vois l'erreur "Incorrect date Value '' for column dateFIMO". En gros, tu essayes d'envoyer "" dans un champ de type DATE, ce qui n'est pas correct.
A ta place, je remplacerai par une requête préparée, comme ça c'est PDO qui s'occupera des formats.
Code:

1 2 3 4 5 6 7 8 9 10 11 public function create(){ include "../include/connexion.php"; try{ $req = "INSERT INTO salarie (numSalarie, numNiveau, numPoste, nomSalarie, prenomSalarie, dateFIMO, dateFCO, dateValiditeCarteConducteur, dateValiditeQualificationConducteur, dateVisiteMedicale, AIPR, dateValiditeAIPR) VALUES (NULL,?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)"; $stmt = $pdo->prepare($req); $stmt->execute([$this->_niveauSalarie, $this->_posteSalarie, $this->_nomSalarie, $this->_prenomSalarie, $this->_dateFIMO, $this->_dateFCO, $this->_dateValiditeCarteConducteur, $this->_dateValiditeQualificationConducteur, $this->_dateVisiteMedicale, $this->_AIPR, $this->_dateValiditeAIPR]); } catch(Exception $e){ print "Erreur ! ".$e->getMessage(). "<br/>"; } }
Et pense bien à initialiser les champs qui ne sont pas des VARCHAR avec null (ou tous les champs pour être tranquille ;))

Bien, j'ai pris le temps de lire vos réponses et de les mettre en pratique, voici ce que j'ai fait:

Modification de salarie_create.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
 
<?php 
include "../include/connexion.php";
//Si le nom du salarié est bien indiqué, ainsi que son prénom, son poste et son niveau de droit sur l'application
if (isset($_POST['nomSalarie']) && !empty($_POST['prenomSalarie']) && !empty($_POST['poste']) && !empty($_POST['droit'])) {
	$nom = $_POST['nomSalarie'];
	$prenom = $_POST['prenomSalarie'];
	$poste = $_POST['poste'];
	$niveau = $_POST['droit'];
	//On crée un nouveau salarié
	include "../Classe/salarie.php";
	$unSalarie = new Salarie ($nom, $prenom, $poste, $niveau);
 
	//On le met à jour en fonction des informations rentrées avant de l'enregistrer dans la BDD
	//SI *propriété* est mise, alors appelle de la fonction Setter avec param, SINON on vide l'attribut pour éviter que le nouveau salarié récupère la propriété d'un précédent salarié
	$vide = ""; //Variable pour indiquer un champ non renseigné
	$dateVisiteMedicale = ( isset($_POST['$dateVisiteMedicale']) ) ? $_POST['dateVisiteMedicale'] : $vide; // (condition ternaire)
	$unSalarie->setDateVisiteMedicale( $dateVisiteMedicale );
 
	$fimo = ( isset($_POST['fimo']) ) ? $_POST['fimo'] : $vide;
	$unSalarie->setDateFIMO($fimo);
 
	$fco = ( isset($_POST['fco']) ) ? $_POST['fco'] : $vide;
	$unSalarie->setDateFCO($fco);
 
	$carte = ( isset($_POST['carte']) ) ? $_POST['carte'] : $vide;
	$unSalarie->setDateValiditeCarteConducteur($carte);
 
	$qualification = ( isset($_POST['qualification']) ) ? $_POST['qualification'] : $vide;
	$unSalarie->setDateQualificationConducteur($qualification);
 
	$aipr = ( isset($_POST['aipr']) ) ? $_POST['aipr'] : $vide;
	$unSalarie->setAIPR($aipr);
 
	$dateAIPR = ( isset($_POST['dateAIPR']) ) ? $_POST['dateAIPR'] : $vide;	
	$unSalarie->setDateValiditeAIPR($dateAIPR);
 
	//Enregistrement dans la BDD
	$unSalarie->create();
}
 
//retour à la page salarie
header("Location: ../page_salarie.php");
 
?>

J'ai mis en place l'optimisation de jreaux62, qui est effectivement une belle optimisation, ainsi que le retour de $vide= "" . J'ai également retirer le "SELECT" pour repasser à l'ancien format: manipuler l'objet $unSalarie de manière à l'insérer dans la BDD sans avoir à l'update par la suite.

Alors que j'allais écrire ma requête préparé, j'ai vu le message de Celira et j'ai donc directement copié son code :P (Tout en le comprenant)

Pour CinePhil:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
CREATE TABLE `salarie` (
 `numSalarie` int(11) NOT NULL AUTO_INCREMENT,
 `numNiveau` int(11) NOT NULL,
 `numPoste` int(11) NOT NULL,
 `nomSalarie` char(32) NOT NULL,
 `prenomSalarie` varchar(128) NOT NULL,
 `dateFIMO` date DEFAULT NULL,
 `dateFCO` date DEFAULT NULL,
 `dateValiditeCarteConducteur` date DEFAULT NULL,
 `dateValiditeQualificationConducteur` date DEFAULT NULL,
 `dateVisiteMedicale` date DEFAULT NULL,
 `AIPR` varchar(32) DEFAULT NULL,
 `dateValiditeAIPR` date DEFAULT NULL,
 PRIMARY KEY (`numSalarie`),
 KEY `FK_salarie_niveauSalarie` (`numNiveau`),
 KEY `FK_salarie_posteSalarie` (`numPoste`)
) ENGINE=MyISAM AUTO_INCREMENT=6 DEFAULT CHARSET=latin1

Tout ce bazar proviendrait-il de mon DEFAULT CHARSET=latin1 ?

Résulat, je n'ai plus d'insertion possible (même lorsque tous les champs sont indiqués) et aucun message d'erreur visible.

1-

Citation:

"Incorrect date Value '' for column dateFIMO"

le format SQL pour DATE est : YYYY-MM-DD

Du coup, essaie de remplacer (pour toutes les dates) :
Code:

1 2 3 4 $videDate = "0000-00-00"; // format DATE $fimo = ( isset($_POST['fimo']) ) ? $_POST['fimo'] : $videDate; $unSalarie->setDateFIMO($fimo);
N.B. Perso, je n'utilise pas NULL (toutes mes colonnes sont en NOT NULL), car je trouve ça ~~ch....~~ pénible à utiliser dans une requête préparée.

2- Remets echo $req pour voir ce que ça donne.
Affiche aussi toutes les variables.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 public function create(){ include "../include/connexion.php"; try{ $req = "INSERT INTO salarie (numSalarie, numNiveau, numPoste, nomSalarie, prenomSalarie, dateFIMO, dateFCO, dateValiditeCarteConducteur, dateValiditeQualificationConducteur, dateVisiteMedicale, AIPR, dateValiditeAIPR) VALUES (NULL,?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)"; $stmt = $pdo->prepare($req); $params = [$this->_niveauSalarie, $this->_posteSalarie, $this->_nomSalarie, $this->_prenomSalarie, $this->_dateFIMO, $this->_dateFCO, $this->_dateValiditeCarteConducteur, $this->_dateValiditeQualificationConducteur, $this->_dateVisiteMedicale, $this->_AIPR, $this->_dateValiditeAIPR]; echo $req; var_dump($params); $stmt->execute( $params ); } catch(Exception $e){ print "Erreur ! ".$e->getMessage(). "<br/>"; } }
3- Concernant le CHARSET, rien à voir ici (à priori).
Par contre, le charset standard pour l'ensemble du site/bdd devrait plutôt être UTF-8.

Passez à l'UTF-8 sans manquer une étape

Si je peux me permettre pour la date, il crée la table en mettant par default : null

Donc il serait plus correcte de noter ceci
Code:

1 2 3 4 5 $videDate = null; // format DATE $fimo = ( isset($_POST['fimo']) ) ? $_POST['fimo'] : $videDate; $unSalarie->setDateFIMO($fimo);
Maintenant, si on veut mettre une date, le plus simple est de mettre une vraie date
Code:

1 2 3 4 5 $videDate = new DateTime(); // format DATE $fimo = ( isset($_POST['fimo']) ) ? $_POST['fimo'] : $videDate->format('Y-m-d'); $unSalarie->setDateFIMO($fimo);

2 pièce(s) jointe(s)

Citation:

Envoyé par jreaux62

3- Concernant le CHARSET, rien à voir ici (à priori).
Par contre, le charset standard pour l'ensemble du site/bdd devrait plutôt être UTF-8.

Oui, seul ma BDD était en latin 1, j'ai modifié le script SQL de manière à la passer en utf-8.

Après avoir rajouté cette variable $videDate = "0000-00-00", de nouveau je n'ai plus d'insertion (même si tous les champs sont renseignés) mais le traitement s'effectue sans l'insertion et sans m'afficher de message d'erreur. J'ai donc glissé une erreur afin d'accéder à ces messages que voici:

Pièce jointe 388097

On constate que seul la dateVisiteMedicale est correctement attribué ("0000-00-00") car j'ai en effet mal recopié ton code, j'ai ommis le "$" dans la condition...

Après correction, je reste sur même résultat: le traitement s'effectue mais toujours aucune insertion (formulaire entièrement remplis ou non), voici l'echo et var_dump:

Pièce jointe 388101

Ce que je constate:
-J'en renseigné la dernière date (array 10) mais elle a pourtant al valeur de $videDate
-Après un second test en spécifiant chaque date, le résultat reste le même pour les array() correspondant
-array(9) est vide alors que la "value" que j'ai mise dans la balise <option> est "Aucune"

Par acquis de conscience je met le code de la page salarie_create.php au cas où une erreur m'aurait échappé:
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 <?php include "../include/connexion.php"; //Si le nom du salarié est bien indiqué, ainsi que son prénom, son poste et son niveau de droit sur l'application if (isset($_POST['nomSalarie']) && !empty($_POST['prenomSalarie']) && !empty($_POST['poste']) && !empty($_POST['droit'])) { $nom = $_POST['nomSalarie']; $prenom = $_POST['prenomSalarie']; $poste = $_POST['poste']; $niveau = $_POST['droit']; //On crée un nouveau salarié include "../Classe/salarie.php"; $unSalarie = new Salarie ($nom, $prenom, $poste, $niveau); //On le met à jour en fonction des informations rentrées avant de l'enregistrer dans la BDD //SI *propriété* est mise, alors appelle de la fonction Setter avec param, SINON on vide l'attribut pour éviter que le nouveau salarié récupère la propriété d'un précédent salarié $dateVide = "0000-00-00"; //Variable pour indiquer un champ non renseigné de type date $vide = ""; //Variable pour indiquer un champ non renseigné $dateVisiteMedicale = ( isset($_POST['$dateVisiteMedicale']) ) ? $_POST['dateVisiteMedicale'] : $dateVide; // (condition ternaire) $unSalarie->setDateVisiteMedicale( $dateVisiteMedicale ); $fimo = ( isset($_POST['$fimo']) ) ? $_POST['fimo'] : $dateVide; $unSalarie->setDateFIMO($fimo); $fco = ( isset($_POST['$fco']) ) ? $_POST['fco'] : $dateVide; $unSalarie->setDateFCO($fco); $carte = ( isset($_POST['$carte']) ) ? $_POST['carte'] : $dateVide; $unSalarie->setDateValiditeCarteConducteur($carte); $qualification = ( isset($_POST['$qualification']) ) ? $_POST['qualification'] : $dateVide; $unSalarie->setDateQualificationConducteur($qualification); $aipr = ( isset($_POST['$aipr']) ) ? $_POST['aipr'] : $vide; $unSalarie->setAIPR($aipr); $dateAIPR = ( isset($_POST['$dateAIPR']) ) ? $_POST['dateAIPR'] : $dateVide; $unSalarie->setDateValiditeAIPR($dateAIPR); //Enregistrement dans la BDD $unSalarie->create(); } //retour à la page salarie header("Location: ../page_salarie.php"); ?>
Et merci de toutes vos réponses!

12/06/2018, 14h37
Invité

Code:

isset($_POST['$fimo'])

Il n'y a pas de $ ici ! (mea culpa : c'est moi qui avais fait l'erreur...)

Code:

isset($_POST['fimo'])

À modifier pour tous.
12/06/2018, 15h15
Skunka

4 pièce(s) jointe(s)

Citation:

Envoyé par jreaux62

Il n'y a pas de $ ici ! (mea culpa : c'est moi qui avais fait l'erreur...)

Cela me semblait bizarre tout de même! Enfin bref, après modification, je peux de nouveau faire une insertion SI, ET SEULEMENT SI, je renseigne tous les champs du formulaire. Je vaos donc reglisser mon erreur afin de voir la différence quand je renseigne chaque champ à quand certains champs sont restés vierges.

Tous les champs renseignés:

Pièce jointe 388152

Plusieurs champs vierges:

Pièce jointe 388153

Bon, là ça devient du n'importe quoi non? Les attributs de mon salariés ne sont pas mis à "0000-00-00" alors que j'ai justement retiré les $ qui est normalement une erreur ?:weird:
Je me demande ce qu'il se passerait si je modifie les condition ( isset($_POST['champ'])) par ($_POST['champ'] == "") ? $dateVide : $_POST['champ'] ? C'est à dire contrôler le contenu et non si la variable est remplie.

Pièce jointe 388155

On remarque que array(4) prend bien la valeur "0000-00-00", c'est donc la condition ternaire qui posait problème !
J'applique mes modifications au reste:

Pièce jointe 388167

Après un essaie, l'insertion sans renseigner tous les champs ne fonctionne toujours pas, la raison vient sûrement du fait que les dates sont en string (lenght 10)alors quand dans la BDD ce sont des attributs DATE (lenght 8).

EDIT: je viens de lire un message de MaitrePylos me parlant de mes dates. Il a senti le problème arriver il faut croire! Cependant, les dates non renseignés seront passé en format date de part ce qu'il à montré mais pour les dates qui sont déjà renseignés ?
12/06/2018, 15h24
MaitrePylos

Au risque de me répéter.
La date 0000-00-00 est incorecte : https://dev.mysql.com/doc/refman/5.7...o_zero_in_date
Il vaut mieux mettre null .
12/06/2018, 15h35
Skunka

Désolé, quand j'ai vu que tu indiquais de remettre $videDate = null , je n'ai pas directement remarqué qu'il n'y avait pas de "" ou '' , je m'en excuse!
J'ai mis en place la modification et effectivement cela fonctionne à merveille! Un premier merci pour m'avoir mis une documentation m'expliquant tes propos, elle est bien garnie!

Merci à tous ceux qui ont intervenu et m'ont apporté de l'aide, ce sujet est enfin résolu !
Très cordialement,
Skunka.