Closed
Bug 232557
Opened 21 years ago
Closed 21 years ago
emptying trash should optionally confirm folder deletion
Categories
(MailNews Core :: Backend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Bienvenu, Assigned: Bienvenu)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
mscott
:
superreview+
|
Details | Diff | Splinter Review |
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)
Comment 1•21 years ago
|
||
*** Bug 232573 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 2•21 years ago
|
||
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).
Assignee | ||
Updated•21 years ago
|
Attachment #140423 -
Flags: superreview?(mscott)
Updated•21 years ago
|
Attachment #140423 -
Flags: superreview?(mscott) → superreview+
Comment 3•21 years ago
|
||
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).
Assignee | ||
Comment 4•21 years ago
|
||
marking fixed.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•21 years ago
|
||
re-opening, new fix upcoming
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•21 years ago
|
||
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.
Assignee | ||
Updated•21 years ago
|
Attachment #140423 -
Attachment is obsolete: true
Assignee | ||
Updated•21 years ago
|
Attachment #142068 -
Flags: superreview?(mscott)
Updated•21 years ago
|
Attachment #142068 -
Flags: superreview?(mscott) → superreview+
Assignee | ||
Comment 7•21 years ago
|
||
fixed on trunk and m4 branch
Status: REOPENED → RESOLVED
Closed: 21 years ago → 21 years ago
Resolution: --- → FIXED
Comment 8•21 years ago
|
||
It appears this confirmation is only available for IMAP, correct?
Is there a reason the confirmation behavior is not considered desirable for POP
accounts?
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•