Discussion :

[Chrisvetyver]

Bonjour à tous,

Il y a eu une mise à jour sur mon serveur. Voici la version : 5.7.25. Mon site fonctionnait très bien avant cette nouvelle version que m'a mis l'hébergeur.
Depuis j'ai un soucis avec la date. Voici le message d'erreur :

errno : 1292
erreur : Incorrect date value: '--' for column 'Societe_DateCreation' at row 1

Malgré ce message d'erreur, tout le reste s'enregistre dans la BDD ! Et si je mets une date dans le champs, alors l'erreur ne s'affiche pas !
J'ai fait certaines recherches et malheureusement je n'arrive pas à comprendre ce que je dois changer. Apparemment c'est lorsque le champs date est vide que le problème se pose.
Actuellement j'ai ceci dans ma colonne Societe_DateCreation : Null = non - Value : 0000-00-00

Dans le formulaire j'ai ceci :

Code :

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

1
2
<label>Date de cr&eacute;ation :</label><input name="date_de_creation" id="date_de_creation" value="<?php echo @date4html($societe["Societe_DateCreation"]); ?>" /> <img src="images/calendar.gif" alt="Calendrier" onclick="displayCalendar(document.getElementById('date_de_creation'),'dd/mm/yyyy',this)" align="middle" style="cursor:pointer" /><br />
<br />

Pourtant j'ai bien la date qui s'affiche sous la forme : jj-mm-aaaa
En effet, j'ai ceci dans ma page création de ma fiche :

Code :

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

$sql_new.= ', "'.date4sql($_POST["date_de_creation"]).'"';

Peut-être la solution serait de mettre NULL = oui dans ma colonne mais j'ai peur que cela modifie tout le reste !
Soyez indulgent avec moi car j'ai appris sur le tas la programmation

Merci par avance de vos réponse.

[rawsrc]

Salut,

si le champ peut être effectivement vide en base de données alors oui, c'est une bonne idée d'autoriser la valeur NULL.
Ensuite, si la valeur reçue dans le $_POST est vide alors

Code :

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

$sql_new.= ', NULL';

[Chrisvetyver]

Bonjour Rawsrc,

Merci pour ta réponse. Je voulais savoir, que penses-tu si je mets ceci :

Code :

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

1
2
3
4
5
 
	if($_POST["date_de_creation"] <> "")
		$sql_new.= ', "'.date4sql($_POST["date_de_creation"]).'"';
	else
		$sql_new.= ', NULL';

Penses-tu que ce soit judicieux ? Ou cela peut poser un problème ?

[rawsrc]

Salut,

opte plutôt pour empty() :

Code :

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

$sql_new .= empty($_POST["date_de_creation"]) ? ', NULL' : ', "'.date4sql($_POST["date_de_creation"]).'"';

et si la date est invalide, tu fais comment ?

Je pense qu'il faille que ta fonction date4sql() fasse tous les cas de figure et te renvoie la valeur pour le SQL
Quelque chose dans ce genre :

Code :

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

1
2
3
4
5
6
function date4sql($p) {
    $date = empty($p) ? false : new \DateTime($p);
    return ($date === false) ? 'NULL' : '"'.$date->format('Y-m-d').'"';
}
 
$sql_new .=  ', '.date4sql($_POST['date_de_creation'] ?? false);