Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Complex rule doesn't get restored correctly #256

Open
Betterbird opened this issue Jan 20, 2024 · 4 comments
Open

Complex rule doesn't get restored correctly #256

Betterbird opened this issue Jan 20, 2024 · 4 comments
Labels

Comments

@Betterbird
Copy link
Owner

Start with
image

which in msgFilterRules.dat is stored as
"AND (to,contains,A) AND (subject,contains,A) OR [to,contains,B) AND (subject,contains,B] OR [to,contains,C) AND (subject,contains,C]"

That's wrong, so the filter is restored to
image

Starting with
image

is stored as
"AND (to,contains,A) AND (subject,contains,A) AND [subject,contains,dummy-dummy-dummy) OR [to,contains,B) AND (subject,contains,B] OR [to,contains,C) AND (subject,contains,C]]"

and that gets restored properly.

@Betterbird
Copy link
Owner Author

Betterbird commented Jan 21, 2024

We took a look at our test data, since we had tested some scenarios. Turned out that we had a non-sensical test case:
Ex4

That actually works, but as soon as you switch "all" to "any" in the inner groups, it all falls apart. So much for letting the developers do the testing 😭

As an immediate mitigation we will disallow groups following groups, like this:
image

So into a group, you need to add a term first before starting another group. That's along the lines of your suggestion (quote): I would suggest prohibiting entering second-level nested terms at all.

@kupietools: Please voice objections now or it will ship on the coming Tuesday, 23rd Jan. 2024.

@ThiloteE
Copy link

I have following filter:
image
Will it still work? Saw this issue too late. I am now afraid of updating to 115.7.0-bb23 (23 January 2024)

@Betterbird
Copy link
Owner Author

Yes, that will work. What doesn't work and never worked is this:
image

@ThiloteE
Copy link

Thank you :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants