LP#1677902: Remove search formats from advanced search limiter block
authorKathy Lussier <klussier@masslnc.org>
Wed, 17 May 2017 16:11:33 +0000 (12:11 -0400)
committerKathy Lussier <klussier@masslnc.org>
Tue, 23 May 2017 17:51:26 +0000 (13:51 -0400)
The original behavior of the advanced search limiter block was to remove
search formats from the display because it is available for limiting in the
search bar. Other code broke this behavior, and I've now restored it by
adding the ignore search format code to the searchbar.tt2 file. It also fixes
a problem where the search format limiters were incorrectly displaying on the
low hits page.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Open-ILS/src/templates/opac/parts/result/table.tt2
Open-ILS/src/templates/opac/parts/searchbar.tt2

index a503484..e2fa5ba 100644 (file)
@@ -8,21 +8,6 @@
 
     result_count = ctx.result_start;
 
-    # don't display a box for the search_format filter,
-    # as that's got its own widget
-    ignore_filters = ['search_format'];
-
-    trimmed_filters = [];
-    FOR filter IN ctx.query_struct.filters;
-        fname = filter.name;
-        IF ignore_filters.grep('^' _ fname _ '$').size;
-            NEXT;
-        END;
-        trimmed_filters.push(filter);
-    END;
-
-    ctx.query_struct.filters = trimmed_filters;
-
 %]
 
 [% PROCESS "opac/parts/result/paginate.tt2" %] 
index 5e25605..ddf68cf 100644 (file)
@@ -12,7 +12,23 @@ FOR f IN ctx.query_struct.filters;
     fcount = fcount + 1;
 END;
 
+    # don't display a box for the search_format filter,
+    # as that's got its own widget
+    ignore_filters = ['search_format'];
+
+    trimmed_filters = [];
+    FOR filter IN ctx.query_struct.filters;
+        fname = filter.name;
+        IF ignore_filters.grep('^' _ fname _ '$').size;
+            NEXT;
+        END;
+        trimmed_filters.push(filter);
+    END;
+
+    ctx.query_struct.filters = trimmed_filters;
+
  %]
+
 <div id="search-wrapper">
     [% UNLESS took_care_of_form -%]
     <form action="[% ctx.opac_root %]/results" method="get">