fix: clean up code

acke · acke · commit 2d196bca3237 · 2025-01-20T14:14:50.000+01:00
diff --git a/plugin/src/main/java/io/snyk/eclipse/plugin/html/StaticPageHtmlProvider.java b/plugin/src/main/java/io/snyk/eclipse/plugin/html/StaticPageHtmlProvider.java
@@ -149,6 +149,7 @@ public String getInitHtml() {
 		return replaceCssVariables(html);
 	}
 
+	//TODO update this when we got new design from Andy
 	public String getSummaryInitHtml() {
 		var html = """
 				<!DOCTYPE html>
diff --git a/plugin/src/main/java/io/snyk/eclipse/plugin/views/snyktoolview/SummaryBrowserHandler.java b/plugin/src/main/java/io/snyk/eclipse/plugin/views/snyktoolview/SummaryBrowserHandler.java
@@ -24,9 +24,6 @@ public Object function(Object[] arguments) {
 				Preferences.getInstance().store(Preferences.ENABLE_DELTA, Boolean.FALSE.toString());
 				updateConfiguration();
 
-				// TODO remove this when we get the HTML from Snyk Language Server
-				browser.execute("document.body.innerHTML += '<p>All issues tab clicked</p>';");
-
 				return null;
 			}
 		};
@@ -37,9 +34,6 @@ public Object function(Object[] arguments) {
 				Preferences.getInstance().store(Preferences.ENABLE_DELTA, Boolean.TRUE.toString());
 				updateConfiguration();
 
-				// TODO remove this when we get the HTML from Snyk Language Server
-				browser.execute("document.body.innerHTML += '<p>Delta issues tab clicked</p>';");
-
 				return null;
 			}
 
@@ -53,14 +47,11 @@ private void updateConfiguration() {
 			// Update the Snyk Language Server configuration.
 			final var lc = SnykExtendedLanguageClient.getInstance();
 			lc.updateConfiguration();
-
-			// TODO do we want to start scan here?
-			lc.triggerScan(null);
 		});
 	}
 
 	public void setDefaultBrowserText() {
-		browser.setText(StaticPageHtmlProvider.getInstance().getSummaryInitHtml2());
+		browser.setText(StaticPageHtmlProvider.getInstance().getSummaryInitHtml());
 	}
 
 	public void setBrowserText(String summary) {

Original file line number	Diff line number	Diff line change
`@@ -149,6 +149,7 @@ public String getInitHtml() {`
`149`	`149`	`return replaceCssVariables(html);`
`150`	`150`	`}`
`151`	`151`
	`152`	`+ //TODO update this when we got new design from Andy`
`152`	`153`	`public String getSummaryInitHtml() {`
`153`	`154`	`var html = """`
`154`	`155`	`<!DOCTYPE html>`