On line 202 in Muuttaa_Process, it reads:
$errors = $this->db->getCol($sql, 0, array(
looking through the rest of the code, it seems to use $this->db() instead. I'm not sure if this was intended or not.
Thanks for releasing this code from digg, it's very insightful!
Comment #1
Posted on Jan 7, 2009 by Helpful DogThis is a bug (typo?), and I've fixed it in r108.
It doesn’t cause problems in this case; the $this->db()
method sets the $this->db
member, so it doesn’t trigger a function call on non-object fatal error like it would
if db()
kept the DB instance in a function-local static variable.
Nevertheless, it’s incorrect and should be fixed.
Status: Fixed
Labels:
Type-Defect
Priority-Medium