Closed Bug 312598 Opened 19 years ago Closed 19 years ago

Duplicated thread in threaded view of virtual folder and quick search

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: u69748, Assigned: Bienvenu)

Details

(Keywords: fixed1.8)

Attachments

(3 files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b5) Gecko/20051015 Firefox/1.4.1
Build Identifier: version 1.5 Beta 2 (20051015)

I see Duplicated thread in threaded virtual folder.

Ex.
+ message1
  + message1-1
  + message1-2
+ message1
  + message1-1
  + message1-2


Reproducible: Always

Steps to Reproduce:
Attached image screen shot
This virtual folder is created form Inbox.
See Order Received column.
Flags: blocking1.8rc1?
This looks exactly like the screenshot in bug 270873. See also bug 178999, bug
194444, and bug 154403.
Version: unspecified → 1.5
I can see same problem in threaded view of quick search in Inbox.

Note: Compact folder and rebuilding .msf did not solve this problem.
Summary: Duplicated thread in threaded virtual folder → Duplicated thread in threaded view of virtual folder and quick search
Attached image another screen shot
At least, most threads are duplicated in virtual folder.
must be a problem in the code that detects if a thread is already in the view...
Assignee: mscott → bienvenu
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch proposed fixSplinter Review
the problem came about because I removed the check that we were sorting by Id
to fix some other problem. But the code that builds up the list of root threads
assumed we were sorting by ID, since it did a binary search on the thread ids,
based on sort order. So the fix is to make sure the initial thread roots are
sorted by id, and then if the requested sort is not by ID, sort the thread
roots afterwards, before expanding them.

I was only able to recreate this problem by going into threaded mode in a quick
search view, and then using the view | sort by subject/sender, etc menu item to
sort the top level threads.
Attachment #199748 - Flags: superreview?(mscott)
Flags: blocking1.8rc1?
Attachment #199748 - Flags: superreview?(mscott)
Attachment #199748 - Flags: superreview+
Attachment #199748 - Flags: approval1.8rc1+
Status: NEW → RESOLVED
Closed: 19 years ago
Keywords: fixed1.8
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: