-
Notifications
You must be signed in to change notification settings - Fork 11
Merging to a filter with "Subject" template breaks filter #161
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
Comments
Hi PotatoCarl, can you give me more detail? Did you merge with subject? I assume this is a normal filter on Thunderbird? note that quickFilters does not currently support merging into "complicated" filters in Betterbird that have mixed any / all operators. You can export the broken filter by highlighting it in the message filters window and then clicking the backup button while holding CTRL: |
heres the filter |
So I am finding that when I add a new subject using the merge function with the "subject" template, there is a problem with internally "mixed" operators (any / all) this definitely needs fixing ASAP. In the meantime, you can fix such cases in Thunderbird via the troubleshoot button: You can skip the first step but if you get a message like this: You can ask quickFilters to repair these filters. It will replace the wrong operators "all" with "any". (Internally Thunderbird allows mixing any / all, there is just no UI that supports this. Careful Betterbird Users: Some Filters may have mixed any / all conditions and parentheses (created through their new UI) - these can currently not be repaired reliably with quickFIlters! This is just too complex at the moment. In Betterbird, if you want to repair potentially broken filters with mixed any/all conditions, you can only do it by exporting to a file and then editing that manually. quickFilters might detect working filters (with mixed conditions) as faulty that are actually working as intended. We have to add a lot of programming logic to support troubleshooting this (and the merging function) when "mixed operator" filters are created. It will definitely come in some future versions, but for now I am concentrating on Thunderbird. |
Here is a first attempt at fixing this: quickFilters-wx-5.8.1pre11.zip This will check if there are already search terms and use the "any" operator (booleanAnd = false) if the very first search term also uses "any". The main problem is if you already have message up filters there may already be a single mixed "all" operator in one of the following search terms, so I feel like I really need to do a full sanity check on all the "booleanAnd" parameters of every search term in the existing filter when merging. This however would need to be done with caution in case we are in Betterbird and there was a genuine mixed operator filter, such as:
in this example we have:
in this case what should the new term D use? I could imagine it to try to insert into the last group, such as: Term a AND For this it would need to copy the operator (booleanAnd=false) from the last term, and then remove endsGrouping and set it to the new element instead. This can get quite complicated and messy quickly. Anyway, for Thunderbird please download the zip file above and drag it into your Thunderbird Add-ons Manager to install and try out. |
Fixed in 5.8.1 released on 18/02/2023 |
Hi,
When I create a filter with a subject filter, it breaks an existing filter an no filters are used anymore.
The text was updated successfully, but these errors were encountered: