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

Merging filters can lead to duplicate conditions #51

Closed
RealRaven2000 opened this issue Mar 19, 2021 · 2 comments
Closed

Merging filters can lead to duplicate conditions #51

RealRaven2000 opened this issue Mar 19, 2021 · 2 comments
Assignees
Labels
Tb78 Thunderbird 78 regression

Comments

@RealRaven2000
Copy link
Owner

This may be a regression caused by Thunderbird 78. when we add new filter conditions via assistant (or the context command "quickFilters: create filter from message" and merge into an existing filter, using the "Based on Sender (from)" template, this can lead to duplicate conditions, if the same address is already present as a "from - contains - email address".

Expected behavior: duplicate conditions should not be generated while merging.

@RealRaven2000 RealRaven2000 added the Tb78 Thunderbird 78 regression label Mar 19, 2021
@RealRaven2000 RealRaven2000 self-assigned this Mar 19, 2021
@RealRaven2000 RealRaven2000 pinned this issue Mar 19, 2021
@RealRaven2000
Copy link
Owner Author

I checked and I think I only avoid duplicates within the newly added terms (say if you use multiple mails to generate a filter and "harvest" the same email address twice). Here is a prerelease version that guards against adding a duplicate search term (as long as the search value is a string).

quickFilters-wx-5.2pre28.zip

(To install, download the zip file and then drag that file into Add-ons Manager).

I also see that the searchTerms list will be in a completely different format in Thunderbird 88 so I need to rewrite quite a bit of code dealing with filter search terms to make it work in the next ESR. See issue #49 for news on the next generation!

@RealRaven2000 RealRaven2000 changed the title Merging new filter conditions leads to duplicates Merging filters can lead to duplicate conditions Mar 26, 2021
@RealRaven2000
Copy link
Owner Author

Fixed in releases 4.5 + 5.2

@RealRaven2000 RealRaven2000 unpinned this issue Apr 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tb78 Thunderbird 78 regression
Projects
None yet
Development

No branches or pull requests

1 participant