Merge pull request #8716 from sdedic/gradle/npe-error-report

sdedic · web-flow · commit 078fb05fe16f · 2025-08-07T10:15:27.000+02:00
Prevents NPE if buildscript does not exist
diff --git a/extide/gradle/src/org/netbeans/modules/gradle/loaders/AbstractProjectLoader.java b/extide/gradle/src/org/netbeans/modules/gradle/loaders/AbstractProjectLoader.java
@@ -18,6 +18,7 @@
  */
 package org.netbeans.modules.gradle.loaders;
 
+import java.io.File;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.HashMap;
@@ -124,7 +125,11 @@ static GradleProject createGradleProject(GradleFiles gf, ProjectInfoDiskCache.Qu
             }
         }
         for (String s : problems) {
-            reps.add(GradleProject.createGradleReport(gf.getBuildScript().toPath(), s));
+            File p = gf.getBuildScript();
+            if (p == null) {
+                p = gf.getSettingsScript();
+            }
+            reps.add(GradleProject.createGradleReport(p == null ? null : p.toPath(), s));
         }
         return new GradleProject(info.getQuality(), reps, results.values());
 

Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,7 @@`
`18`	`18`	`*/`
`19`	`19`	`package org.netbeans.modules.gradle.loaders;`
`20`	`20`
	`21`	`+import java.io.File;`
`21`	`22`	`import java.util.Collection;`
`22`	`23`	`import java.util.Collections;`
`23`	`24`	`import java.util.HashMap;`
`@@ -124,7 +125,11 @@ static GradleProject createGradleProject(GradleFiles gf, ProjectInfoDiskCache.Qu`
`124`	`125`	`}`
`125`	`126`	`}`
`126`	`127`	`for (String s : problems) {`
`127`		`- reps.add(GradleProject.createGradleReport(gf.getBuildScript().toPath(), s));`
	`128`	`+ File p = gf.getBuildScript();`
	`129`	`+ if (p == null) {`
	`130`	`+ p = gf.getSettingsScript();`
	`131`	`+ }`
	`132`	`+ reps.add(GradleProject.createGradleReport(p == null ? null : p.toPath(), s));`
`128`	`133`	`}`
`129`	`134`	`return new GradleProject(info.getQuality(), reps, results.values());`
`130`	`135`