Merge pull request #87 from AlexanderBlanchardAC/summon_facet_results

Repeating merged pr, moving from 24.07 to 24.08
PTFS-Europe · Jul 17, 2024 · aa074df · aa074df
2 parents 70362dc + 125cc9e
commit aa074df
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/code/web/release_notes/24.08.00.MD b/code/web/release_notes/24.08.00.MD
@@ -18,6 +18,8 @@
 - Fixed bug with unexpected 404 errors on Web Builder pages.  (Ticket 123122) (*KP*)
 
 // alexander
+### Summon Updates
+- Adjust code to ensure that the filter applies correctly no matter what page of results users are on when they apply it. (*AB*)
 
 // jacob
 

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;
 				}