]> git.koha-community.org Git - koha.git/commit
Bug 20996: Remove prefix use of borrower category
authorAndrew Isherwood <andrew.isherwood@ptfs-europe.com>
Tue, 3 Jul 2018 10:36:58 +0000 (11:36 +0100)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 2 Nov 2018 10:33:02 +0000 (10:33 +0000)
commit1c8952a36b3b810b7bc290a7eb7cc85cc766ed61
tree552a8bc499cceb80e5f979daddfe535d183fd16e
parent7085772e8de25fe49b77555e318cc03ab6bcb33b
Bug 20996: Remove prefix use of borrower category

This patch removes the potential use of borrower category as a ILL
request ID prefix. It makes no sense. We provide the ability for a site
to define a request prefix based on branch, there is no use case for
using the borrower category. Add to this that the borrower for every
request was being retrieved in order to get the category, it's a huge
performance hit also.

We also now require the <branch> block in the <interlibrary_loans> block
and complain if it's not present. The request prefix should be defined
in this block.

This patch also improves the performance of the API request that returns all
requests, optionally including additional data.

It also deprecates the overloaded TO_JSON method and moves the request
augmentation code into the API route's controller. It may be that we
want to shift it out of there at some point, but it is fine where it is
for now.

Signed-off-by: Magnus Enger <magnus@libriotech.no>
Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Koha/Illrequest.pm
Koha/Illrequest/Config.pm
Koha/REST/V1/Illrequests.pm
about.pl
ill/ill-requests.pl
koha-tmpl/intranet-tmpl/prog/en/modules/about.tt
koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt