Correctly set active search facet #1905
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Correctly set the active search facet
Motivation and Context
My previous hacky attempt didn't actually set the
@opds:activeFacet
in the OPDS feed. I thought the URL would pick up on it and do some background magic, but it didn't look like it did.This now builds out a facet link by using
cls.facet_link
to ensure theactiveFacet
property is correctly set. But this generates a.../feed/...
URL and we want a.../search...
URL, so then we replace the href with whatannotator.search_url
builds before adding the facet to the OPDS feed.How Has This Been Tested?
More manual testing and verifying the search OPDS feed for the active facet is set through
+@opds:activeFacet: "true"
Checklist: