]> git.koha-community.org Git - koha.git/commit
Bug 13636 - Staff search results item status incorrect for holds
authorKyle M Hall <kyle@bywatersolutions.com>
Wed, 28 Jan 2015 13:31:30 +0000 (08:31 -0500)
committerChris Cormack <chris@bigballofwax.co.nz>
Sat, 14 Feb 2015 07:59:10 +0000 (20:59 +1300)
commite8c4bd8ab4e8986bfe2cfae58e9b14ac666bfe3f
treec2ddf8af0aef3f4f93eb08ea6e8badec9fe54d89
parentfa74808736d9aea8df6fff1bf9e146c62d95046c
Bug 13636 - Staff search results item status incorrect for holds

Imagine this scenario: we have one record with four items. Two of those
items are checked out, one of those items is a waiting hold, and one of
those items is available. We would expect to see this on the search
results page. Instead, we will see both non-checked out items as
unavailable due to waiting holds.

This is due to a semantic issue GetReserveStatus.
C4::Search::searchResults uses GetReserveStatus to get the reserve
status of each item, but unlike all other calls to the sub, this one
passes in not only itemnumber, but biblionumber.

When no reserve is found for the available item, the subroutine uses the
biblionumber to grab what is essentially an arbitrary reserve to use for
the status. This makes no sense and this functionality should be
entirely removed from the subroutine so regressions like this will be
prevented in the future.

Test Plan:
1) Create one record with 4 items
   a) check two of the items out to patrons
   b) set one of the items as a waiting hold
   c) leave the fourth item as available
2) Run a search where this record will be in the results list
3) Note that the results list 2 items on loan, two unavailable
4) Apply this patch, reload the search results
5) Note that the results list 1 available, 2 on loan, 1 unavailable

Signed-off-by: John Andrews <jandrews@washoecounty.us>
Signed-off-by: Sheila Kearns <sheila.kearns@state.vt.us>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Note: This is for the staff search result list!

Works as expected.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
(cherry picked from commit fcaa6f35c080c079ba422a70dec49ca327346c82)
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
C4/Reserves.pm
C4/Search.pm