Merge pull request #1903 from NYPL-Simplified/OE-927

OE-917 [fix] Set the original facet to active

core/opds.py

@@ -1034,6 +1034,8 @@ def make_link(ep):
                 # the facet url for this given ordering facet instead of the original facet.
                 facets.order = order
                 facet["href"] = annotator.search_url(lane, query, pagination=None, facets=facets)
+                if order == original_facet:
+                    facet["href"] += "&active=true"
                 AcquisitionFeed.add_link_to_feed(feed=opds_feed.feed, **facet)
         facets.order = original_facet
 

core/tests/test_opds.py

@@ -1296,6 +1296,9 @@ def make_page(pagination):
         assert 'Title' in sort_by_facets
         assert 'Author' in sort_by_facets
 
+        [active_facet] = [facet['title'] for facet in facets if getattr(facet, "activefacet", False) == 'true']
+        assert active_facet == 'Author'
+
         # The feed has no breadcrumb links, since we're not
         # searching the lane -- just using some aspects of the lane
         # to guide the search.