Closed Bug 232557 Opened 21 years ago Closed 21 years ago

emptying trash should optionally confirm folder deletion

Categories

(MailNews Core :: Backend, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Bienvenu, Assigned: Bienvenu)

References

Details

Attachments

(1 file, 1 obsolete file)

If you empty the trash, and that will cause folders to get deleted, we should warn the user (perhaps optionally, based on a pref). 4.x always warned the user. We should also warn if the user deletes a folder from the trash (I thought we did that, but it doesn't seem to be happening)
Blocks: m6
Status: NEW → ASSIGNED
*** Bug 232573 has been marked as a duplicate of this bug. ***
Attached patch proposed fix (obsolete) (deleted) — Splinter Review
add pref mail.imap.confirm_emptyTrashFolderDeletion, and prompt user for each top-level folder in trash that will be deleted. If user says no, we'll skip the folder, and if they say cancel, we'll skip the rest of the folders (we will still delete the folders the user said yes to).
Attachment #140423 - Flags: superreview?(mscott)
Attachment #140423 - Flags: superreview?(mscott) → superreview+
I checked in a bustage fix (fixed the case of an #include), but you might want to back out your previous bustage fix, which seems wrong (QueryInterface and GetInterface should not be interchangeable, anyway).
marking fixed.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
re-opening, new fix upcoming
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached patch proposed fix (deleted) — Splinter Review
we need to pre-flight the deletes, because the imap protocol code deletes the sub-folders when we delete all messages! And we can't pick and choose anymore; the previous patch just prevented the deletion of the folders in the ui.
Attachment #140423 - Attachment is obsolete: true
Attachment #142068 - Flags: superreview?(mscott)
Attachment #142068 - Flags: superreview?(mscott) → superreview+
fixed on trunk and m4 branch
Status: REOPENED → RESOLVED
Closed: 21 years ago21 years ago
Resolution: --- → FIXED
It appears this confirmation is only available for IMAP, correct? Is there a reason the confirmation behavior is not considered desirable for POP accounts?
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: