23089 - selecting "undo" after deleting message causes trash folder to load

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P2)

Description

[Chris Waterson]

I'm not sure if this is a bug ofr a feature, but selecting "undo" after deleting
a message causes the "trash" folder to be stealthily opened and downloaded in
the background. This slows the products responsiveness significantly! (If you
have a large trash folder, like I do ;-)

Anyway, logging this a s a bug. If it's not a bug, just INVALID it.

Comment 1

[jefft]

This may not be a bug. In theory, copying messages to the trash folder doesn't
require to select the trash folder (meaning loading). However, undo a delete
requires to store the deleted flag to the message which must be done in the
selected state. This basically the reason to open the trash folder in the
background. I'll look into to make sure this is the case.

Comment 2

[jefft]

In order to be able to undelete a deleted message we have to store the deleted
flag to the message copied to the trash folder. To do that the trash folder
needs to be in a selected state. That's the reason why we are loading the trash
folder in the background. Marking this bug as wontfix.

Comment 3

[David :Bienvenu]

We need to be in the selected state, but that doesn't mean we have to download
all the headers. remember liteSelect?

Comment 4

[jefft]

Okay, I'll look into that. Thanks for reminding.

Comment 5

[jefft]

Fix checked in. Make sure we are in selected state by using lite select service 
then refresh the folder if needed.

Comment 6

[Chris Waterson]

Ugh, I'm seeing this again with IMAP. Symptoms are:

1. Delete message from inbox
2. Choose "Edit -> Undo Delete Message"
3. Trash folder loads in the background.

Furthermore,

4. While folder is loading in the background, new messages that get selected as 
a result of a "delete" operation do not display. You can *force* a message to 
display by clicking on them in the thread pane, but the "autodisplay" that 
should happen (delete+select next) doesn't work.

Comment 7

[Chris Waterson]

PDT: Because (4) creates a very confusing application state, I think we should 
consider this a beta blocker.

Comment 8

[Chris Waterson]

I don't know if this helps but, after switching to another folder, all of the 
messages I had previously "autoselected" by clicking delete showed up in rapid 
succession in the message pane. After which, switching back to my Inbox returned 
things to "normal". Looks like maybe operations are being queued up somewhere?

Comment 9

[leger]

Putting on PDT- radar for beta1.

Comment 10

[kmurray1115]

Beta 1 has come and gone, and this didn't get resolved. However, this should 
make the nsbeta3 cut under "correctness." Removing Beta1 keyword, nominating for 
nsbeta3 and "correctness."

Comment 11

[selmer (gone)]

Please retest.

Comment 12

[lchiang]

laurel - here is an undo case.

Comment 13

[laurel]

I don't see this behavior using linux rh6.0, aug08 m18 commercial build. Am
using IMAP, new profile, MoveToTrash.

Comment 14

[lchiang]

per mail triage, reassign to naving.  Perhaps we can mark this "worksforme" per 
Laurel's recent findings?

Comment 15

[lchiang]

mark wfm

Comment 16

[lchiang]

Interesting.  I know I can't make this happen again, but I am seeing this in the 
9/26 Win32 MN6 build.  I deleted a message and undid the deletion.  Now, the 
unread count on my Trash folder is incrementing and I'm not currently doing 
anything in the mail windows (Inbox selected).  One thing I did before this was 
that I was in 4.x mail and used it to delete a few messages before I started 
Netscape 6.

Comment 17

[David :Bienvenu]

this happened to me yesterday - undo is still (or again) doing a select of the
trash and downloading headers.

Comment 18

[lchiang]

clearing b3 need info - b3 has passed.  Should we nominate for rtm?

Comment 19

[scottputterman]

sorry for the extra email. Removing mail2 keyword.

Comment 20

[Chris Waterson]

I'd love for this to be fixed by mozilla1.0...

Comment 21

[David :Bienvenu]

I'll look at this.

Comment 22

[David :Bienvenu]

changing priorities

Comment 23

[Sheela Ravindran]

change qa contact to myself

Comment 24

[scottputterman]

marking nsbeta1+ and moving to mozilla0.8

Comment 25

[scottputterman]

reassigning to naving.

Comment 26

[Navin Gupta]

fix checked in

Comment 27

[Sheela Ravindran]

verified on win98(2001020508),mac(2001020512),linux(2001020512)