Get unique html element id from record view helper

module/VuFind/src/VuFind/View/Helper/Root/Record.php

@@ -419,10 +419,10 @@ public function getSearchResult($view)
      */
     public function getCheckbox($idPrefix = '', $formAttr = false, $number = null)
     {
-        $id = $this->driver->getSourceIdentifier() . '|'
-            . $this->driver->getUniqueId();
-        $context
-            = ['id' => $id, 'number' => $number, 'prefix' => $idPrefix];
+        $context = compact('number') + [
+            'id' => $this->getUniqueIdWithSourcePrefix(),
+            'checkboxElementId' => $this->getUniqueHtmlElementId($idPrefix)
+            ];
         if ($formAttr) {
             $context['formAttr'] = $formAttr;
         }
@@ -711,4 +711,35 @@ protected function deduplicateLinks($links)
     {
         return array_values(array_unique($links, SORT_REGULAR));
     }
+
+    /**
+     * Get the source identifier + unique id of the record without spaces
+     *
+     * @param string $idPrefix Prefix for HTML ids
+     * @throws \Exception If no record driver can be found
+     *
+     * @return string
+     */
+    public function getUniqueHtmlElementId($idPrefix = "")
+    {
+        return preg_replace(
+            "/\s+/",
+            "_",
+            ($idPrefix ? $idPrefix . '-' : '') . $this->getUniqueIdWithSourcePrefix()
+        );
+    }
+
+    /**
+     * Get the source identifier + unique id of the record
+     *
+     * @return string
+     */
+    public function getUniqueIdWithSourcePrefix()
+    {
+        if ($this->driver) {
+            return "{$this->driver->tryMethod('getSourceIdentifier')}"
+                . "|{$this->driver->tryMethod('getUniqueId')}";
+        }
+        throw new \Exception("No record driver found.");
+    }
 }

module/VuFind/tests/unit-tests/src/VuFindTest/View/Helper/Root/RecordTest.php

@@ -365,6 +365,21 @@ public function testGetLink(
         $this->assertEquals($expected, $record->getLink('bar', 'foo'));
     }
 
+    /**
+     * Helper function for testGetCheckbox to simulate withConsecutive deprecated in PhpUnit 9.6
+     * stolen from https://stackoverflow.com/a/75536028
+     *
+     * @param ...$args
+     * @return callable
+     */
+    public function consecutiveCalls(...$args): callable
+    {
+        $count = 0;
+        return function ($arg) use (&$count, $args) {
+            return $arg == $args[$count++];
+        };
+    }
+
     /**
      * Test getCheckbox.
      *
@@ -373,21 +388,18 @@ public function testGetLink(
     public function testGetCheckbox()
     {
         $context = $this->getMockContext();
-        $this->expectConsecutiveCalls(
-            $context,
-            'renderInContext',
-            [
-                [
+        $context->expects($this->exactly(2))->method('renderInContext')
+            ->with(
+                self::callback(self::consecutiveCalls(
                     'record/checkbox.phtml',
-                    ['id' => 'Solr|000105196', 'number' => 1, 'prefix' => 'bar', 'formAttr' => 'foo'],
-                ],
-                [
+                    ['id' => 'bar-Solr|000105196', 'number' => 1, 'prefix' => 'bar', 'formAttr' => 'foo']
+                )),
+                self::callback(self::consecutiveCalls(
                     'record/checkbox.phtml',
-                    ['id' => 'Solr|000105196', 'number' => 2, 'prefix' => 'bar', 'formAttr' => 'foo'],
-                ],
-            ],
-            'success'
-        );
+                    ['id' => 'bar-Solr|000105196', 'number' => 2, 'prefix' => 'bar', 'formAttr' => 'foo']
+                ))
+            )
+            ->willReturnOnConsecutiveCalls('success', 'success');
         $record = $this->getRecord(
             $this->loadRecordFixture('testbug1.json'),
             [],