Merge pull request #108 from snyk/fix/road-1244-broken-wizard

fix: null exception in wizard [ROAD-1244]
snyk · Dec 9, 2022 · 8c377f6 · 8c377f6
2 parents b73c496 + ae751a3
commit 8c377f6
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/plugin/src/main/java/io/snyk/eclipse/plugin/properties/preferences/Preferences.java b/plugin/src/main/java/io/snyk/eclipse/plugin/properties/preferences/Preferences.java
@@ -144,6 +144,10 @@ public String getCliPath() {
   public boolean isInsecure() {
     return store.getBoolean(INSECURE_KEY, false);
   }
+
+  public void setIsInsecure(boolean isInsecure) {
+	  store.put(INSECURE_KEY, Boolean.toString(isInsecure));
+  }
 
 
   public boolean isManagedBinaries() {

diff --git a/plugin/src/main/java/io/snyk/eclipse/plugin/wizards/SnykWizardModel.java b/plugin/src/main/java/io/snyk/eclipse/plugin/wizards/SnykWizardModel.java
@@ -5,18 +5,18 @@
 
 public class SnykWizardModel {
   private String initialEndpoint;
-  private String initialUnknownCerts;
+  private boolean initialUnknownCerts;
   private String initialAuthToken;
 
   public SnykWizardModel() {
-    this.initialEndpoint = Preferences.getInstance().getPref(Preferences.ENDPOINT_KEY);
-    this.initialUnknownCerts = Preferences.getInstance().getPref(Preferences.INSECURE_KEY);
-    this.initialAuthToken = Preferences.getInstance().getPref(Preferences.AUTH_TOKEN_KEY);
+    this.initialEndpoint = Preferences.getInstance().getPref(Preferences.ENDPOINT_KEY, "");
+    this.initialUnknownCerts = Preferences.getInstance().isInsecure();
+    this.initialAuthToken = Preferences.getInstance().getAuthToken();
   }
 
   public void resetPreferences() {
     Preferences.getInstance().store(Preferences.ENDPOINT_KEY, initialEndpoint);
-    Preferences.getInstance().store(Preferences.INSECURE_KEY, initialUnknownCerts);
+    Preferences.getInstance().setIsInsecure(initialUnknownCerts);
     Preferences.getInstance().store(Preferences.AUTH_TOKEN_KEY, initialAuthToken);
 
     new LsConfigurationUpdater().configurationChanged();