diff --git a/module/VuFind/src/VuFind/Controller/Plugin/Renewals.php b/module/VuFind/src/VuFind/Controller/Plugin/Renewals.php index 888c8dc840d659be92a1c1cf8edb2500cb61b853..20c006454f21c59634424b69f38b5ef6aa1375c9 100644 --- a/module/VuFind/src/VuFind/Controller/Plugin/Renewals.php +++ b/module/VuFind/src/VuFind/Controller/Plugin/Renewals.php @@ -113,7 +113,7 @@ class Renewals extends AbstractPlugin return $renewResult['details']; } else { // System failure: - $flashMsg->setNamespace('error')->addMessage('renew_system_error'); + $flashMsg->setNamespace('error')->addMessage('renew_error'); } } else if (!empty($all) || !empty($selected)) { // Button was clicked but no items were selected: diff --git a/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php b/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php index 9d99c673b607dc4bbbbf5cb7d4a820982e92faf7..8371457fe325de56e02dae5a90754ff12e4bd0d4 100644 --- a/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php +++ b/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php @@ -1300,7 +1300,7 @@ EOT; array('AuthenticatePatronService' => false), array(), 'POST', $xml ); if ($response === false) { - throw new ILSException('renew_error_system'); + throw new ILSException('renew_error'); } $xml = <<<EOT @@ -1324,7 +1324,7 @@ EOT; array('RenewService' => false), array(), 'POST', $xml ); if ($response === false) { - throw new ILSException('renew_error_system'); + throw new ILSException('renew_error'); } // Process @@ -1336,7 +1336,9 @@ EOT; // The service doesn't actually return messages (in Voyager 8.1), // but maybe in the future... foreach ($response->xpath('//ser:message') as $message) { - if ($message->attributes()->type == 'system') { + if ($message->attributes()->type == 'system' + || $message->attributes()->type == 'error' + ) { return false; } }