Merge pull request #72 from AlexanderBlanchardAC/summon_facet_results

fix: summon results filtering fix
PTFS-Europe · Jul 12, 2024 · 8262dc3 · 8262dc3
2 parents 381c3fd + 2128d2f
commit 8262dc3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/code/web/release_notes/24.07.00.MD b/code/web/release_notes/24.07.00.MD
@@ -75,6 +75,7 @@
 
 ### Summon Updates
 - Adjustments to code in Summon getFacetSet function to correct a bug. Individual facet filters can now be unchecked and the filter unset by clicking the checkbox. (*AB*)
+- Adjust code to ensure that the filter applies correctly no matter what page of results users are on when they apply it. (*AB*)
 
 ### Other Updates
 - Show the original translation term when translating text within Aspen. (*MDN*) 

diff --git a/code/web/sys/SearchObject/SummonSearcher.php b/code/web/sys/SearchObject/SummonSearcher.php
@@ -496,7 +496,7 @@ public function getFacetSet() {
 					if ($isApplied) {
 						$facetSettings['removalUrl'] = $this->renderLinkWithoutFilter($facetId . ':' . $facetValue);
 					} else {
-						$facetSettings['url'] = $this->renderSearchUrl() . '&filter[]=' . $facetId . ':' . urlencode($facetValue);
+						$facetSettings['url'] = $this->renderSearchUrl() . '&filter[]=' . $facetId . ':' . urlencode($facetValue) . '&page=1';
 					}
 					$list[] = $facetSettings;
 				}