[PEPr] Comment on Database::MDB2_ActiveRecord

Lukas Smith (http://pear.php.net/user/lsmith) has commented on the proposal for Database::MDB2_ActiveRecord. Comment: Some more comments: - using SELECT * is ugly IMHO, you should just list all of the relevant columns; this could also make it easier to mask/hide columns or add abstract columns (like CONCAT(..) ..) - it seems all columns are passed to the update method, even those that were not modified; this would make sense only if you would prepare the query only once and reuse it, but you are using autoExecute() that immediatly discards the prepared statement Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=368 -- Sent by PEPr, the automatic proposal system at http://pear.php.net