diff --git a/module/VuFind/src/VuFind/ILS/Driver/Symphony.php b/module/VuFind/src/VuFind/ILS/Driver/Symphony.php index 9ede1a56b5ddf9e87a35a85c85a0e8b395dcc028..32b996e48d890d9f9be8a4a665c0a6b650c78914 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/Symphony.php +++ b/module/VuFind/src/VuFind/ILS/Driver/Symphony.php @@ -452,6 +452,23 @@ class Symphony extends AbstractBase implements ServiceLocatorAwareInterface return $this->makeRequest('standard', 'lookupTitleInfo', $params); } + /** + * Determine if a library is excluded by LibraryFilter configuration. + * + * @param string $libraryID the ID of the library in question + * @return boolean true if excluded, false if not + */ + protected function libraryIsFilteredOut($libraryID) { + $notInWhitelist = !empty($this->config['LibraryFilter']['include_only']) + && !in_array( + $libraryID, $this->config['LibraryFilter']['include_only'] + ); + $onBlacklist = in_array( + $libraryID, $this->config['LibraryFilter']['exclude'] + ); + return $notInWhitelist || $onBlacklist; + } + /** * Parse Call Info * @@ -475,15 +492,7 @@ class Symphony extends AbstractBase implements ServiceLocatorAwareInterface $libraryID = $callInfo->libraryID; $library = $this->translatePolicyID('LIBR', $libraryID); - $notInWhitelist = !empty($this->config['LibraryFilter']['include_only']) - && !in_array( - $libraryID, $this->config['LibraryFilter']['include_only'] - ); - $onBlacklist = in_array( - $libraryID, $this->config['LibraryFilter']['exclude'] - ); - - if ($notInWhitelist || $onBlacklist) { + if ($this->libraryIsFilteredOut($libraryID)) { continue; }