Return to Answer

Don't do that, you must separate presentation from logic.

Don't do that, you must separate presentation from logic.

You have some phpDoc comments, you need one for every function. But please note that I've been accused of overcommenting :)

PS. You should remove ?> from the end of every class script, as leo suggests.

You have some phpDoc comments, you need one for every function. But please note that I've been accused of overcommenting :)

PS. You should remove ?> from the end of every class script, as leo suggests.

you could help the IDE by adding the public keyword. That's also a readibillity point.

You have some phpDoc comments, you need one for every function. But please note that I've been accused of overcommenting :)

you could help the IDE by adding the public keyword. That's also a readibillity point and a PHP4 style (function foo()) vs PHP5 style (public function foo()).

You have some phpDoc comments, you need one for every function. But please note that I've been accused of overcommenting :)

PHP4 style class members

You are using some PHP4 style class members:

var $name = __CLASS__;
var $helpers = array();
var $validate = array();
var $posts = array();

You should be using PHP5 access control (public / private / protected) members everywhere.