From b914b6fb38bf52f79d552c9f9687bd2e95ed6631 Mon Sep 17 00:00:00 2001
From: Demian Katz <demian.katz@villanova.edu>
Date: Tue, 1 Mar 2016 14:57:21 -0500
Subject: [PATCH] Fixed bug in DOM ID generation. - We need to base DOM IDs on
the FULL, UNIQUE identifier, not just the search class; otherwise, multiple
filtered views of the same search backend will yield duplicate IDs.
---
module/VuFind/src/VuFind/Controller/CombinedController.php | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/module/VuFind/src/VuFind/Controller/CombinedController.php b/module/VuFind/src/VuFind/Controller/CombinedController.php
index b9dbcbd0a5a..350df2e2914 100644
--- a/module/VuFind/src/VuFind/Controller/CombinedController.php
+++ b/module/VuFind/src/VuFind/Controller/CombinedController.php
@@ -166,7 +166,7 @@ class CombinedController extends AbstractSearch
// Calculate a unique DOM id for this section of the search results;
// $searchClassId may contain colons, which must be converted.
$combinedResults[$current]['domId']
- = 'combined_' . str_replace(':', '____', $searchClassId);
+ = 'combined_' . str_replace(':', '____', $current);
$combinedResults[$current]['view']
= (!isset($settings['ajax']) || !$settings['ajax'])
--
GitLab