From 1e6c97edeacdedbb2e6eb997471a802eed60fe8a Mon Sep 17 00:00:00 2001
From: Ere Maijala <ere.maijala@helsinki.fi>
Date: Mon, 26 Nov 2018 17:38:50 +0200
Subject: [PATCH] SierraRest: Fix canceling holds and display of position in
hold queue. (#1271)
---
module/VuFind/src/VuFind/ILS/Driver/SierraRest.php | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/module/VuFind/src/VuFind/ILS/Driver/SierraRest.php b/module/VuFind/src/VuFind/ILS/Driver/SierraRest.php
index 6ab8c7b8f60..1f013f66797 100644
--- a/module/VuFind/src/VuFind/ILS/Driver/SierraRest.php
+++ b/module/VuFind/src/VuFind/ILS/Driver/SierraRest.php
@@ -783,7 +783,7 @@ class SierraRest extends AbstractBase implements TranslatorAwareInterface,
$position = $entry['priorityQueueLength'] . ' / '
. $entry['priorityQueueLength'];
} else {
- $position = ($entry['priority'] + 1) . ' / '
+ $position = $entry['priority'] . ' / '
. $entry['priorityQueueLength'];
}
$lastPickup = !empty($entry['pickupByDate'])
@@ -826,7 +826,7 @@ class SierraRest extends AbstractBase implements TranslatorAwareInterface,
public function getCancelHoldDetails($holdDetails)
{
return $holdDetails['available'] || $holdDetails['in_transit'] ? ''
- : $holdDetails['item_id'];
+ : $holdDetails['requestId'];
}
/**
@@ -849,7 +849,7 @@ class SierraRest extends AbstractBase implements TranslatorAwareInterface,
foreach ($details as $holdId) {
$result = $this->makeRequest(
- ['v5', 'patrons', 'holds', $holdId], [], 'DELETE', $patron
+ ['v5', 'patrons', 'holds', $holdId], '', 'DELETE', $patron
);
if (!empty($result['code'])) {
--
GitLab