From: caillon%returnzero.com <>
Date: Sun, 5 May 2002 10:08:50 +0000 (+0000)
Subject: Rename getSelection to get_selection.  Fixes bug 141716, Mozilla context menus were... 
X-Git-Tag: bugzilla-2.16rc1~24
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=3b0b95d36d3e8b4ebea4d8e32d5337623cc13ba2;p=thirdparty%2Fbugzilla.git

Rename getSelection to get_selection.  Fixes bug 141716, Mozilla context menus were broken on query.cgi because of getSelection overriding the native method which is used by Mozilla's context menu code.  r=bbaetz x2
---

diff --git a/template/en/default/search/form.html.tmpl b/template/en/default/search/form.html.tmpl
index 9160397433..790feee640 100644
--- a/template/en/default/search/form.html.tmpl
+++ b/template/en/default/search/form.html.tmpl
@@ -207,7 +207,7 @@ function merge_arrays(a, b, b_is_select) {
   #      indexes
   #    - want_values: boolean; we store values when true and indexes when
   #      false %]
-function getSelection(control, findall, want_values) {
+function get_selection(control, findall, want_values) {
     var ret = new Array();
 
     if ((!findall) && (control.selectedIndex == -1)) {
@@ -281,7 +281,7 @@ function selectProduct(f) {
 
     [%# if nothing selected, pick all %]
     var findall = f.product.selectedIndex == -1;
-    sel = getSelection(f.product, findall, false);
+    sel = get_selection(f.product, findall, false);
     if (!findall) {
         [%# save sel for the next invocation of selectProduct() %]
         var tmp = sel;
@@ -297,10 +297,10 @@ function selectProduct(f) {
         last_sel = tmp;
     }
     [%# save original options selected %]
-    var saved_cpts = getSelection(f.component, false, true);
-    var saved_vers = getSelection(f.version, false, true);
+    var saved_cpts = get_selection(f.component, false, true);
+    var saved_vers = get_selection(f.version, false, true);
     [% IF Param('usetargetmilestone') %]
-    var saved_tms = getSelection(f.target_milestone, false, true);
+    var saved_tms = get_selection(f.target_milestone, false, true);
     [% END %]
 
     [%# do the actual fill/update, reselect originally selected options %]