Closed Bug 517723 Opened 12 years ago Closed 12 years ago

Shredder "Empty Trash" on top of Trash tree does nothing in Smart-Folder view

Categories

(Thunderbird :: Mail Window Front End, defect, P2)

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0rc1

People

(Reporter: henry.nestler, Assigned: Bienvenu)

References

Details

(Whiteboard: [no l10n impact])

Attachments

(1 file, 1 obsolete file)

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; de; rv:1.9.1.2) Gecko/20090730 SUSE/3.5.2-2.1 Firefox/3.5.2
Build Identifier: Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.9.1.4pre) Gecko/20090915 Thunderbird/3.0b4

The Trash have a list of folders (once per account). If I stay on top of Trash (and not on an account) and use the menu "File"* "Empty Trash" or the mouse context menu "empty trash", then TB asked me "doing this now?", and than it does nothing.


Reproducible: Always

Steps to Reproduce:
1. Have some messages in "Trash" * "Local Folder"
2. Select the "Trash" self, not the "Local Folder"
3. use menu "Empty Trash" or right mouse key and "Empty Trash"
4. If a question popped up "You are sure?" (or so), than say yes.
Actual Results:  
All messages in "Trash" * "Local Folder" still exist.

Expected Results:  
Empty the complete Trash for all accounts.

That no problem, that I needs to select first an account (for example "Local Folder") and than use the "Empty Trash" there.

Please disable the menu "Empty Trash", if no account is selected.
Problem exist only in the "Smart-Folder" view.
Summary: Shredder "Empty Trash" on top of Trash tree does nothing → Shredder "Empty Trash" on top of Trash tree does nothing in Smart-Folder view
Blocks: 515237
confirming on Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1.4pre) Gecko/20090915 Thunderbird/3.0b4. Error console is empty.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Version: unspecified → 3.0
Flags: blocking-thunderbird3?
there are several bugs having to do with executing commands on smart folders (empty trash, compact inbox, etc), that we should try to fix for rc1. It shouldn't be too hard to fix them, so I'm going to mark this blocking.
Assignee: nobody → bienvenu
Status: NEW → ASSIGNED
Flags: blocking-thunderbird3? → blocking-thunderbird3+
Target Milestone: --- → Thunderbird 3.0rc1
I believe this won't have l10n impact.
Whiteboard: [no l10n impact]
Priority: -- → P2
Smart Folders should not the default view for new profile.
Attached patch proposed fix (obsolete) — Splinter Review
The nsLocalMailFolder.cpp change is really just extra protection - with this patch, we shouldn't try to call empty trash on the smart folders trash, but just in case some other UI tries to do that, it would be nice not to wipe out the smart trash.
Attachment #402913 - Flags: superreview?(bugzilla)
Attachment #402913 - Flags: review?(bugzilla)
Whiteboard: [no l10n impact] → [no l10n impact][has patch for r/sr standard8]
Attached patch proposed fix, v2Splinter Review
Standard8 pointed out that empty trash when you've got the smart inbox selected should empty all the trashes. So I made empty trash with any top-level smart folder selected empty all the trashes.

If you have a non-smart folder selected, e.g., the inbox of one of your accounts, I just empty the trash of the selected account.
Attachment #402913 - Attachment is obsolete: true
Attachment #403596 - Flags: superreview?(bugzilla)
Attachment #403596 - Flags: review?(bugzilla)
Attachment #402913 - Flags: superreview?(bugzilla)
Attachment #402913 - Flags: review?(bugzilla)
Comment on attachment 403596 [details] [diff] [review]
proposed fix, v2

(no sr required).

This seems reasonable now, though I feel we should just get clarkbw to give his ok.
Attachment #403596 - Flags: ui-review?(clarkbw)
Attachment #403596 - Flags: superreview?(bugzilla)
Attachment #403596 - Flags: review?(bugzilla)
Attachment #403596 - Flags: review+
Comment on attachment 403596 [details] [diff] [review]
proposed fix, v2

This looks good though I feel like in the future we might want an alternate confirmation dialog for this.  The current dialog does say that the trash and subfolders will be emptied so I think it's alright now.  However in the future I'd prefer something that let you know how many / which accounts trash folders you were emptying.
Attachment #403596 - Flags: ui-review?(clarkbw) → ui-review+
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [no l10n impact][has patch for r/sr standard8] → [no l10n impact]
Hardware: x86_64 → All
You need to log in before you can comment on or make changes to this bug.