Closed Bug 582576 Opened 12 years ago Closed 11 years ago

quick filter state on folder switch involving Account central does not match button state

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 10.0

People

(Reporter: bc6xv001, Assigned: iagosrl)

Details

(Keywords: student-project)

Attachments

(1 file, 4 obsolete files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.8) Gecko/20100722 Firefox/3.6.8 ( .NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.7) Gecko/20100713 Thunderbird/3.1.1

If you have a quick filter selected for a folder, then select a "main" folder ("Local Folders"), then switch back to the original, the button for the quick filter stays depressed, but the filter is not applied.

Reproducible: Always

Steps to Reproduce:
1. Select the Inbox in an account folder.  
2. Apply the "Unread" quick filter by pressing the button.  
3. Select any account folder (e.g.  "Local Folders" ) (not a folder of the account -- the actual bold account folder)
4. Select the same Inbox you selected in Step 1.
Actual Results:  
The unread quick filter button stays selected (regardless of pin state), but the filter is not actually applied (i.e. the folder shows all the messages, not just the unread ones)

Expected Results:  
Depends on pin state.  If pin selected, unread filter should be applied.

If the pin is not selected, the quick filter button state should have been reset to off.

Seems to work as expected when you select another mail folder instead of an account folder.  Also works as expected when, in step 4, you select a different mail folder instead of the original mail folder.

Bug also appears on the Mac version.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: quick filter state on folder switch does not match button state → quick filter state on folder switch involving Account central does not match button state
Excellent bug report, thank you.
Keywords: student-project
Attached file File with purposed bugfix (obsolete) —
Hi,
 I'm working in an extension that works with the filtering capabilities of Thunderbird called "Unified Search", I found this bug breaking my code and I did a bugfix from my extension, available from version 0.6.3 (https://addons.mozilla.org/en-US/developers/addon/unified-search-187593/versions/1272083) to try it.
 It works, after that I decide modify the original file to apply my bugfix and this is what I'm uploading here called quickFilterBar-fixed.js.
 Original file is located at omni.jar!/chrome/messenger/content/messenger/quickFilterBar.js
 I don't know if is the best solution, or the best code, but works. I expect that, if is not the valid code, almost help to find the best solution.
Patch over the previous complete file. The original file I used is from release 7.0.1.
I forget to say the bug happens too in Linux (Ubuntu 11.04), with Thunderbird versions from 3.1 to 7.0.1.
Comment on attachment 564352 [details] [diff] [review]
Same proposed bugfix that previous file but in patch format

Setting the r? so that your patch can get reviewed.

Wondering if we need to also add a unit test.
Attachment #564352 - Flags: review?(dbienvenu)
Assignee: nobody → iagosrl
Status: NEW → ASSIGNED
Attachment #564348 - Attachment is obsolete: true
Comment on attachment 564352 [details] [diff] [review]
Same proposed bugfix that previous file but in patch format

this doesn't actually apply as a patch against our repo...I'll try to convert it into a patch.
Attached patch in patch format (obsolete) — Splinter Review
Attachment #564352 - Attachment is obsolete: true
Attachment #567450 - Flags: review?(dbienvenu)
Attachment #564352 - Flags: review?(dbienvenu)
Attached patch tweaked patch (obsolete) — Splinter Review
I tweaked the patch (edited the comments, shortened some lines, got rid of temp vars, etc) but the patch didn't fix the issue. I don't think I messed up the logic of the original patch, but it didn't work for me.
Attachment #567450 - Attachment is obsolete: true
Attachment #567560 - Flags: review?
Attachment #567450 - Flags: review?(dbienvenu)
Comment on attachment 567560 [details] [diff] [review]
tweaked patch

Robert, could you try my version of the patch against the trunk and see if it works for you? Thx!
Attachment #567560 - Flags: review? → review-
Hi David,

 The logic of your last patch is correct but there is a small mistype (missed letter 'p' at aFolderDisplay) here:

if (!aFolderDislay.displayedFolder ||
ah, thx much, that works. r+. I'll land this when I get a chance, and the tree is open...
Attachment #567560 - Attachment is obsolete: true
Attachment #567854 - Flags: review+
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 10.0
You need to log in before you can comment on or make changes to this bug.