diff --git a/module/VuFind/src/VuFind/Resolver/Driver/Redi.php b/module/VuFind/src/VuFind/Resolver/Driver/Redi.php index c51805fe6a40ed295a59708c53dad67167af7eeb..54ada2c19da63e54865f36cb83521235642cd76f 100644 --- a/module/VuFind/src/VuFind/Resolver/Driver/Redi.php +++ b/module/VuFind/src/VuFind/Resolver/Driver/Redi.php @@ -86,7 +86,7 @@ class Redi implements DriverInterface */ public function fetchLinks($openURL) { - $url = $this->baseUrl.'?'.$openURL; + $url = $this->baseUrl . '?' . $openURL; $feed = $this->httpClient->setUri($url)->send()->getBody(); return $feed; } @@ -139,7 +139,7 @@ class Redi implements DriverInterface ->query("//dd[@class='doi_d']"); if ($doiTerm->length == $doiDefinition->length) { - for ($i=0; $i<$doiTerm->length; $i++) { + for ($i = 0; $i<$doiTerm->length; $i++) { $href = $xpath ->query(".//@href", $doiDefinition->item($i)) ->item(0)->textContent; @@ -172,7 +172,7 @@ class Redi implements DriverInterface $infoTokenNodes = $xpath->query("//div[@id='t_ezb']/div[@class='t']/p/sup"); if ($infoTokenNodes->length > 0) { - for ($i=0; $i<$infoTokenNodes->length; $i++) { + for ($i = 0; $i<$infoTokenNodes->length; $i++) { if ($infoToken == $infoTokenNodes->item($i)->textContent) { return $xpath ->query("//div[@id='t_ezb']/div[@class='t']/p/sup/..") @@ -204,7 +204,7 @@ class Redi implements DriverInterface ->query("//div[@class='t_ezb_result']/p/span[@class='t_link']/a"); if ($ezbResultsNodesText->length == $ezbResultsNodesURL->length) { - for ($i=0; $i<$ezbResultsNodesText->length; $i++) { + for ($i = 0; $i<$ezbResultsNodesText->length; $i++) { $itemInfo = ''; @@ -236,7 +236,7 @@ class Redi implements DriverInterface */ protected function postProcessing() { - for ($i=0; $i < count($this->links); $i++) { + for ($i = 0; $i < count($this->links); $i++) { if (isset($this->links[$i]['title'])) { $this->links[$i]['title'] = $this ->removeDoubleAngleQuotationMarks($this->links[$i]['title']); @@ -258,7 +258,7 @@ class Redi implements DriverInterface protected function removeDoubleAngleQuotationMarks($string) { return str_replace( - ['»', chr(194).chr(160)], + ['»', chr(194) . chr(160)], ['', ''], $string ); // hack to replace \u00a0 diff --git a/module/VuFind/tests/unit-tests/src/VuFindTest/Resolver/Driver/RediTest.php b/module/VuFind/tests/unit-tests/src/VuFindTest/Resolver/Driver/RediTest.php index 95bc1fc648ed57dc8aaf6aa0851262f5ca2817fc..f218fa337076442609e232190f294e01fccb8376 100644 --- a/module/VuFind/tests/unit-tests/src/VuFindTest/Resolver/Driver/RediTest.php +++ b/module/VuFind/tests/unit-tests/src/VuFindTest/Resolver/Driver/RediTest.php @@ -93,7 +93,7 @@ class RediTest extends \VuFindTest\Unit\TestCase 'title' => "Zur Zeitschriftenhomepage* (via www.ncbi.nlm.nih.gov)", 'href' => "http://www-fr.redi-bw.de/links/?rl_site=ubl&rl_action=link&rl_link_target=ezb&rl_link_name=1.article&rl_citation=9443914d0e261c0c1f6a3fd8151213c1d4cec05f5d3053097da6fa5597bbb9d7", 'coverage' => "*Es konnte nicht zuverlässig festgestellt werden, ob der gesuchte Aufsatz in den Zeitraum fällt, für den bei diesem Anbieter der Volltext verfügbar ist.", - 'service_type' =>"getFullTxt", + 'service_type' => "getFullTxt", ], ];