LP#1519055: Some JS fixes

Adding basic javascript to remove our search filter from the
textbox on an advanced search form.  Because of the interplay among
various checkboxes/modifiers/filters, there may be duplicate filters
applied from another function's form submit; this removes all at once.

Signed-off-by: Jake Litrell <jake@masslnc.org>
Signed-off-by: Ben Shum <ben@evergreener.net>

diff --git a/Open-ILS/src/templates/opac/results.tt2 b/Open-ILS/src/templates/opac/results.tt2
index 15bce7d..284fbb9 100644 (file)
--- a/Open-ILS/src/templates/opac/results.tt2
+++ b/Open-ILS/src/templates/opac/results.tt2
@@ -104,15 +104,12 @@
                     -%]
                         <input type="hidden" name="detail_record_view" value="1" />
                     [% END %]
-
                     <label class="results_header_lbl" for='opac.result.excludeelec'>
-                        <input type='checkbox' name="fi:-search_format" value="electronic"
-                            onchange="search_modifier_onchange('available', this, true)"
-                            [%- CGI.param('fi:-search_format').grep('electronic').size ? ' checked="checked"' : '' %]
-                            id='opac.result.excludeelec' />
+                        <input type='checkbox' id='opac.result.excludeelec' name="fi:-search_format" value="electronic"
+                            onchange="exclude_onchange(this)"
+                            [%- CGI.param('fi:-search_format').grep('electronic').size ? ' checked="checked"' : '' %] />
                         [% l("Exclude Electronic Resources") %]
                     </label>
-
             </div>
         </div>
     </div>

diff --git a/Open-ILS/web/js/ui/default/opac/simple.js b/Open-ILS/web/js/ui/default/opac/simple.js
index 7fa20cb..2f57bf6 100644 (file)
--- a/Open-ILS/web/js/ui/default/opac/simple.js
+++ b/Open-ILS/web/js/ui/default/opac/simple.js
@@ -100,3 +100,14 @@ function search_modifier_onchange(type, checkbox, submitOnChange) {
         checkbox.form.submit(); 
     }
 }
+
+function exclude_onchange(checkbox) {
+    if (checkbox.form._adv && !checkbox.checked) {
+        var search_box = $('search_box');
+        // Other functions' form submits may create duplicates of this, so /g
+        var reg = /-search_format\(electronic\)/g;
+        search_box.value = search_box.value.replace(reg, "");
+    }
+
+    checkbox.form.submit();
+}