Closed Bug 1727366 Opened 3 years ago Closed 3 years ago

Disabled status of message filters not shown in list of filters

Categories

(Thunderbird :: Filters, defect)

Thunderbird 91
defect

Tracking

(thunderbird_esr91+ fixed, thunderbird92 fixed, thunderbird93 fixed)

RESOLVED FIXED
93 Branch
Tracking Status
thunderbird_esr91 + fixed
thunderbird92 --- fixed
thunderbird93 --- fixed

People

(Reporter: reg_bugzilla.mozilla.org, Assigned: Paenglab)

References

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/92.0.4515.159 Safari/537.36

Steps to reproduce:

Open Message Filter
Toggle "enabled" state for some filters by clicking the checkmark
Close Message Filter
Open Message Filter again

Actual results:

all the previous disabled Message Filters were enabled again

Expected results:

Thunderbird should save the enabled/disabled state of the message filters after closing the Message Filter window.

Further infos:
Win 10 Pro 64 bit, Thunderbird 91.0.2 64 bit
Tried it with troubleshoot mode and created a new profile, bug persists in both cases.

I see this with 92.0b3. Nothing in error console.

This could lead to dataloss, so pretty serious.

I think I have disabled filters that are not being listed, so perhaps it is partly a display issue?

Blocks: tb91found
Severity: -- → S3
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(mkmelin+mozilla)

I looked at my msgfilters.dat file and I see rules with enabled="no". So it is definitely a display issue.

Attached patch 1727366-filter-checked.patch (deleted) — Splinter Review

The toolkit rule check only on [checked] and the checkbox uses checked="false". With this the rule is always true.

Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Flags: needinfo?(mkmelin+mozilla)
Attachment #9237994 - Flags: review?(alessandro)
Summary: Disabled status of message filters not saved → Disabled status of message filters not shown in list of filters
Comment on attachment 9237994 [details] [diff] [review] 1727366-filter-checked.patch Review of attachment 9237994 [details] [diff] [review]: ----------------------------------------------------------------- Thanks for taking care of this. Things will be way easier once we get rid off of all these XUL elements.
Attachment #9237994 - Flags: review?(alessandro) → review+
Target Milestone: --- → 93 Branch
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED

Comment on attachment 9237994 [details] [diff] [review]
1727366-filter-checked.patch

[Approval Request Comment]
User impact if declined: disabled filters are still shown as active
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): should be low, it's a simple JS change

Attachment #9237994 - Flags: approval-comm-esr91?
Attachment #9237994 - Flags: approval-comm-beta?

Comment on attachment 9237994 [details] [diff] [review]
1727366-filter-checked.patch

[Triage Comment]
Approved for beta

Attachment #9237994 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9237994 [details] [diff] [review]
1727366-filter-checked.patch

[Triage Comment]
Approved for esr91

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

Attachment

General

Created:
Updated:
Size: