| commit | 0bf785018d18c3c9a548c87ce2769032e181d348 | |
| author | Jonathan Druart <jonathan.druart@bugs.koha-community.org> | |
| Sun, 12 May 2019 16:51:28 +0000 (12 11:51 -0500) | ||
| committer | Nick Clemens <nick@bywatersolutions.com> | |
| Tue, 14 May 2019 19:37:00 +0000 (14 19:37 +0000) | ||
| tree | dc7674ae666d2daf23872241a8dd34e238582811 | treesnapshot (tar.gz zip) |
| parent | bc0d3e1a34e78fcf00f512e2bdff73c9de0bea8c | commitdiff |
Bug 22891: (bug 21206 follow-up) Fix RenewHold (ILS-DI)
From
commit 31c29fd31f557306233e6a2936148a5bb10b89a1
Bug 21206: Replace C4::Items::GetItem
A missing 'unless' test made RenewHold return prematurely:
return $issue;
should have been
return unless $issue;
Test plan:
Hit /cgi-bin/koha/ilsdi.pl?service=RenewLoan&patron_id=51&item_id=756
With different situations:
1. Item is checked out to the patron
2. Item is not checked out to the patron
3. Item does not exist
4. Patron does not exist
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
From
commit 31c29fd31f557306233e6a2936148a5bb10b89a1
Bug 21206: Replace C4::Items::GetItem
A missing 'unless' test made RenewHold return prematurely:
return $issue;
should have been
return unless $issue;
Test plan:
Hit /cgi-bin/koha/ilsdi.pl?service=RenewLoan&patron_id=51&item_id=756
With different situations:
1. Item is checked out to the patron
2. Item is not checked out to the patron
3. Item does not exist
4. Patron does not exist
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
| C4/ILSDI/Services.pm | diffblobblamehistory |