]> git.koha-community.org Git - koha.git/commit
Bug 13297 - Shelving location PROC does not work according to manual
authorKyle M Hall <kyle@bywatersolutions.com>
Wed, 19 Nov 2014 12:24:20 +0000 (07:24 -0500)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Sun, 23 Nov 2014 13:11:28 +0000 (10:11 -0300)
commit4c1ae53d861e20051a776f0195f5da1cfa9a24e3
tree9c18d8dac92bb4ae4eb71d2b8b219a79eb61ea1c
parent1cb86a2b781d2d16f877e7b97ff789aab7a40684
Bug 13297 - Shelving location PROC does not work according to manual

According to the manual, "Items will stay in the PROC location until
they are checked in".

This is not the actual behavior. Right now items will only change from
PROC to CART, and that is only if InProcessingToShelvingCart is enabled.
Some libraries want to use the PROC to permanent location feature,
without using the CART.

Additionally, the location is only removed if using returns.pl, but
that is not what the manual says either. What if the library uses
SIP2 devices for handling returns? This should be taken into
account.

Test Plan:
1) Apply this patch
2) Set an item's current location to PROC, and it's permananet location
   to a different location.
3) Check the item in any way you wish
4) Note the shelving location is updated to the permanent location
5) prove t/db_dependent/Circulation/Returns.t

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
I tested this with items which had items.location set to 'PROC' and
items.permanent_location set to NULL, '', and a real value, and it
worked correctly in all cases. I tested with check-ins from returns.pl
and from the table of checkouts in circulation and the PROC location was
correctly removed in both cases.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
C4/Circulation.pm
circ/returns.pl
t/db_dependent/Circulation/Returns.t [new file with mode: 0644]