Verwijder rows na 7 dagen
Ik heb lang zitten googlen en kwam uit op deze code:
Code (php)
1
2
3
4
5
6
7
2
3
4
5
6
7
<?php
$del = mysql_query("DELETE FROM sfi_lost_pass WHERE date + INTERVAL 7 DAY < CURDATE()");
if($del) {
Echo mysql_error();
}
?>
$del = mysql_query("DELETE FROM sfi_lost_pass WHERE date + INTERVAL 7 DAY < CURDATE()");
if($del) {
Echo mysql_error();
}
?>
Het is de bedoeling dat hij kijkt naar codes die ouder zijn als 7 dagen en vervolgens ze verwijderd.
Maar hij doet niks.
Het veld Date is een datetime veld en kan hem niet naar date zetten in verband met dat er een code mee geactiveert word die maar precies 24u geldig is.
Error reporting weergeeft niks.
Fouthandeling weergeeft niks.
Weet iemand wat ik fout doe?
Gewijzigd op 11/03/2011 20:30:21 door Jordi Kroon
Zijn er verder wel records die dan ook effectief ouder zijn dan 7 dagen?
Toevoeging op 11/03/2011 20:30:05:
@writedown ik heb hem op curdate gezet maar zie nog geen verschil
Waarom verwijderen?
Ik wil een melding geven als de link verlopen is maar ik wil ze ook niet oneindig lang in de database laten staan. Vandaar dat ik die melding maar 7dagen geef en daarna gaat die row de prullenbak in
WHERE 2012+7 < huidige datum
Wanneer kan dat, dat kan als we al 2012 zijn! Zie je de fout?
@Karl
Waarom zou je onnodige data in de database laten staan als er niets meer mee gebeurd? Is toch onnodig?
Oh nu zie ik het ja domme fout van mij. Dat krijg je na een dagje zwemmen word je helemaal moe van en ga je andere dingen zien :)
Bwa, iedereen heeft dat wel eens voor. Ik heb soms ook al een script herschreven omdat ik de fout niet vond. En dan kom je eigenlijk tot het besef dat je de fout over het hoofd hebt gezien :P
Je kunt dit gebruiken voor statistieken...
Gewijzigd op 11/03/2011 20:55:47 door Jordi Kroon
Karl Karl op 11/03/2011 20:55:10:
Je kunt dit gebruiken voor statistieken...
Dan denk ik dat het interessanter is een aparte tabel te gebruiken. Maar goed, het doet er niet toe. De TS wil de data niet gebruiken, dus is hij nutteloos.
@karl Ik heb er een beveiliging op gemaakt dat er maar 1 link per dag (per account) kan worden aangevraagd. Ongeacht of hij ook daadwerkelijk is gereset. Vandaar dat ik dit niet gebruik voor statistieken
Write Down op 11/03/2011 21:01:08:
Dan denk ik dat het interessanter is een aparte tabel te gebruiken. Maar goed, het doet er niet toe. De TS wil de data niet gebruiken, dus is hij nutteloos.
Karl Karl op 11/03/2011 20:55:10:
Je kunt dit gebruiken voor statistieken...
Dan denk ik dat het interessanter is een aparte tabel te gebruiken. Maar goed, het doet er niet toe. De TS wil de data niet gebruiken, dus is hij nutteloos.
Hoezo een aparte tabel? Dat lijkt mij niet logisch