[Zope-dev] ZCatalog FieldIndex Bug (Fix?)

[Chris McDonough]

You can run them by doing "python testCatalog.py -p"... some will fail due
to not having various files on the filesystem, but most should pass.

I haven't had time to actually even look at the bug in the bugreport.
Thanks for the possible fix to the bug that I don't understand yet, however!
;-)

----- Original Message -----
From: "Chris Withers" <chrisw@nipltd.com>
To: <zope-dev@zope.org>; "Chris McDonough" <chrism@digicool.com>
Sent: Thursday, April 05, 2001 6:41 PM
Subject: [Zope-dev] ZCatalog FieldIndex Bug (Fix?)


> Chris Withers wrote:
> > Correct me if I'm wrong, but if I query a field index with an empty
string, I
> > expect to get returned all catalogued objects where that attribute's
value _is_
> > the empty string.
> >
> > Anyway, this happens most of the time and I wish it happened all the
time.
> > Basically, the only situation where this _doesn't_ happen is when _none_
of the
> > objects catalogued in the field index have a value of the empty string,
which
> > seems really bug and nasty to me ;-)
>
> I think I've got a solution, but I dunno if it's right. The patch is at
the end.
> How do I get the unit tests in ZCatalog/tests to run so I can check I
haven't
> screwed anything else up?
>
> cheers,
>
> Chris
>
> E:\Zope\2.3.1\lib\python\SearchIndex>diff -u unindex.py.old unindex.py
> --- unindex.py.old      Thu Apr 05 23:35:40 2001
> +++ unindex.py  Thu Apr 05 23:37:46 2001
> @@ -407,9 +407,9 @@
>
>          if r is None:
>              if anyTrue:
> -                r=IISet()
> -            else:
>                  return None
> +            else:
> +                r=IISet()
>
>          return r, (id,)
>
>
> _______________________________________________
> Zope-Dev maillist  -  Zope-Dev@zope.org
> http://lists.zope.org/mailman/listinfo/zope-dev
> **  No cross posts or HTML encoding!  **
> (Related lists -
>  http://lists.zope.org/mailman/listinfo/zope-announce
>  http://lists.zope.org/mailman/listinfo/zope )
>