LP#1736419: Located URIs vs QueryParser, round 2
authorMike Rylander <mrylander@gmail.com>
Thu, 4 Jan 2018 23:12:17 +0000 (18:12 -0500)
committerChris Sharp <csharp@georgialibraries.org>
Mon, 6 Aug 2018 15:16:27 +0000 (11:16 -0400)
commitcad66c013873909614ea8e2f7eb530259b58796b
tree7a7649cbfb633e5d9e39d7f58be5119993558863
parentede72f0a11c9f89eab79bb177a626f0c4a87d533
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