Re: [evolution-patches] patch for #58701 (addressbook)
- From: Chris Toshok <toshok ximian com>
- To: evolution-patches lists ximian com
- Subject: Re: [evolution-patches] patch for #58701 (addressbook)
- Date: Sat, 17 Jul 2004 17:28:34 -0700
On Sat, 2004-07-17 at 11:14 +0530, Sivaiah N wrote:
> Ok, So in case of *_start_book_view call i don't think it matters
> whether backends use e_book_backend_sexp or not as anyway
> e_data_book_view_notify_update does the matching by calling
> e_book_backend_sexp_match_contact before updating the view
Hmm, actually it seems silly to use e_data_book_view_notify_update, as
the only thing that can ever happen is an _add from start_book_view, but
that's beside the point I suppose.
> Groupwise backend does the things similar to file backend both in
> start_book_view and get_contact_list methods.
How does it work? iterate over all contacts on the server? or does it
generate a query?
> Exchange and Ldap backends seems not calling
> e_book_backend_sexp_match_contact in get_contact_list method
Yeah, the sexp stuff in the backends that require it (to translate an
evolution sexp into something the server can handle) should really be
rich enough to represent queries over any fields, unless the server
doesn't support such queries. In those cases it makes sense to use a
more general query and call e_book_backend_sexp_match_contact.
> And now that i see i should move e_contact_address_free inside if to
> make sure that i don't leak address in any case.
Yeah, just before the "if (rv)" check.
> Ok to commit this ?
Yeah, we can always go back later and fix the uses of
> On Fri, 2004-07-16 at 22:55, Chris Toshok wrote:
> > looks good here, although this only works for the backends that actually
> > use e-book-backend-sexp.c (file/vcf). I can't remember if the
> > exchange/ldap/gw backends do things properly.
> > Chris
> > On Fri, 2004-07-16 at 19:53 +0530, Sivaiah N wrote:
> > > Hi,
> > > The attached patch implements compare_address function in
> > > e-book-backend-sexp.c so that search can happen overs values present in
> > > address files in contact
> > > Thanks,
> > > Siva
> > >
> > >
> > >
> > _______________________________________________
> > evolution-patches mailing list
> > evolution-patches lists ximian com
> > http://lists.ximian.com/mailman/listinfo/evolution-patches
> evolution-patches mailing list
> evolution-patches lists ximian com
] [Thread Prev