Closed Bug 285144 Opened 19 years ago Closed 19 years ago

Unread flag removed on parent folder even if other subfolders still have unread messages if ONE subfolder changes to no more unread messages

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
Windows 2000
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: nuesken, Assigned: Bienvenu)

References

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; de-DE; rv:1.7.6) Gecko/20050226 Firefox/1.0.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; de-DE; rv:1.7.6) Gecko/20050226 Firefox/1.0.1

A mail folder is marked in bold if it contains unread messages or a folder that
contains unread messages.  I am working with a server that distinguishes
subfolder folders (directories as pine callas them) and message only folders,
don't know whether this is important.
If now one subfolder changes from "contains unreads" to "contains no unread"
also the parent folder changes to contains NO unread EVEN if there are other
subfolders  that still have unread mails.

Reproducible: Always

Steps to Reproduce:
1. Go to a folder that has at least two subfolders with unread mail.
2. Go to one of the subfolders and read all unread mail in it.  Leave at least
one other subfolder with an unread mail.
3. Close the parent folder.

Actual Results:  
The parent folder is not marked as having unread mail under it any more.

Expected Results:  
The parent folder should be flagged as long as there is some subfolder contains
an unread mail.
I see this, and I'm not having any luck finding a dupe.
Reproduced with TB 1.0+0303 and Moz 1.8b-0303, Win2K.
Severity: normal → minor
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → Windows 2000
This is a dupe of bug #276650
*** Bug 276650 has been marked as a duplicate of this bug. ***
the problem is here:

http://lxr.mozilla.org/mozilla/source/mailnews/base/src/nsMsgFolderDataSource.cpp#1734

This needs to be changed to force a recalculation of this attribute, not forcing
it off. I'll try to figure out how to do that.
Assignee: mscott → bienvenu
Attached patch proposed fixSplinter Review
see comments in diff. The one thing I'm not sure about is if I should do this
w/o a comptr - I suppose I could just pass in &aNode and do an NS_RELEASE after
the call to createHasUnreadMessagesNode (which addrefs the result). But the
comptr does the same. The folder data source holds a ref to the true and false
literals, so we don't have to worry about it going away when the comptr goes
out of scope...if we were worried about that, we could duplicate the calls to
notify property changed and notify ancestor changed inside the scope of the
comptr...
Attachment #191266 - Flags: superreview?(mscott)
Attachment #191266 - Flags: superreview?(mscott) → superreview+
Attachment #191266 - Flags: approval1.8b4?
Attachment #191266 - Flags: approval1.8b4? → approval1.8b4+
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Just found this bug. I always just figured this is how it was meant to be. 
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: