5cfb74ffcc83a58a509424c14f2aeb68761d1957 larrym Fri Sep 9 10:56:12 2011 -0700 fix code that was broken when I moved stuff from hiddens to hgTracks object (see redmine #5268) diff --git src/hg/js/hgTracks.js src/hg/js/hgTracks.js index 586103a..eab4e67 100644 --- src/hg/js/hgTracks.js +++ src/hg/js/hgTracks.js @@ -1942,35 +1942,35 @@ jQuery('body').css('cursor', 'wait'); $.ajax({ type: "GET", url: "../cgi-bin/hgTracks", data: data, dataType: "html", trueSuccess: handleViewImg, success: catchErrorOrDispatch, error: errorHandler, cmd: cmd, cache: false }); } else if (cmd == 'openLink' || cmd == 'followLink') { var href = selectedMenuItem.href; var vars = new Array("c", "l", "r"); - var hiddens = new Array("chromName", "l", "r"); + var valNames = new Array("chromName", "winStart", "winEnd"); for (i in vars) { // make sure the link contains chrom and window width info (necessary b/c we are stripping hgsid and/or the cart may be empty); // but don't add chrom to wikiTrack links (see redmine #2476). - var val = $("input[name=" + hiddens[i] + "]").val(); + var val = hgTracks[valNames[i]]; var v = vars[i]; if(val && id != "wikiTrack" && (href.indexOf("?" + v + "=") == -1) && (href.indexOf("&" + v + "=") == -1)) { href = href + "&" + v + "=" + val; } } if(cmd == 'followLink') { // XXXX This is blocked by Safari's popup blocker (without any warning message). location.assign(href); } else { // Remove hgsid to force a new session (see redmine ticket 1333). href = removeHgsid(href); if(window.open(href) == null) { windowOpenFailedMsg(); } }