952399f2eae3acbca7dc79b7b311a0fe1e936d03
tdreszer
  Wed Sep 1 12:37:49 2010 -0700
Made getAllVars understand subtrack vis. Redid cfg popup OK with subtrack vis understanding in mind
diff --git src/hg/js/ajax.js src/hg/js/ajax.js
index ff962e9..d9df3d9 100644
--- src/hg/js/ajax.js
+++ src/hg/js/ajax.js
@@ -97,37 +97,12 @@
     var values = [];
     if($(obj) == undefined)
         obj = $('document');
-    var inp = $(obj).find('input');
-    var sel = $(obj).find('select');
-    //warn("obj:"+$(obj).attr('id') + " inputs:"+$(inp).length+ " selects:"+$(sel).length);
-    $(inp).filter('[name]:enabled').each(function (i) {
-        var name  = $(this).attr('name');
-        var val = $(this).val();
-        if(name != undefined && name != "Submit" && val != undefined) {
-            names.push(name);
-            values.push(val);
-        }
-    });
-    $(sel).filter('[name]:enabled').each(function (i) {
-        var name  = $(this).attr('name');
-        var val = $(this).val();
-        if(name != undefined && val != undefined) {
-            if(subtrackName != undefined && name == subtrackName) {
-                names.push(name+"_sel");  // subtrack is controld by two vars
-                names.push(name);
-                if(val == 'hide') {
-                   values.push("0");    // Can't delete "_sel" because default takes over
-                    values.push("[]");  // can delete vis because subtrack vis should be inherited.
-                } else {
-                    values.push("1");
-                    values.push(val);
-                }
-            } else {
-                names.push(name);
-                values.push(val);
-            }
+
+    var o = getAllVars(obj);
+    for (var i in o) {
+        names.push(i);
+        values.push(o[i]);
         }
-    });
     if(names.length > 0) {
         //warn("variables:"+names+"  values:"+values);
         setCartVars(names,values);