256330 - IMAP delete of folder doesn't move folder to Trash.

Status: VERIFIED FIXED

(MailNews Core :: Networking: IMAP, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Build ID: 2004-08-20-07, Windows XP, Seamonkey Trunk.

Summary: IMAP delete of folder doesn't move folder to Trash.

Steps to Reproduce:

1.  Create a folder "DeleteMe" as a sibling under Inbox (I'm on Cyrus)
2.  Delete it using the context menu

Actual Results:

"Unsubscribing from folder INBOX.DeleteMe..." appears in the window.status area

Expected Results:

Folder is moved visually to the subfolder of INBOX.trash

I just tested this in 1.7 (Netscape 7.2) and it works fine with this server.

Log to follow.

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Logfile

Comment 2

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Screenshot; this is all I see until I restart.

Comment 3

[David :Bienvenu]

Stephen, do you have empty trash on exit set?

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #3)
> Stephen, do you have empty trash on exit set?

Yes, yes I do.  Want me to try it without and get a log for that?

Comment 5

[David :Bienvenu]

yes, I believe we're deleting the folder from the trash when you exit, instead
of just the messages in the trash folder.  That explains why the folder doesn't
appear as a sub-folder of the trash when you restart, at least.

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

With 'Empty trash on exit' OFF...

Launch (Inbox loads by default, no messages selected)
Create folder DeleteMe via context menu
Select DeleteMe (contains no messages)
Delete (context menu)
File | Empty Trash
Exit

I still see the same behavior -- until I restart, the folder is visible in the
Folder Pane's tree view, but when selected, alerts me that the folder does not
exist.

And according to this snippet from the logfile, the delete/unsubscribe of the
folder "DeleteMe" actually works, but we're failing to notify the front end,
somehow.

calls)412[213ed90]: 216a4a0:imap.indiana.edu:S-INBOX.Trash:SendData: 6 delete
"INBOX.Trash.DeleteMe"412[213ed90]: ReadNextLine [stream=216b640 nb=16 needmore=0]
412[213ed90]: 216a4a0:imap.indiana.edu:S-INBOX.Trash:CreateNewLineFromSocket: 6
OK Completed412[213ed90]: 216a4a0:imap.indiana.edu:S-INBOX.Trash:SendData: 7
unsubscribe "INBOX.Trash.DeleteMe"412[213ed90]: ReadNextLine [stream=216b640
nb=16 needmore=0]
412[213ed90]: 216a4a0:imap.indiana.edu:S-INBOX.Trash:CreateNewLineFromSocket: 7
OK Completed412[213ed90]: 216a4a0:imap.indiana.edu:S-INBOX.Trash:SendData: 8
IDLE412[213ed90]: ReadNextLine [stream=216b640 nb=12 needmore=0]
412[213ed90]: 216a4a0:imap.indiana.edu:S-INBOX.Trash:CreateNewLineFromSocket: +
go ahead2848[20107e0]: 20121e8:imap.indiana.edu:S-INBOX:SendData:
DONE2848[20107e0]: ReadNextLine [stream=2015c70 nb=17 needmore=0]
2848[20107e0]: 20121e8:imap.indiana.edu:S-INBOX:CreateNewLineFromSocket: 17 OK
Completed2848[20107e0]: 

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Logfile with Empty Trash on Exit off.

Comment 8

[David :Bienvenu]

yup, I see this happening as well.

Comment 9

[Scott MacGregor]

David's going to try to fix this for tbird 0.8. 

Comment 10

[David :Bienvenu]

Attachment: proposed fix

make OpenMailDBFromFileSpec return success if the caller says to create a db if
one is missing. This is only called from a couple places, and I've tested that
the existing callers are fine with the change.

Comment 11

[David :Bienvenu]

fixed on trunk and branch, thx, Stephen.

Comment 12

[David :Bienvenu]

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

Comment 13

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED on today's 2004-09-09-16 Seamonkey trunk build on Windows XP.

Thanks David!

Comment 14

[Stephen Donner [:stephend] Not actively reading bugmail]

The build date ID is wrong on today's build, but the bug is indeed fixed (sorry
for the spam!)