Merge pull request #227 from jborgers/master

Fixes issue #226: prevent NullPointerException; Configuration descrip…

Author: amitdev

gradle.properties

@@ -20,7 +20,7 @@ pluginUntilBuild =
 pluginVerifierIdeVersions = IC-2023.3, IC-2024.1, IC-2024.2, IC-2024.3, IC-2025.1
 
 platformType = IC
-platformVersion = 2023.3
+platformVersion = 2024.1
 #platformVersion = 251-EAP-SNAPSHOT
 org.jetbrains.intellij.platform.downloadSources=true
 

src/main/java/com/intellij/plugins/bodhi/pmd/ConfigOption.java

@@ -8,7 +8,7 @@
  * Configuration options enumeration. Separation between key for persistent state and description to show in the UI.
  */
 public enum ConfigOption {
-    TARGET_JDK("Target JDK", "Target JDK (max: " + latestSupportLanguageVersionByPmd("java") + ")", latestSupportLanguageVersionByPmd("java")),
+    TARGET_JDK("Target JDK", "Target Java version (max: " + latestSupportLanguageVersionByPmd("java") + ")", latestSupportLanguageVersionByPmd("java")),
     TARGET_KOTLIN_VERSION("Target Kotlin version", "Target Kotlin version (max: " + latestSupportLanguageVersionByPmd("kotlin") + ")", latestSupportLanguageVersionByPmd("kotlin")),
     STATISTICS_URL("Statistics URL", "Statistics URL to export usage anonymously", ""),
     THREADS("Threads", "Threads (fastest: " + PMDUtil.AVAILABLE_PROCESSORS + ")", String.valueOf(PMDUtil.AVAILABLE_PROCESSORS));

src/main/java/com/intellij/plugins/bodhi/pmd/PMDProjectComponent.java

@@ -303,7 +303,12 @@ public void loadState(PersistentData state) {
                 optionToValue.put(ConfigOption.STATISTICS_URL, "");
             }
             else {
-                optionToValue.put(ConfigOption.fromKey(key), state.getOptionKeyToValue().get(key));
+                // make sure the value is not null, if so, make it the default value, see #226
+                String optionValue = state.getOptionKeyToValue().get(key);
+                if (optionValue == null) {
+                    optionValue = ConfigOption.fromKey(key).getDefaultValue();
+                }
+                optionToValue.put(ConfigOption.fromKey(key), optionValue);
             }
         }