Closed Bug 1635923 Opened 5 years ago Closed 5 years ago

Implement multiword search for address book sidebar / select addresses dialog

Categories

(SeaMonkey :: MailNews: Address Book & Contacts, enhancement)

enhancement

Tracking

(seamonkey2.53+ fixed, seamonkey2.57esr? affected)

RESOLVED FIXED
seamonkey 2.75
Tracking Status
seamonkey2.53 + fixed
seamonkey2.57esr ? affected

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

References

Details

(Whiteboard: SM2.53.3)

Attachments

(1 file)

STR (Taken from Bug 1000775)

  1. have AB contact card with
  • Display name: "Doe, John" (without quotes)
  • email: "john@asdf.com"
  1. compose new msg, F9 to open contacts side bar for step 3b)
    3a) in recipient pane with autocompletion, type:
    John Doe
    3b) in AB quick search (address book sidebar or Options -> Select Addresses), type:
    John Doe

Actual result
3a) recipient autocomplete uses john AND doe algorithm, hence successfully finds "Doe, John" (good)
3b) AB quick search uses john doe algorithm, hence finds nothing (this bug)

Expected result
3b) Should be the same as in 3a, "Doe, John" contact should be found and offered in the AB quick search results list. E.g. for intuitive basic searches like "John Doe", all matching cards should be found regardless where exactly in any field these words are present or contained.

At the same we should implement a mechanism (e.g. quoting) to search for whole string as typed, including spaces, so that "word1 word2" matches all cards what have "word1 word2" in one of their searched fields.

Port the relevant parts of the following bugs to SeaMonkey:

  • Bug 1000775 - For AB Quicksearch / contacts side bar, implement split multiword search pattern
  • Bug 1091675 - Implement quoting to disable multi-word search in addressbook

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: inconsistent search behaviour
Testing completed (on m-c, etc.): 2.53.3
Risk to taking this patch (and alternatives if risky): none
String changes made by this patch: none

Attachment #9146310 - Flags: review?(frgrahl)
Attachment #9146310 - Flags: approval-comm-release?
Attachment #9146310 - Flags: approval-comm-esr60?
Blocks: 1636098
Comment on attachment 9146310 [details] [diff] [review] Implement AB multiword search LGTM
Attachment #9146310 - Flags: review?(frgrahl)
Attachment #9146310 - Flags: review+
Attachment #9146310 - Flags: approval-comm-release?
Attachment #9146310 - Flags: approval-comm-release+
Attachment #9146310 - Flags: approval-comm-esr60?
Attachment #9146310 - Flags: approval-comm-esr60+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/f349036e7311
Implement multiword search for address book sidebar / select addresses dialog. r=frg

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey 2.75
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: