Hide OPAC-invisible holdings from SRU/Z39.50 user/dbs/sru_hide_opac_invisible_2_1
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:18:54 +0000 (13:18 -0400)
commitfc1ecdbd1ff1fb13e9a12e3b709ac9bc468a0207
treeb2bcb761eb2acc3b87f3800181c6834897ea484f
parent192ab45b37d5a37187d37a253a420c3578adf570
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