<ionut.g.stan@gmail.com> wrote:
> On 12/15/2008 17:04, till wrote:
>>
>> 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.
No offense meant. And I'm also not sure how well test coverage is, but
yeah -- unit tests should help you.
> 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.
Yeah, I think that's sort of what I had in mind. An optional parameter
shouldn't introduce any issues.
Till
没有评论:
发表评论