De vaste lezers weten het inmiddels wel dat ik iedere dag de fouten-log van de website bekijk om te kijken of alles soepel verloopt en er geen gekke dingen gebeuren. Gebeuren er gekke dingen dan is het meestal het resultaat van een bug, groot of klein, waar naar gekeken moet worden.

Zojuist zag ik weer een hele rits meldingen in de log staan, die me de wenkbrauwen doen fronsen:


Deze melding heb ik wel vaker voorbij zien komen en ik ben er ook wel in gedoken om te achterhalen waar het nu precies fout gaat. In de betreffende regel (86) wordt alleen een kleine berekening gemaakt. Natuurlijk probeer ik dan zelf in dit geval ook zelf of ik de fout zelf kan veroorzaken, maar mij lukte dat niet. Als je de website op een normale manier gebruikt dan gaat alles goed en is er niks aan de hand.

Wat ik me kan bedenken is dat iemand moedwillig verkeerde waarden aan bepaalde parameters aan een pagina probeert mee te geven in de hoop dat het systeem onderuit gaat. In dit geval wordt er blijkbaar ergens een niet nummerieke waarde meegegeven in plaats van alleen een cijfer.

Helaas voor degene die het probeerde, MultiSite gaat echt niet zo snel onderuit, maar geeft mij wel de nodige meldingen terug in de logs. Ik schrik toch altijd wel van dergelijke meldingen en ban dan echt wel scherp en vraag me af, wat men probeerde te doen, want dat kan ik helaas niet zien. Maar het is aan mij wel de taak om dit nu eindelijk eens aan te gaan pakken, zodat dit soort meldingen tot het verleden behoren.

Dus ik ben wat dieper in de code gaan duiken en zag dat er geen controle werd uitgevoerd of er een nummerieke waarde wordt meegegeven of niet. De code van de paginator heb ik ooit eens van het internet geplukt en naar mijn idee aangepast, zodat ie breed inzetbaar is op de hele website. Het is dus een universele functie, of eigenlijk een class met verschillende functies.

Er werd wel gecontroleerd of een bepaalde variabele is gezet en dus wordt meegegeven, maar er werd niet gekeken naar de inhoud daarvan. Slordigheidje van de originale programmeur of van mij? Dat laat ik maar even in het midden.

Intussen heb ik he stukje van 3 regels code vervangen voor de onderstaande regel:


Deze regel doet 3 dingen ineens, het checkt of de variabele 'p' aanwezig is en of deze een nummerieke waarde heeft en maakt er een integer van en dat is een nummerieke waarde. Wordt daar niet aan voldaan dan is het standaard antwoord een 1.

Met andere woorden: als men loopt te klooien, zal het antwoord altijd 1 zijn en blijft alles gewoon doorlopen zonder meldingen in de logs. Dus MultiSite is alweer wat beter en robuuster geworden.

Nu kan ik me weer gaan richten op de echte problemen en die zijn er momenteel niet, tenminste niet in de logs. Dus eigenlijk kan ik me nu verder gaan richten om het verder verbeteren van MultiSite door middel van uitbreidingen, zoals de ratings bij de verschillende onderdelen van de website.