Discussion :

[laurentSc]

Bonjour,

grâce à cette discussion, je peux tester l'effacement...et constater qu'il ne marche pas

Pourquoi ?

Code php :

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

1
2
3
4
5
//effacer les enregistrements qui ont plus d'un an
$time_1_an=time()-365*24*3600;
$req='DELETE from visites where (timestamp<'.$time_1_an.")";
echo 'req:'.$req.'<br/>';
$result=mysqli_query($db,$req);

[sabotage]

Tu as une colonne qui s'appelle "timestamp" ?

[laurentSc]

C'était bien le cas ; comme je me suis dit que ça risquait d'être un mot-clé et qu'il y avait peut-être un problème, je l'ai renommé en "timestamp1" mais le problème reste...

[sabotage]

Il est effectivement une mauvaise d'utiliser les mots reservés mysql comme nom d'objet.
Un format date offre plus de possibilité qu'un timestamp. Tu peux cependant faire avec le timestamp directement le calcul dans la requête. Cela permet au passage de prendre en compte les années bissextiles.

Code SQL :

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

DELETE FROM visites WHERE `timestamp` < UNIX_TIMESTAMP(DATE_SUB(CURDATE(), INTERVAL 1 YEAR))";

Ca c'etait purement pour la partie écriture.
Pour le fond de ton problème, quand on a problème sur une requête il faut evidemment afficher/consulter les erreurs SQL.

[laurentSc]

Pour l'affichage des erreurs SQL (je ne le faisais pas), j'ai rajouté

Code php :

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

1
2
if (!mysqli_error()) echo "pas d'erreur<br/>";
else echo mysqli_error()."<br/>";

et ta requête ne fonctionne pas...et pourtant en enlevant le " en trop, je comprends pas...

Code sql :

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

DELETE FROM visites WHERE (`timestamp1` < UNIX_TIMESTAMP(DATE_SUB(CURDATE(), INTERVAL 1 YEAR)));

[sabotage]

Qu'obtiens-tu dans ton traitement d'erreur ?

[laurentSc]

Justement : "pas d'erreur" ; ce qui semble signifier que la requête est correcte sauf qu'elle ne fait rien...

[sabotage]

Pour debuguer il suffit d'executer le SELECT correspondant dans Phpmyadmin :

Code sql :

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

SELECT * FROM visites WHERE `timestamp1` < UNIX_TIMESTAMP(DATE_SUB(CURDATE(), INTERVAL 1 YEAR))

Tu n'auras tres probablement aucun retour ce qui signifie que tu n'as aucune entrée de plus d'un an.
"Timestamp" est bien une colonne contenant des timestamp unix ?

[laurentSc]

"Timestamp" est bien une colonne contenant des timestamp unix ?

Le problème semble être là, car quand je fais SELECT * from visites, il me retourne des enregistrements du type ( 82.122.12.106,2000-01-01 00:00:00) donc ce n'est pas le format unix-timestamp, alors que la requête d'insertion (je l'affiche) est INSERT INTO visites (ip,timestamp1) VALUES('82.122.12.106','1407613896') ; comment se fait-il ?

[sabotage]

Quel est le type de la colonne ?

[laurentSc]

datetime

[sabotage]

DATETIME c'est une date, pas un timestamp
L'un et l'autre sont valable mais il faut faire pareil partout.

[laurentSc]

OUPS...Comme tout mon code est fait pour manipuler des timestamp, il était plus rapide de changer le type de la colonne ; c'est fait. Par contre, j'ai un code censé insérer une date (au format "timestamp" ) dans la bdd ; il n'insère rien et je comprends pas :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
$date1 = new DateTime($_POST['year'].'-'.$_POST['month'].'-'.$_POST['day']);
echo "<br/>date=".$date1."<br/>";
if ($date === false)
    {
        echo 'Date invalide';
    }
else
    {
	$sql = "INSERT INTO visites (`timestamp1`) VALUES ('".strtotime($date1)."')";
	echo "<br/>".$sql."<br/>";
        $qry = mysqli_query($db, $sql);       
    }

[sabotage]

$date1 est un objet PHP Datetime, tu ne peux pas l'utiliser tel quel avec strtotime.
De plus tu as un $date qui se ballade aussi/

[laurentSc]

Merci pour le $date qui se ballade ; c'est corrigé.

Pour le strtotime, bien que j'ai lu

Plus simple, rapide, intégré, toussa : utiliser strtotime tout simplement.
Merci à Éric Daspet et à Oncle Tom pour le rappel.

Une fonction utilitaire1 pour convertir une chaîne de type DATETIME ou DATE en un timestamp UNIX.
Les formats pris en charges en entrée sont AAAA-MM-JJ HH:MM:SS et AA-MM-JJ HH:MM:SS. L’heure étant optionnelle dans le cas de la conversion d’une date.

ici, je peux peut-être faire $date1->getTimestamp() mais en fait, ça bugue même avant :

Code :

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

1
2
3
4
 
echo "<br/>test2<br/>";
$date1 = new DateTime($_POST['year'].'-'.$_POST['month'].'-'.$_POST['day']);
echo "<br/>date=".$date1."<br/>";

test2 est affiché, mais pas $date1 ; pourquoi ?

[sabotage]

Dans ton article, ils parlent d'une chaine DATETIME ("2014-08-12 12:00:24"), pas d'un objet PHP Datetime (que tu obtiens avec New DATETIME)

Tu peux faire simplement :

Code :

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

1
2
3
if (checkdate($_POST['month'], $_POST['day'], $_POST['year'])) {
    $timestamp = strtotime($_POST['year'].'-'.$_POST['month'].'-'.$_POST['day']);
}

[laurentSc]

OK, grâce à ton aide, $date1 s'affiche et comme elle est au format timestamp, j'ai modifié la requête d'insertion, mais la valeur insérée est 0000-00-00 00:00:00 . Il y a donc encore un problème ; vois-tu lequel ?

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
 
    echo "<br/>test2<br/>";
    if (checkdate($_POST['month'], $_POST['day'], $_POST['year'])) 
    {
        $date1 = strtotime($_POST['year'].'-'.$_POST['month'].'-'.$_POST['day']);
    }
    echo "<br/>date=".$date1."<br/>";
    if ($date1 === false)
    {
        echo 'Date invalide';
    }
    else
    {
        $sql = "INSERT INTO visites (`timestamp1`) VALUES ('".$date1."')";
        echo "<br/>".$sql."<br/>";
        $qry = mysqli_query($db, $sql);       
    }

[sabotage]

Ta colonne est toujours au format DATETIME.

[laurentSc]

Ben non, timestamp...

[sabotage]

TIMESTAMP est un type DATETIME spécial, pas un timestamp unix.
Tu peux utiliser INT.