Bug 21301: Remove patron informations in reserves for GetRecords ILS-DI service

Actually, the GetRecords service gives many information :
borrowernumber (reserves field)

The issue data should not be sent by these service, its a privacy problem.

Test plan :
1) Enable ILS-DI webservice
2) Place hold on an item
3) Go to {opac}/cgi-bin/koha/ilsdi.pl?service=GetRecords&id={biblionumber}
4) Check the reserves tag content

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
Fridolin Somers 2019-10-03 15:43:56 +02:00 committed by Jonathan Druart
parent 8ae0ef6eb5
commit 37c042b9f3
1 changed files with 6 additions and 1 deletions

View File

@ -227,7 +227,6 @@ sub GetRecords {
# Get most of the needed data
my $biblioitemnumber = $biblioitem->{'biblioitemnumber'};
my $holds = $biblio->current_holds->unblessed;
my $checkouts = Koha::Checkouts->search(
{ biblionumber => $biblionumber },
{
@ -279,6 +278,12 @@ sub GetRecords {
push @{ $biblioitem->{items}->{item} }, \%item;
}
# Holds
my $holds = $biblio->current_holds->unblessed;
foreach my $hold (@$holds) {
delete $hold->{'borrowernumber'};
}
# Hashref building...
$biblioitem->{'reserves'}->{'reserve'} = $holds;
$biblioitem->{'issues'}->{'issue'} = $checkouts;