Closed Bug 1848904 Opened 1 years ago Closed 1 years ago

Adding a folder type flag does not add the folder's descendants to unified/smart folders

Categories

(MailNews Core :: General, defect)

defect

Tracking

(thunderbird_esr115 fixed)

RESOLVED FIXED
118 Branch
Tracking Status
thunderbird_esr115 --- fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

References

Details

Attachments

(1 file)

If a folder is created with a folder type flag (e.g. Archive) it is automatically added to the unified folder for that flag. After bug 378833, we also add the descendants of that folder to the unified folder.

However if a folder that already exists is given a flag, it gets added to the unified folder but its descendants do not. This can happen in some obscure cases as an account is set up.

If a folder's flag changes, or a folder already with a flag is added or removed, it AND its
descendants should be added to the appropriate virtual folder. This patch tests that and fixes
some inconsistent behaviour.

In reality for most folder types this is a very rare occurrence, and for some it would never
happen, but it might as well work the same way for all types.

Target Milestone: --- → 118 Branch

Pushed by brendan@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/601bb5f64dcb
Check a folder and its descendants are added to/removed from virtual folders when flags change. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 1 years ago
Resolution: --- → FIXED

Comment on attachment 9349159 [details]
Bug 1848904 - Check a folder and its descendants are added to/removed from virtual folders when flags change. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined:
Testing completed (on c-c, etc.): in 118
Risk to taking this patch (and alternatives if risky):

Attachment #9349159 - Flags: approval-comm-esr115?

Comment on attachment 9349159 [details]
Bug 1848904 - Check a folder and its descendants are added to/removed from virtual folders when flags change. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr115

Attachment #9349159 - Flags: approval-comm-esr115? → approval-comm-esr115+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: