> Maybe generalize the patch and try add support to
> Zend_Db_Table_Abstract without breaking BC (= the existing API).
> Submit the patch on the issue tracker, you'll need to sign a CLA for
> this.
>
> Till
>
I know what BC is and I think the unit tests accompanying ZF would help
me a lot in not breaking anything. I'm curious though what were your
thoughts when you said to generalize. Did you have something specific in
mind?
The only means I thought I could generalize, is to actually add a
second, optional parameter to Zend_Db_Select which would specify DB
fields to be returned. This way it will only be a matter of passing some
arguments from Zend_Db_Table_Abstract::select(), to
Zend_Db_Table_Select::__construct() and finally
Zend_Db_Select::__construct(). Anyway, these are just assumptions as I
haven't studied the code base that well.
没有评论:
发表评论