diff --git a/module/fid/src/Controller/MyResearchController.php b/module/fid/src/Controller/MyResearchController.php index e0e032ed4b256207679c326f6c3484d69bad4dba..71002c9c96a732bb023cdec61081db0af616c759 100644 --- a/module/fid/src/Controller/MyResearchController.php +++ b/module/fid/src/Controller/MyResearchController.php @@ -105,7 +105,9 @@ class MyResearchController extends \VuFind\Controller\MyResearchController try { $library = $this->getHomeLibrary($patron); - $view->homeLibrary = $library->getDbis(); + if (!is_null($library)) { + $view->homeLibrary = $library->getDbis(); + } $view->map_bibid_to_dbis = false; } catch (ClientException $exception) { // TODO: Implement logging (in database?) @@ -159,14 +161,14 @@ class MyResearchController extends \VuFind\Controller\MyResearchController protected function getHomeLibrary($patron): ?Library { if (is_a($patron['user'], 'fid\Service\DataTransferObject\User') - && $patron['user']->getHomeLibrary()) + && $homeLibrary = $patron['user']->getHomeLibrary()) { - if (isset($patron['libs'][$patron['user']->getHomeLibrary()])) { - return $patron['libs'][$patron['user']->getHomeLibrary()]; + if (isset($patron['libs'][$homeLibrary])) { + return $patron['libs'][$homeLibrary]; } /* no home library found in patron object? => fetch library from api directly */ - return $this->fidClient->requestLibraryById($patron['user']->getHomeLibrary()); + return $this->fidClient->requestLibraryById($homeLibrary); } return null;