50601 - Shouldn't delete folder when users "Cancel" Delete folder from Trash

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect, P3)

Description

[Karen Huang]

Shouldn't delete folders when users "Cancel" Delete folder from Trash

Used 08-25-08-M18 commercial build

This is follow-up issue from bug 50232:

1) Tried to delete folders from the Trash folder.
2) A Confirm dialog displayed: "Deleting this folder is not undoable and will 
delete all of the messages it contains, Are you sure you still want to delete 
this fodler?
3) Select "Cancel" of this confirm message dialog
4) Actual Results: Folders of the trash folder have been deleted.


Expected results: After select "Cancel" of the confirm dialog, should just 
dismiss this confirm message dialog without delete any folders from the Trash 
folder.

Comment 1

[lchiang]

This is not good - if the user chooses Cancel, we should not proceed with the
folder delete.

Comment 2

[David :Bienvenu]

we don't really, we just delete them locally but they'll reappear next time you
restart. Pretty sure this bug has always been there, but you wouldn't have
noticed it because delete from trash wasn't working at all.

Comment 3

[lchiang]

No loss of data although this can be confusing. Also, user would have to go into 
the trash folder to delete the folder rather than emptying trash.  

- minus.

Comment 4

[David :Bienvenu]

this was fixed a while ago, I believe.

Comment 5

[Karen Huang]

Verified WinNT 11-08-01-MN6 build
Verified Linux 11-08-07-MN6 build
Verified Mac 11-08-00-MN6 build
After select "Cancel" of the confirm dialog, dialog dismiss without delete any 
subfolders from the Trash folder. Marking as verified.