Fix Skyline external tool download URL on the tool detail page. (#342)

vagisha · web-flow · commit 5d4f1734b79f · 2023-04-06T16:13:12.000-07:00
diff --git a/SkylineToolsStore/src/org/labkey/skylinetoolsstore/view/SkylineToolDetails.jsp b/SkylineToolsStore/src/org/labkey/skylinetoolsstore/view/SkylineToolDetails.jsp
@@ -684,7 +684,7 @@ a { text-decoration: none; }
             });
         }
 
-        window.location.href = "<%=h(urlFor(SkylineToolsStoreController.DownloadToolAction.class))%>id=" + toolId;
+        window.location.href = <%= q(urlFor(SkylineToolsStoreController.DownloadToolAction.class).addParameter("id", tool.getRowId())) %>;
     }
 
     function popToolOwners() {

Original file line number	Diff line number	Diff line change
`@@ -684,7 +684,7 @@ a { text-decoration: none; }`
`684`	`684`	`});`
`685`	`685`	`}`
`686`	`686`
`687`		`- window.location.href = "<%=h(urlFor(SkylineToolsStoreController.DownloadToolAction.class))%>id=" + toolId;`
	`687`	`+ window.location.href = <%= q(urlFor(SkylineToolsStoreController.DownloadToolAction.class).addParameter("id", tool.getRowId())) %>;`
`688`	`688`	`}`
`689`	`689`
`690`	`690`	`function popToolOwners() {`