252759 - Crash when doing multiple advanced address book searches

Status: RESOLVED FIXED

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

Description

[Stefan Borggraefe]

MailNews suffers from the same crash as TB did. See bug 251605.

Comment 1

[Stefan Borggraefe]

Attachment: Port of the Thunderbird Fix

Comment 2

[[e-mail]]

Comment on attachment 154097 [details] [diff] [review]
Port of the Thunderbird Fix

I think that the sort column and direction should be retained across searches.
Checking against the message search, I notice that bienvenu broke it over a
year ago - the indicators no longer appear :-(

Comment 3

[Stefan Borggraefe]

Attachment: Hopefully better patch

Ok, this patch retains the sort order and fixes the crash. I tested the Address
Book window itself, the Advanced Address Book Search dialog and the Select
Addresses dialog with different profiles (including newly created profiles with
no address book entries).

Comment 4

[Stefan Borggraefe]

Comment on attachment 156134 [details] [diff] [review]
Hopefully better patch

Asking for approval. This fixes a crasher in the advanced address book search
and makes sure the sort order of the columns is retained after a search
operation. Should be pretty safe.

Comment 5

[Asa Dotzler [:asa]]

Comment on attachment 156134 [details] [diff] [review]
Hopefully better patch

a=asa for checkin to 1.8a3

Comment 6

[Stefan Borggraefe]

Fix checked in.

Comment 7

[R.K.Aa.]

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