]> git.koha-community.org Git - koha.git/commit
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)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 11 Jun 2018 08:56:43 +0000 (09:56 +0100)
commita28c4056c63ab40a451ef434577c9905e781c3b5
tree52d6719669a910988857e2324dad34d64f93703a
parent95235be2bdcf44f6efe77c488871bb44bb5d657b
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>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
C4/Circulation.pm