diff options
author | cyeh%bluemartini.com <> | 2000-08-22 05:16:00 +0200 |
---|---|---|
committer | cyeh%bluemartini.com <> | 2000-08-22 05:16:00 +0200 |
commit | ff85006b3da36c24439c87f016aafe3317347ef0 (patch) | |
tree | 704cafb8dbc2611be6d900f5edeaf3db7d5dbf04 /query.cgi | |
parent | e8b93c7c68c4b56cdc10249ba3740a504f30e2ee (diff) | |
download | bugzilla-ff85006b3da36c24439c87f016aafe3317347ef0.tar.gz bugzilla-ff85006b3da36c24439c87f016aafe3317347ef0.tar.xz |
fix for #39044 : Some vers. of some browsers give a script warning on query page when not
using milestones. patch submitted by nkiesel@tbdnetworks.com
Diffstat (limited to 'query.cgi')
-rwxr-xr-x | query.cgi | 55 |
1 files changed, 28 insertions, 27 deletions
@@ -431,40 +431,41 @@ function selectProduct(f) { } } - var tmsel = new Array(); - for (i=0 ; i<f.target_milestone.length ; i++) { - if (f.target_milestone[i].selected) { - tmsel[f.target_milestone[i].value] = 1; + if (f.target_milestone) { + var tmsel = new Array(); + for (i=0 ; i<f.target_milestone.length ; i++) { + if (f.target_milestone[i].selected) { + tmsel[f.target_milestone[i].value] = 1; + } } - } - - f.target_milestone.options.length = 0; - - for (tm in tms) { - if (typeof(tms[v]) == 'function') continue; - var doit = doall; - for (i=0 ; !doit && i<f.product.length ; i++) { - if (f.product[i].selected) { - var p = f.product[i].value; - for (j in tms[tm]) { - if (typeof(tms[tm][j]) == 'function') continue; - var p2 = tms[tm][j]; - if (p2 == p) { - doit = true; - break; + + f.target_milestone.options.length = 0; + + for (tm in tms) { + if (typeof(tms[v]) == 'function') continue; + var doit = doall; + for (i=0 ; !doit && i<f.product.length ; i++) { + if (f.product[i].selected) { + var p = f.product[i].value; + for (j in tms[tm]) { + if (typeof(tms[tm][j]) == 'function') continue; + var p2 = tms[tm][j]; + if (p2 == p) { + doit = true; + break; + } } } } - } - if (doit) { - var l = f.target_milestone.length; - f.target_milestone[l] = new Option(tm, tm); - if (tmsel[tm]) { - f.target_milestone[l].selected = true; + if (doit) { + var l = f.target_milestone.length; + f.target_milestone[l] = new Option(tm, tm); + if (tmsel[tm]) { + f.target_milestone[l].selected = true; + } } } } - } // --> </script> |