Merge pull request #104 from TheNextLvl-net/fix-npe

NonSwag · web-flow · commit 630302996972 · 2025-06-04T20:13:54.000+02:00
Fix NullPointerException in version checker
diff --git a/velocity/src/main/java/net/thenextlvl/commander/velocity/version/CommanderVersionChecker.java b/velocity/src/main/java/net/thenextlvl/commander/velocity/version/CommanderVersionChecker.java
@@ -34,8 +34,8 @@ public SemanticVersion parseVersion(String version) {
 
     @Override
     public boolean isSupported(HangarVersion version) {
-        return version.platformDependencies().get(Platform.VELOCITY).stream()
-                .anyMatch(plugin.server().getVersion().getVersion()::startsWith);
+        var versions = version.platformDependencies().get(Platform.VELOCITY);
+        return versions != null && versions.stream().anyMatch(plugin.server().getVersion().getVersion()::startsWith);
     }
 
     public void checkVersion() {

Original file line number	Diff line number	Diff line change
`@@ -34,8 +34,8 @@ public SemanticVersion parseVersion(String version) {`
`34`	`34`
`35`	`35`	`@Override`
`36`	`36`	`public boolean isSupported(HangarVersion version) {`
`37`		`- return version.platformDependencies().get(Platform.VELOCITY).stream()`
`38`		`- .anyMatch(plugin.server().getVersion().getVersion()::startsWith);`
	`37`	`+ var versions = version.platformDependencies().get(Platform.VELOCITY);`
	`38`	`+ return versions != null && versions.stream().anyMatch(plugin.server().getVersion().getVersion()::startsWith);`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`public void checkVersion() {`