Bug 33447: (follow-up) Fix tests and make assumption explicit
authorNick Clemens <nick@bywatersolutions.com>
Thu, 20 Apr 2023 15:44:53 +0000 (15:44 +0000)
committerJacob O'Mara <jacobomara901@gmail.com>
Tue, 9 May 2023 20:46:55 +0000 (21:46 +0100)
commit4e51e04ce150896a6429ccf9af07d6460efb89e1
treed32c6f0d1ca36d0a6ec15dd15df627c40b6da87b
parent5eec29d2dabfb7a08ed092032f8222f527cc5041
Bug 33447: (follow-up) Fix tests and make assumption explicit

The patches made an assumption that patron would always be passed. It is
within Koha, but not in the Biblios tests.

There is no scenario where we can determine pickup locations that are not in
reference to a patron (who is picking it up?) so we should always have
this parameter

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit ef8adf34fde2352680934ba6a36de89e3f905a36)
Signed-off-by: Jacob O'Mara <jacobomara901@gmail.com>
Koha/Biblio.pm
Koha/Biblios.pm
Koha/Item.pm
t/db_dependent/Koha/Biblios.t