Direct Admin Web Error Log

Kan het zijn dat de query wordt uitgevoerd voordat de sessie is gestart?

Hij heeft het over de "$_SESSION['id']", de reden is waarschijnlijk dat er geen controle op zit of de id echt gezet is. Zet boven de query dit neer:

if(!isset($_SESSION['id'))
    $_SESSION['id'] = '';

Hierdoor zal error meteen verdwijnen, daarnaast de reden dat alles gewoon werkt omdat dit niet perse een error is, en dat staat er ook, het is een notice. Notices en warnings zijn geen errors en het programma draait dus gewoon door. Alleen bij fatal errors stopt de code.

De query "gaat goed" omdat deze qua syntax (vorm) goed is.

Alhoewel de volgende query "correct" is:

UPDATE klant
SET online=NOW()
WHERE id_klant=''
LIMIT 1

levert deze (waarschijnlijk) nooit een resultaat op. Ook zie ik niet helemaal waarom die LIMIT 1 er bij moet. Het kan geen kwaad, maar op deze manier krijg ik de indruk dat je er niet van uit kan gaan dat het goed gaat als je dit weglaat :). Dat zou inhouden dat je de informatie in je sessie niet helemaal kan vertrouwen, en als dat aan de orde is heb je andere problemen :).

En je krijgt de notice omdat $_SESSION['id'] niet bestaat. Ofwel omdat je sessie niet gestart is ten tijde van het uitvoeren van de query, of dat $_SESSION['id'] niet geinitieerd is. Dit laatste kan weer verschillende redenen hebben als:
- een gebruiker was niet ingelogd op het moment van uitvoeren van de query
- de sessie was verlopen bij het (opnieuw) opvragen van een pagina

Je zou dit inderdaad op kunnen lossen door op het bestaan van $_SESSION['id'] te controleren, maar die controle zelf hoort waarschijnlijk op een andere plaats thuis, en niet direct voor de uitvoer van de query... Bijvoorbeeld bij het laden van de pagina - daar had je waarschijnlijk al ingelogd moeten zijn om de pagina te mogen zien, dat (bij het opvragen van de pagina) zou dan ook een geschiktere plaats moeten zijn als je het mij vraagt.

Headers zijn onderdeel van output. Deze worden pas verstuurd op het moment dat er output wordt geproduceerd.

Er ontbreekt een exit in dat if-statement. ALLE code die daarna nog volgt wordt dus gewoon uitgevoerd. Zoals ik al eerder heb beschreven: header('Location: ...') transporteert je niet direct automagisch naar een nieuwe locatie.

Ik denk dat het de moeite loont als je je code controleert op dit soort constructies. Wat mogelijk nog beter is is dat je een redirect-functie of methode maakt, zodat je de "exit" nooit meer vergeet. Daarbij wordt je code ook nog eens een stuk korter:

<?php
function redirect($link) {
    header('HTTP/1.1 303 See Other');
    header('Location: '.$link);
    exit;
}
?>

ik kan dit ook niet vaak genoeg zeggen en benadrukken: het verschil tussen het plaatsen of ontbreken van een exit-statement kan het verschil betekenen tussen een veilige of onveilige applicatie.

Ik zou zeggen, dump je sessie-data eens naar je scherm:

<?php
// helper function; assumption: everything is UTF-8 encoded
function dump($a) {
    echo '<pre>'.htmlspecialchars(print_r($a, true), ENT_QUOTES, 'UTF-8').'</pre>';
}


// otherwise there is no point dumping $_SESSION
session_start();


// dump $_SESSION
dump($_SESSION);
?>

isset() retourneert ook false als de waarde van de variabele NULL IS. Let er op dat er een NOT-operator (het uitroepteken) voor je isset() statement staat.

Voor de leesbaarheid zou je ook kunnen overwegen om ofwel het volgende te gebruiken:

<?php
if (isset($a) === false) {
    echo '$a is not set';
} else {
    echo '$a is set';
?>

Of je draait de delen om:

<?php
if (isset($a)) {
    echo '$a is set';
} else {
    echo '$a is not set';
?>

Misschien is dat nog beter: gebruik zo min mogelijk operators waardoor je je mogelijk kunt vergissen.

Of gebruik te maken van empty, maar je moet dan ook goed opletten dat de variabele ook als "leeg" wordt beschouwd als de waarde van de variabele een "lege" waarde heeft (de lege string, het cijfer 0 (al dan niet tussen quotes), false of NULL).

<?php
if (empty($a)) {
    echo '$a is empty';
} else {
    echo '$a is not empty';
}
?>