SAK-32736 - Launch button is rendered when it's not required (sakaipr…

…oject#4653) * SAK-32736 - Launch button is rendered when it's not required * SAK-32736 - Fix per Stephen
UHaooooo · Aug 1, 2017 · cf25727 · cf25727
1 parent 0b4cf31
commit cf25727
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/basiclti/tsugi-util/src/java/org/tsugi/basiclti/BasicLTIUtil.java b/basiclti/tsugi-util/src/java/org/tsugi/basiclti/BasicLTIUtil.java
@@ -553,14 +553,18 @@ public static String postLaunchHTML(
 		}
 
 		// Paint the submit button
-		text.append("<input type=\"submit\" value=\"");
-		text.append(htmlspecialchars(launchtext));
-		text.append("\">\n");
-
 		if ( debug ) {
+			text.append("<input type=\"submit\" value=\"");
+			text.append(htmlspecialchars(launchtext));
+			text.append("\">\n");
+
 			text.append(" <input type=\"Submit\" value=\"Show Launch Data\" onclick=\"document.getElementById('ltiLaunchDebug_");
 			text.append(submit_uuid);
 			text.append("').style.display = 'block';return false;\">\n");
+		} else {
+			text.append("<input type=\"submit\" style=\"display: none\" value=\"");
+			text.append(htmlspecialchars(launchtext));
+			text.append("\">\n");
 		}
 
 		if ( extra != null ) {