Bug 20825: (bug 19943 follow-up) call notforloan on itemtype instead of biblioitem
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 28 May 2018 14:24:43 +0000 (11:24 -0300)
committerNick Clemens <nick@bywatersolutions.com>
Tue, 29 May 2018 11:37:32 +0000 (11:37 +0000)
commit96070c9ea431063052f8c360f52032db6568c139
tree6825acb750f6fb244de6cb6db92c4e521e80da40
parentb64262af2a237505c775739e7cbd2465637498ec
Bug 20825: (bug 19943 follow-up) call notforloan on itemtype instead of biblioitem

On bug 19943:
 -        elsif ($biblioitem->{'notforloan'} == 1){
 +        elsif ($biblioitem->notforloan == 1){

The biblioitems table does not contain a notforloan column, this comes
from the item type.

This bug only appears when item type is defined at biblio level
(item-level_itypes=0)

Test plan:
Set item-level_itypes = biblio
Check an item out

Without this patch it explodes with
"The method notforloan is not covered by tests!"

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
I reproduced the error condition and verified the tests failed without
this patch. After this patch is applied, tests pass and checkout
succeeds.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
C4/Circulation.pm