Quote:
Originally Posted by xentek
@mgordon
Well, if you need to move from find to fetchAll, the amount of refactoring would be kind of low, since you would already have to modify it a bit to do the join anyway.
|
@xentek
I still don't get the downside of including the table name, don't all the databases accept that kind of very basic queries?
I do have a problem with switching to fetchAll since you then can argue: why have a find() at all? I think this small change would allow a higher flexibility; the manual mentions joins but not much of all the functions that are affected by this change. Both the find() and the _getWhereQuery() in the table row are affected by it (probably even more)...