Hide OPAC-invisible holdings from SRU/Z39.50 user/dbs/sru_hide_opac_invisible
authorDan Scott <dscott@laurentian.ca>
Mon, 16 Jul 2012 17:27:54 +0000 (13:27 -0400)
committerDan Scott <dscott@laurentian.ca>
Tue, 17 Jul 2012 17:15:47 +0000 (13:15 -0400)
commit6abb6470c0a24d8d8ac95eae856bb03bcd764d7e
tree7b2d61e928e354c28213e60fe4a2539beab9757b
parent961037561d3fe7b70fbe9d7a196502527374d8f6
Hide OPAC-invisible holdings from SRU/Z39.50

The basic_holdings Supercat method filtered out deleted call numbers and
copies, but didn't filter out copies based on the OPAC visibility status
of copy location / status / the copy itself. This has undesired results
when third-party services request copies via SRU or Z39.50 and expose
copies that should not be visible to the public.

We wrap all of the visibility and deletedness checks for a given copy up
into a subroutine so that we can avoid repeating ourselves in the
basic_holdings logic. Also, if we missed a test, we can add it in one
handy place :)

(Thanks to Galen Charlton for finding two missing tests: circ lib
visibility and owning lib visibility!)

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Open-ILS/src/perlmods/lib/OpenILS/Application/SuperCat.pm