diff --git a/module/VuFind/src/VuFind/ILS/Driver/NoILS.php b/module/VuFind/src/VuFind/ILS/Driver/NoILS.php index 2a82ba87d0846bfca6a1dc3c826ae0fbffe04e73..0b7ec1ffb822062718c19b696b143c44a8949286 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/NoILS.php +++ b/module/VuFind/src/VuFind/ILS/Driver/NoILS.php @@ -89,7 +89,7 @@ class NoILS extends AbstractBase implements TranslatorAwareInterface */ public function getConfig($function, $params = null) { - return isset($this->config[$function]) ? $this->config[$function] : false; + return $this->config[$function] ?? false; } /** @@ -99,8 +99,7 @@ class NoILS extends AbstractBase implements TranslatorAwareInterface */ protected function getIdPrefix() { - return isset($this->config['settings']['idPrefix']) - ? $this->config['settings']['idPrefix'] : null; + return $this->config['settings']['idPrefix'] ?? null; } /** @@ -131,8 +130,7 @@ class NoILS extends AbstractBase implements TranslatorAwareInterface */ public function getStatus($id) { - $useStatus = isset($this->config['settings']['useStatus']) - ? $this->config['settings']['useStatus'] : 'none'; + $useStatus = $this->config['settings']['useStatus'] ?? 'none'; if ($useStatus == "custom") { $status = $this->translate($this->config['Status']['status']); return [ @@ -173,8 +171,7 @@ class NoILS extends AbstractBase implements TranslatorAwareInterface */ public function getStatuses($idList) { - $useStatus = isset($this->config['settings']['useStatus']) - ? $this->config['settings']['useStatus'] : 'none'; + $useStatus = $this->config['settings']['useStatus'] ?? 'none'; if ($useStatus == "custom" || $useStatus == "marc") { $status = []; foreach ($idList as $id) { @@ -201,8 +198,7 @@ class NoILS extends AbstractBase implements TranslatorAwareInterface */ public function getHolding($id, array $patron = null) { - $useHoldings = isset($this->config['settings']['useHoldings']) - ? $this->config['settings']['useHoldings'] : 'none'; + $useHoldings = $this->config['settings']['useHoldings'] ?? 'none'; if ($useHoldings == "custom") { return [ @@ -250,8 +246,7 @@ class NoILS extends AbstractBase implements TranslatorAwareInterface */ protected function getFormattedMarcDetails($recordDriver, $configSection) { - $marcStatus = isset($this->config[$configSection]) - ? $this->config[$configSection] : false; + $marcStatus = $this->config[$configSection] ?? false; if ($marcStatus) { $field = $marcStatus['marcField']; unset($marcStatus['marcField']); @@ -285,8 +280,7 @@ class NoILS extends AbstractBase implements TranslatorAwareInterface */ public function hasHoldings($id) { - $useHoldings = isset($this->config['settings']['useHoldings']) - ? $this->config['settings']['useHoldings'] : ''; + $useHoldings = $this->config['settings']['useHoldings'] ?? ''; // "none" will be processed differently in the config depending // on whether it's in or out of quotes; handle both cases. @@ -342,8 +336,7 @@ class NoILS extends AbstractBase implements TranslatorAwareInterface */ public function getOfflineMode() { - return isset($this->config['settings']['mode']) - ? $this->config['settings']['mode'] : "ils-offline"; + return $this->config['settings']['mode'] ?? 'ils-offline'; } /** @@ -355,8 +348,7 @@ class NoILS extends AbstractBase implements TranslatorAwareInterface */ public function loginIsHidden() { - return isset($this->config['settings']['hideLogin']) - ? $this->config['settings']['hideLogin'] : false; + return $this->config['settings']['hideLogin'] ?? false; } /**