LP#1736419: Located URIs vs QueryParser, round 2
authorMike Rylander <mrylander@gmail.com>
Thu, 4 Jan 2018 23:12:17 +0000 (18:12 -0500)
committerKathy Lussier <klussier@masslnc.org>
Tue, 9 Jan 2018 23:59:24 +0000 (18:59 -0500)
commit13d0596d1500c03c10e2fb600edf473542354eb5
treea07951addbb0596170ada8da4be679346050f99d
parentbfd5b2f43a0d603972440d727310c7f03a5583e1
LP#1736419: Located URIs vs QueryParser, round 2

The site() filter and #staff modifier are used to decide when and how to
include certain query filters, such as circ_lib or luri_org. Unfortunately,
site() is sometimes excluded (whole-tree search) and the test for staff-
iness was not specific enough, leading to incorrect queries in those cases
where information was lacking.  Now, we treat site() specially, forcing a
default of "top-of-tree", and we check for the #staff modifier directly
where necessary.

Note: this commit also addresses LP#1736992 which is about staff searches
using the limit-to-available modifier.  As a side effect of the special
site() treatment, that issue is resolved.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Driver/Pg/QueryParser.pm