Fix check for getUrlsForRecord to include the record id for MultiBack…

…end support. (vufind-org#2858)
moravianlibrary · Apr 27, 2023 · 8a423fa · 8a423fa
1 parent 34157f4
commit 8a423fa
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/module/VuFind/src/VuFind/RecordDriver/Feature/IlsAwareTrait.php b/module/VuFind/src/VuFind/RecordDriver/Feature/IlsAwareTrait.php
@@ -171,7 +171,8 @@ public function getRealTimeTitleHold()
      */
     public function getURLs()
     {
-        return $this->hasILS() && $this->ils->checkCapability('getUrlsForRecord')
+        $params = [$this->getUniqueId()];
+        return $this->hasILS() && $this->ils->checkCapability('getUrlsForRecord', $params)
             ? $this->ils->getUrlsForRecord($this->getUniqueId())
             : [];
     }

diff --git a/module/VuFind/tests/unit-tests/src/VuFindTest/RecordDriver/Feature/MarcAdvancedTraitTest.php b/module/VuFind/tests/unit-tests/src/VuFindTest/RecordDriver/Feature/MarcAdvancedTraitTest.php
@@ -55,10 +55,13 @@ protected function getMockDriverFromFixture(string $fixture): SolrMarc
     {
         $record = new \VuFind\Marc\MarcReader($this->getFixture($fixture));
         $obj = $this->getMockBuilder(SolrMarc::class)
-            ->onlyMethods(['getMarcReader'])->getMock();
+            ->onlyMethods(['getMarcReader', 'getUniqueId'])->getMock();
         $obj->expects($this->any())
             ->method('getMarcReader')
             ->will($this->returnValue($record));
+        $obj->expects($this->any())
+            ->method('getUniqueId')
+            ->will($this->returnValue('123'));
         return $obj;
     }