]> git.evergreen-ils.org Git - Evergreen.git/commit
Revert "lp1895699: search results page lets user know that course materials filter...
authorJane Sandberg <js7389@princeton.edu>
Thu, 16 Nov 2023 05:08:12 +0000 (21:08 -0800)
committerGalen Charlton <gmc@equinoxOLI.org>
Fri, 17 Nov 2023 19:36:41 +0000 (14:36 -0500)
commitead4f82da75728b459d12f595bd1bab5d2da1990
treee7b70ff657897c24d38bc7fb9134b6b7ea8f012f
parent00003d7aa91bdf39272891da9f7c8e80aab24fb1
Revert "lp1895699: search results page lets user know that course materials filter has been applied"

This reverts commit 828a6f871fbe04d85c4c7132c37340b7f1f19811.

This led to a failure in live_t/35-multiclass-search.t, and sometimes led to cases where searching
with the course materials filter would time out or return an error.

The issue lies with the dynamic_filter_compile subroutine in QueryParser.pm.  It makes an
assumption that any filter with an entry in config.record_attr_definition will also have
entries in config.coded_value_map.  This db migration had no such coded_value_map entries,
so it added a function call to the query_int_wrapper function, which works in some environments
and doesn't in others.  We don't want that!

Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Open-ILS/src/sql/Pg/950.data.seed-values.sql
Open-ILS/src/sql/Pg/upgrade/1393.data.limit-search-to-reserves.sql
Open-ILS/src/templates-bootstrap/opac/parts/result/adv_filter.tt2