125195 - LDAP: implement "stop" for addressbook (quick search and advanced search)

Status: NEW

(MailNews Core :: Address Book, defect)

Description

[(not reading, please use [e-mail] instead)]

LDAP:  implement "stop" for addressbook (quick search and advanced search)

I think john @ sun has a patch for this in bug #83023

Comment 1

[yulian chang]

reassign QA contact to myself

Comment 2

[John Marmion]

Here was my original patch for bug #83023:

http://bugzilla.mozilla.org/attachment.cgi?id=54714

It obviously does not apply now, not only because it is out of date but more
seriously this was based on notification of an RDF object. But I believe the
general principles will apply. We need to cater for status like STARTED, STOPPED
and FINISHED which map to nsAbLDAPDirectory::StartSearch(), StopSearch() and
OnSearchFinished(). Executing StopSearch() does cause a query to cease. There
are two other observations: 

1. The original patch supported asynchronous searching/stopping which meant both
LDAP and Outlook. I notice Outlook functionality was lost following the landing
of the quick search feature. I have logged a bug #118119 and a corresponding
patch but there still appears to be a problem in the quick search of Outlook. It
is my difficulty withn the implementation of nsAbView in relation to Outlook
which stops me from progressing this further. Anyway, perhaps you might a chance
to have a look at that sometime. I appreciate that you are a particulary busy
and very productive at the momeent.

2. The current hardcoded limit of 100 LDAP entries returned makes this difficult
to test in a working LDAP Server. 

Comment 3

[Ninoschka Baca]

When this is implemented, will the Search button toggle between "Search" and 
"Stop" similar to the 3pane Advanced Search?

Comment 4

[Mark Banner (:standard8)]

*** Bug 124066 has been marked as a duplicate of this bug. ***

Comment 5

[Mark Banner (:standard8)]

*** Bug 125196 has been marked as a duplicate of this bug. ***