LP#1736419: Located URIs vs QueryParser, round 2
authorMike Rylander <mrylander@gmail.com>
Thu, 4 Jan 2018 23:12:17 +0000 (18:12 -0500)
committerMike Rylander <mrylander@gmail.com>
Fri, 5 Jan 2018 19:17:09 +0000 (14:17 -0500)
commitc4be66fdd462322197bb5e029a2ba96b47887e71
tree0fc5d33b7db402f96ad72830ae6e3ad31b121aa8
parent5cbd3ed1a981d7157c21997a88cd847389e928ae
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>
Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Driver/Pg/QueryParser.pm