27688 - Reparenting threads not working

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P3)

Description

[scottputterman]

I've stepped through the code and I'm pretty sure I'm doing what I'm getting
told to do by the db.  This is what is happening:

Suppose you have thread:

a
 b
 c

If you delete a, currently both b and c are moving up to the top level.  I'm not
sure if that's what we want. Regardless, what I'm seeing happening is that
either b or c has the message counts and the other does not. So even though they
are on the same level, b might say it has 2 total messages when c says it has
none.  Based on the code that gets this info into RDF, this means the thread
think that b is a top level message and c is not.  Finally, if I then go to a
flat view and back to a thread view, c dissapears and can no longer be seen in
the threaded view.  After playing around with delete a bunch this week I'm
seeing my message counts getting messed up because messages will say they have 2
total messages but only one is visible.

Comment 1

[David :Bienvenu]

accepting, nominating as beta blocker, not that I'll have much time to work on 
it for a few days.

Comment 2

[leger]

Putting on the PDT+ radar for beta1.  Missing msgs and incorrect msg count for
navigation are MUST FIXES for beta 1.

Comment 3

[David :Bienvenu]

setting ETA - can't work on this until AOL IMAP stuff is done.

Comment 4

[David :Bienvenu]

Attachment: proposed fix

Comment 5

[David :Bienvenu]

fix checked in.

Comment 6

[fenella]

Linux (2000-02-22-12 M14)
Win32 (2000-02-23-09 M14)
Mac (2000-02-23-08 M14) mozilla

When msg a is deleted, b moves up to be the top level. c becomes the child of b.
Message is still there in both flat and thread view.