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

fix for #39044 : Some vers. of some browsers give a script warning on query page when not
ff85006b · cyeh%bluemartini.com · e8b93c7c · ff85006b
Commit ff85006b authored Aug 22, 2000 by cyeh%bluemartini.com
Hide whitespace changes
Inline Side-by-side

Showing with 28 additions and 27 deletions

query.cgi query.cgi +28 -27

No files found.
--- a/query.cgi
+++ b/query.cgi
@@ -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>