[SearchBuilder-devel] [PATCH] Filters v2

Jesse Vincent jesse at bestpractical.com
Fri Sep 23 16:54:20 EDT 2005




On Sat, Sep 24, 2005 at 12:47:47AM +0400, Ruslan Zakirov wrote:
> Hello.
> 
> Here is filters v2. I've done some major changes to __set method to
> fix some issues:
> 
> 1) __set(value => undef) now works and set column to NULL
> 2) __set() never calls __value because method returns decoded variant.
> I think all wil agree with me that we should compare only encoded
> values, so if value is not fetched from DB or value is allready
> decoded then we don't check eqality of the old and new values.

if the unencoded values are identical (same object, etc), why should we
go through the overhead of encoding both?

> 
> 


More information about the SearchBuilder-devel mailing list