]> git.evergreen-ils.org Git - Evergreen.git/commit
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>
Wed, 10 Jan 2018 00:01:32 +0000 (19:01 -0500)
commita9f5b6d81320d987a152546440a9d95955daef6f
tree7ce86db9254710246a6c9be33b105cc01f4ef5e7
parenteca9b68517617a8d8b05afbcd3a49ae0180057ba
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