summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authortara%tequilarista.org <>2001-01-18 11:14:11 +0100
committertara%tequilarista.org <>2001-01-18 11:14:11 +0100
commitb82afafeadec1e726af8b0007d3e82a33f81cbe0 (patch)
treeceb7d2c8fbb7ec73ee6c7436adabf2b27b504774
parent2093c1cbb3a2837a0d7136d973b0ee594b00d46f (diff)
downloadbugzilla-b82afafeadec1e726af8b0007d3e82a33f81cbe0.tar.gz
bugzilla-b82afafeadec1e726af8b0007d3e82a33f81cbe0.tar.xz
Landing patch for bug #58436 from Henrik Gemal
-rwxr-xr-xquery.cgi24
1 files changed, 12 insertions, 12 deletions
diff --git a/query.cgi b/query.cgi
index ea96e37ee..65a4cb29a 100755
--- a/query.cgi
+++ b/query.cgi
@@ -428,7 +428,7 @@ function selectProduct(f) {
if (doit) {
var l = f.component.length;
f.component[l] = new Option(c, c);
- if (csel[c]) {
+ if (csel.length && csel[c]) {
f.component[l].selected = true;
}
}
@@ -445,13 +445,13 @@ function selectProduct(f) {
for (v in vers) {
if (typeof(vers[v]) == 'function') continue;
- var doit = doall;
+ doit = doall;
for (i=0 ; !doit && i<f.product.length ; i++) {
if (f.product[i].selected) {
- var p = f.product[i].value;
+ p = f.product[i].value;
for (j in vers[v]) {
if (typeof(vers[v][j]) == 'function') continue;
- var p2 = vers[v][j];
+ p2 = vers[v][j];
if (p2 == p) {
doit = true;
break;
@@ -460,15 +460,15 @@ function selectProduct(f) {
}
}
if (doit) {
- var l = f.version.length;
+ l = f.version.length;
f.version[l] = new Option(v, v);
- if (vsel[v]) {
+ if (vsel.length && vsel[v]) {
f.version[l].selected = true;
}
}
}
- if (f.target_milestone) {
+ if ("target_milestone" in f) {
var tmsel = new Array();
for (i=0 ; i<f.target_milestone.length ; i++) {
if (f.target_milestone[i].selected) {
@@ -480,13 +480,13 @@ function selectProduct(f) {
for (tm in tms) {
if (typeof(tms[v]) == 'function') continue;
- var doit = doall;
+ doit = doall;
for (i=0 ; !doit && i<f.product.length ; i++) {
if (f.product[i].selected) {
- var p = f.product[i].value;
+ p = f.product[i].value;
for (j in tms[tm]) {
if (typeof(tms[tm][j]) == 'function') continue;
- var p2 = tms[tm][j];
+ p2 = tms[tm][j];
if (p2 == p) {
doit = true;
break;
@@ -495,9 +495,9 @@ function selectProduct(f) {
}
}
if (doit) {
- var l = f.target_milestone.length;
+ l = f.target_milestone.length;
f.target_milestone[l] = new Option(tm, tm);
- if (tmsel[tm]) {
+ if (tmsel.length && tmsel[tm]) {
f.target_milestone[l].selected = true;
}
}