Bug 20889: Prevent not for loan items to be checked out
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 6 Jun 2018 17:25:19 +0000 (14:25 -0300)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 8 Jun 2018 10:19:41 +0000 (10:19 +0000)
commit6049836023be61771c25fe038f02378ede96fff5
tree5666a022c482ec31d36100d905153d3348f5b1bd
parent326aa0667d86dc8c0d8a8becf440bfb25d2c76c1
Bug 20889: Prevent not for loan items to be checked out

1. Item type defined at item level (item-level_itypes=1)
2. Mark an item type not for loan (itemtypes.notforloan=1)
3. Checkout an item using this item type (items.itype="BK" for instance)
=> Checkout is not blocked!

I suspect
  commit 3953fdb921d7f280ffbaca813956357aa67d42ca
  Bug 19943: Remove itemtype vs itype confusion in CanBookBeIssued
to be the root of this issue.
One occurrence of $item->{itemtype} has not been replaced.
In this case it refers to the biblioitem->{itemtype} value whereas we want to use
$item->{itype}. So this issue does not happen if items.itype==biblioitem.itemtype
(just in case you are not reproducing the problem).

Test plan:
Make sure not for loan items cannot be checked out

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

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

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