Beste SD-ers,

Het was een lange dag en er was veel tijd mee gemoeid, maar daardoor zie ik mijn fout denk ik niet. Ik wil dat elke dag mijn database wordt op geschoond. Dit heb ik door middel van een cronjob gedaan. Echter de cronjob werkt volgens mij wel, maar het script niet. Het script pakt namelijk bij het updaten maar één enkel veld. Ik heb alles nu al drie keer nagelopen, maar kijk er echt gewoon overheen. Misschien iemand van jullie die nog helder genoeg is?! Hope so!!

PHP Code:

<?php
    // Make the time interval
    $cur_time = date('H:i');
    $min_time = '23:50';
    $max_time = '01:00';
    
    if($cur_time > $min_time || $cur_time < $max_time) {
        // The stats page can't be visited
        echo 'It\'s not the time yet';
    } else {
            include 'config.php';
            // Get the records
            $q = "SELECT * FROM moviestats";
            $sql = mysql_query($q) or die(mysql_error());
            $count = mysql_num_rows($sql);
                if($count == false) {
                    // There're no records
                    echo 'There\'re no movies yet';
                } else {
                    // There're records, update them
                    while($fetch = mysql_fetch_object($sql)) {
                        // Update the data fields                        
                        $q = "UPDATE moviestats
                                            SET
                                                ViewsToday = '0'
                                            WHERE
                                                MovieNumber = '".mysql_real_escape_string($fetch->MovieNumber)."'";
                        $sql = mysql_query($q) or die(mysql_error());
                    }
                }
    }
    

?>

Daarnaast laat hij ook nog eens zien: It's no time yet, maar het zit wel binnen dat marge qua tijd. Dus waarom klopt dit niet? Is er misschien een handige functie voor?

Hopelijk kan iemand me helpen.

Stefan

Ik weet het niet zeker, maar misschien dat de waarde van $sql wordt vervangen door de $sql in de while loop waardoor de while loop niet voor een tweede keer wordt uitgevoerd.
Zelf nummer ik de query's in dit soort situaties, meer voor de duidelijkheid voor mezelf, dus weet ook niet of dit daadwerkelijk het probleem is.

Overigens, 0 is een cijfer, oftewel hoeft deze niet tussen quotes bij de update query.
Zelfde geldt overigens voor movienumber.

EDIT:
Zie net dat hetgeen je eigenlijk doet, is alle records ophalen uit de moviestats tabel en bij elk record viewstoday op 0 zetten.
Maar dat kan veel simpeler en korter:
$query = "UPDATE moviestats SET ViewsToday = 0";

Klaar, dan doe je alle records in 1 keer.

@stefanvi: Scherp!! Dat was de oplossing. Ik heb van de onderste string een $q_u en $sql_u gemaakt en dat werkt. Allemaal vrij logisch, maar het werkt nu en daar ben ik blij om.

Ik zal de 0 tussen de haakjes weghalen. Is het heel erg als dit tussen de quotes staat? Of maakt het eigenlijk niet zoveel uit? MovieNumber is in dit geval een soort id met hoofdletters, kleine letters en cijfers, dus daar gaat het dan niet voor op. Maar dat kon je niet weten.

Hartelijk dank!!

Nog even een extra vraagje: Hoe krijg ik het bestand beveiligd? Dat alleen via een cronjob het script wordt uitgevoerd, maar dat een gewone gebruiker er niet bij kan?

Is het een optie om bijvoorbeeld het volgende te gebruiken:

PHP Code:

if(date('H:i') == 00:00) {
  // rest van de code
} 


Of zijn er slimmere opties?

Je kunt het script ook buiten je public_html zetten, dan kan je cronjob er nog wel bij, maar vanuit een browser kun je er niet meer bij. In een van de mappen voor je public_html bedoel ik dan.

Vaak heb je een structuur als
/[gebruikersnaam]/domains/[domein]/public_html

Meesten zetten hem dan in in de eerste map met de gebruikersnaam.

@Stefanvi: Dat zou een oplossing zijn. Het stomme is dat de cronjob niet wordt uitgevoerd. Ik ga eens kijken of een locatie verandering dat probleem verhelpt, maar denk van niet.

Waarschijnlijk geef je alleen het pad naar je script op, maar er moet meer bij. Probeer het eens met de volgende regel:

/usr/local/bin/php -q -f /home/gebruikersnaam/domains/domein.nl/public_html/cronjob.php

Waarbij je de gegevens even naar je eigen situatie aan moet passen uiteraard.

@stefanvi: Ook dat werkt niet. Het script behoort volgens mij gewoon toegankelijk te zijn. Heb de goede locatie ingevoerd, maar er gebeurt niets. Moet ik misschien bepaalde instellingen op de server hebben staan? Wat moet er aan staan om cronjobs uit te kunnen voeren?

Wat voor tijdsinterval heb je voor de cronjob ingevuld?

Minute: 30
Hour: 1
Day of the month: *
Month: *
Day of week: *
Path: /home/*gebruiker*/*domain*/cron/day-mov-stats.php

Heb deze geprobeerd en ook die stefanvi aangaf.

welk controlpanel heb je?
Directadmin / Cpanel / Plesk etc.

Direct Admin.

Heb ook al via google gezocht en daar stond het zoals jij het beschreef (misschien dat jij het daarvandaan hebt), maar dat werkte helaas ook niet ..

En je map opbouw is ook zeker weten correct?
Mis namelijk je domains map tussen gebruiker en *domain*
en cron staat ook echt in de *domain* map, en niet in de public_html?

Als je controleerd op ($curtime == 00:00) dan mag hij ook niet veel doen op het moment dat je de cronjob aanroept op 01:30. Als je dat al weg hebt gehaald heb ik even niets gezegd.

Vanuit zichzelf geeft direct admin het volgende weer:
/home/*gebruiker*/

*gebruiker* is hierbij bv: henk, dus /home/henk/
En het cronbestand staat echt in /home/henk/website.nl/cron/day-mov-stats.php
Aanvullend bericht:
@Joshua: Dat had ik al weg gehaald. Op een of andere manier wordt de cronjob niet uitgevoerd. Terwijl ik volgens mij alles goed had ingesteld.

Beste,

waarom het gebruik van een cronjob ?
Je kan het toch een bestand instellen dat controleer wanneer.

$q = mysql_query("SELECT date FROM cj WHERE id=1");
$r = mysql_fetch_array($q);

while(date('Y-m-d',$r['time']) <> date('Y-m-d')) {
include($day_conjob);
}

Zo ongeveer doe ik het :)

Hoop dat k je hier mee van dienst kan zijn.
En als je ook het verkoopt moet je dit ook niet ombouwen, namelijk niet alle ondersteunen cronjobs.

Mvg,
Aanvullend bericht:
oja zit wel foutje in hoor :D

while(date('Y-m-d',$r['time']) <> date('Y-m-d')) {
include($day_conjob);
$r['time'] += 86400;
}