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

[9.1] Fix conditional filter listener to check the backend id. #3513

Conversation

EreMaijala
Copy link
Contributor

@EreMaijala EreMaijala commented Mar 14, 2024

While this problem would cause no trouble when only a single backend is used within a request, with Blender the conditional filter listener would have added the filters to all backends.

TODO

  • Update changelog when merging (constructor signature change).

@EreMaijala EreMaijala changed the title Fix conditional filter listener to check the backend id. [9.1] Fix conditional filter listener to check the backend id. Mar 14, 2024
@demiankatz demiankatz added this to the 9.1.2 milestone Mar 14, 2024
Copy link
Member

@demiankatz demiankatz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for figuring this out and providing the fix, @EreMaijala!

@demiankatz demiankatz merged commit 9b102f2 into vufind-org:release-9.1 Mar 14, 2024
7 checks passed
@demiankatz demiankatz deleted the release-9.1-fix-conditional-filter-listener branch March 14, 2024 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants