diff --git a/module/VuFind/src/VuFind/Auth/LDAP.php b/module/VuFind/src/VuFind/Auth/LDAP.php index 699402a6a06204be05f0af50266efcd8be3b44d9..451cf553685ef8ba4e6ae908afaa90aebe76a3b6 100644 --- a/module/VuFind/src/VuFind/Auth/LDAP.php +++ b/module/VuFind/src/VuFind/Auth/LDAP.php @@ -279,7 +279,7 @@ class LDAP extends AbstractBase if ($data[$i][$j] == $configValue && !empty($configValue)) { $value = $data[$i][$configValue][0]; $this->debug("found $field = $value"); - if ($field != "cat_password" ) { + if ($field != "cat_password") { $user->$field = $value; } else { $catPassword = $value; diff --git a/module/VuFind/src/VuFind/ILS/Driver/Aleph.php b/module/VuFind/src/VuFind/ILS/Driver/Aleph.php index d07107c515ff23fe92bbc44bbe3ffccfcf739cc2..a55a72cc889b0816ce6cdf88ae13a1a5109437a2 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/Aleph.php +++ b/module/VuFind/src/VuFind/ILS/Driver/Aleph.php @@ -85,7 +85,7 @@ class AlephTranslator $result = array(); $file_handle = fopen($file, "r, ccs=UTF-8"); $rgxp = ""; - while (!feof($file_handle) ) { + while (!feof($file_handle)) { $line = fgets($file_handle); $line = chop($line); if (preg_match("/!!/", $line)) { diff --git a/module/VuFind/src/VuFind/ILS/Driver/Amicus.php b/module/VuFind/src/VuFind/ILS/Driver/Amicus.php index 968b93163fed0df9e65b774f2c88c119406ce4cd..5bbf267770a49c4165ffc6ef4eae168ab4b0bd32 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/Amicus.php +++ b/module/VuFind/src/VuFind/ILS/Driver/Amicus.php @@ -337,7 +337,7 @@ class Amicus extends AbstractBase implements TranslatorAwareInterface $prestados = $this->sacaStatus($row['CPY_ID_NBR']); $reservados = $this->sacaReservas($row['CPY_ID_NBR']); if (!isset($data[$row['BIB_ITM_NBR']])) { - if ($multiple != 1 ) { + if ($multiple != 1) { $multiple = $this->translate("Multiple Locations"); $textoLoc = $this->translate("Multiple"); $textoSign = $this->translate("Multiple Locations"); diff --git a/module/VuFind/src/VuFind/ILS/Driver/HorizonXMLAPI.php b/module/VuFind/src/VuFind/ILS/Driver/HorizonXMLAPI.php index a568765c0f1d7a7817c37b158c829d12670c25c2..b2c4c6880e70003a656a34255ffe6a4766c4ba96 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/HorizonXMLAPI.php +++ b/module/VuFind/src/VuFind/ILS/Driver/HorizonXMLAPI.php @@ -84,7 +84,7 @@ class HorizonXMLAPI extends Horizon implements \VuFindHttp\HttpServiceAwareInter */ public function getConfig($function, $params = null) { - if (isset($this->config[$function]) ) { + if (isset($this->config[$function])) { $functionConfig = $this->config[$function]; } else { $functionConfig = false; diff --git a/module/VuFind/src/VuFind/ILS/Driver/LBS4.php b/module/VuFind/src/VuFind/ILS/Driver/LBS4.php index 3a03c59be99c03f2edaabc0469566b24986debc3..fe1049afa0dc59b931d9a206ce581ec4c5a1b773 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/LBS4.php +++ b/module/VuFind/src/VuFind/ILS/Driver/LBS4.php @@ -351,7 +351,7 @@ class LBS4 extends AbstractBase implements TranslatorAwareInterface */ protected function getNote($loanind, $locid, $callnumber) { - if ( $loanind == 0 && $locid=='000') { + if ($loanind == 0 && $locid=='000') { $note = $this->translate("Textbook Collection"); } else if ($loanind == 1) { $note = $this->translate("Short loan");//Short time loan? diff --git a/module/VuFind/src/VuFind/ILS/Driver/Polaris.php b/module/VuFind/src/VuFind/ILS/Driver/Polaris.php index 32bce071e595a2e9c44974b628058bd4b33aa10a..11990fc54f12e06aaee47aefe56a06b9110d2756 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/Polaris.php +++ b/module/VuFind/src/VuFind/ILS/Driver/Polaris.php @@ -298,7 +298,7 @@ class Polaris extends AbstractBase implements \VuFindHttp\HttpServiceAwareInterf */ public function getConfig($function, $params = null) { - if (isset($this->config[$function]) ) { + if (isset($this->config[$function])) { $functionConfig = $this->config[$function]; } else { $functionConfig = false; diff --git a/module/VuFind/src/VuFind/ILS/Driver/Symphony.php b/module/VuFind/src/VuFind/ILS/Driver/Symphony.php index 39d8c60937f36e8377437884cd66657ace87cb85..ab801d1fb74a0e16557fd61ee9b8d0c784ebf1a4 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/Symphony.php +++ b/module/VuFind/src/VuFind/ILS/Driver/Symphony.php @@ -1448,7 +1448,7 @@ class Symphony extends AbstractBase */ public function getConfig($function, $params = null) { - if (isset($this->config[$function]) ) { + if (isset($this->config[$function])) { $functionConfig = $this->config[$function]; } else { $functionConfig = false; diff --git a/module/VuFind/src/VuFind/ILS/Driver/Unicorn.php b/module/VuFind/src/VuFind/ILS/Driver/Unicorn.php index e3e65bc00b87fbed9a8f0aab28f99201576203de..2b713d0212e84a99729d5f5f827bc2d52571ee55 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/Unicorn.php +++ b/module/VuFind/src/VuFind/ILS/Driver/Unicorn.php @@ -132,7 +132,7 @@ class Unicorn extends AbstractBase implements \VuFindHttp\HttpServiceAwareInterf */ public function getConfig($function, $params = null) { - if (isset($this->config[$function]) ) { + if (isset($this->config[$function])) { $functionConfig = $this->config[$function]; } else { $functionConfig = false; diff --git a/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php b/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php index 42f8cb20c4fe3524b5cfb12a1fd5b9694e5383fc..1feaf83089661cf0d9c5bf719fe7073046291b12 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php +++ b/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php @@ -312,7 +312,7 @@ class VoyagerRestful extends Voyager implements \VuFindHttp\HttpServiceAwareInte */ public function getConfig($function, $params = null) { - if (isset($this->config[$function]) ) { + if (isset($this->config[$function])) { $functionConfig = $this->config[$function]; } else { $functionConfig = false; @@ -1393,7 +1393,7 @@ EOT; protected function checkItemRequests($patronId, $request, $bibId, $itemId = false ) { - if (!empty($bibId) && !empty($patronId) && !empty($request) ) { + if (!empty($bibId) && !empty($patronId) && !empty($request)) { $hierarchy = array(); @@ -1423,7 +1423,7 @@ EOT; // Valid Response if ($reply == "ok") { - if ($check->$request ) { + if ($check->$request) { $requestAttributes = $check->$request->attributes(); if ($requestAttributes['allowed'] == "Y") { return true; diff --git a/module/VuFind/tests/unit-tests/src/VuFindTest/Recommend/WorldCatTermsTest.php b/module/VuFind/tests/unit-tests/src/VuFindTest/Recommend/WorldCatTermsTest.php index 774ce677d0f400a5359e2a093bffdcd7028d5813..fd467b311602550914b163011ced596c85ac93f8 100644 --- a/module/VuFind/tests/unit-tests/src/VuFindTest/Recommend/WorldCatTermsTest.php +++ b/module/VuFind/tests/unit-tests/src/VuFindTest/Recommend/WorldCatTermsTest.php @@ -59,7 +59,7 @@ class WorldCatTermsTest extends \VuFindTest\Unit\TestCase $module->setConfig(''); $module->init($results->getParams(), $request); $module->process($results); - $this->assertEquals( array('exact' => 'exact'), $module->getTerms()); + $this->assertEquals(array('exact' => 'exact'), $module->getTerms()); } /** diff --git a/module/VuFindConsole/src/VuFindConsole/Controller/HarvestController.php b/module/VuFindConsole/src/VuFindConsole/Controller/HarvestController.php index 40de9c8a6087ff0badeec93deb610d333bc36139..887b7baf3aabe9b9f1a0a47fa36f926ec94fcea0 100644 --- a/module/VuFindConsole/src/VuFindConsole/Controller/HarvestController.php +++ b/module/VuFindConsole/src/VuFindConsole/Controller/HarvestController.php @@ -132,7 +132,7 @@ class HarvestController extends AbstractBase Console::writeLine('<collection>'); while (false !== ($file = readdir($handle))) { // Only operate on XML files: - if (pathinfo($file, PATHINFO_EXTENSION) === "xml" ) { + if (pathinfo($file, PATHINFO_EXTENSION) === "xml") { // get file content $filePath = $dir . '/' . $file; $fileContent = file_get_contents($filePath); diff --git a/module/VuFindSearch/src/VuFindSearch/Backend/EIT/Backend.php b/module/VuFindSearch/src/VuFindSearch/Backend/EIT/Backend.php index 98e5abb766fdcc64121a9b4dc40bb5a3baf4ff3f..a06a1c4c22e4915368000880d3e460d32ae26f99 100644 --- a/module/VuFindSearch/src/VuFindSearch/Backend/EIT/Backend.php +++ b/module/VuFindSearch/src/VuFindSearch/Backend/EIT/Backend.php @@ -91,7 +91,7 @@ class Backend extends AbstractBackend public function search(AbstractQuery $query, $offset, $limit, ParamBag $params = null ) { - if (null === $params ) { + if (null === $params) { $params = new ParamBag(); } $params->mergeWith($this->getQueryBuilder()->build($query));