View Single Post
Old 11-29-2007, 05:19 PM   #5 (permalink)
sketchMedia
The Prestige
Advanced Programmer Top Contributor Good Samaritan 
 
sketchMedia's Avatar
 
Join Date: Oct 2007
Location: Manchester, UK
Posts: 854
Thanks: 32
sketchMedia is on a distinguished road
Default

The only thing i can see is the 'var' keyword, which in php5 was deprecated in favor of visibility scope identifiers (as pointed out by wildhoney) but left in php5 for backward compatibility.

Also as Karl says your core does seem a little confused about its purpose, i.e. having the
HTML Code:
MySQLSecure($text)
function inside the core, whereas it could be in the mysql class and also templating functions within the core too, but a good start I'd say
__________________
mysql> SELECT * FROM `users` WHERE `users`.`clue` > 0;
Empty set (0.00 sec)
sketchMedia is offline  
Reply With Quote