Strip tags in EDS getAllSubjectHeadingsFlattened

vufind-org · Oct 8, 2024 · 58038e8 · 58038e8
1 parent 0621131
commit 58038e8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/module/VuFind/src/VuFind/RecordDriver/EDS.php b/module/VuFind/src/VuFind/RecordDriver/EDS.php
@@ -451,7 +451,7 @@ public function getAllSubjectHeadingsFlattened()
     {
         $subject_arrays = array_map(
             function ($data) {
-                return explode(', ', $data['Data']);
+                return explode(', ', rtrim(strip_tags($data['Data']), '.'));
             },
             $this->getItems(null, null, 'Su')
         );

diff --git a/module/VuFind/tests/unit-tests/src/VuFindTest/RecordDriver/EDSTest.php b/module/VuFind/tests/unit-tests/src/VuFindTest/RecordDriver/EDSTest.php
@@ -573,8 +573,8 @@ public function testGetAllSubjectHeadingsFlattened(): void
         $driver = $this->getDriver('valid-eds-record');
         $this->assertEquals(
             [
-                '<a href="../EDS/Search?lookfor=%22PSYCHOTHERAPY%22&amp;type=SU">PSYCHOTHERAPY</a>',
-                '<a href="../EDS/Search?lookfor=%22METAPHOR%2E%22&amp;type=SU">METAPHOR.</a>',
+                'PSYCHOTHERAPY',
+                'METAPHOR',
             ],
             $driver->getAllSubjectHeadingsFlattened()
         );