Improve handling of invalid data in json files on bungee

Author: BenCodez

SimpleAPI/.classpath

@@ -1,11 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <classpath>
-	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17">
-		<attributes>
-			<attribute name="module" value="true"/>
-			<attribute name="maven.pomderived" value="true"/>
-		</attributes>
-	</classpathentry>
 	<classpathentry kind="src" output="target/classes" path="src/main/java">
 		<attributes>
 			<attribute name="optional" value="true"/>
@@ -31,5 +25,16 @@
 			<attribute name="maven.pomderived" value="true"/>
 		</attributes>
 	</classpathentry>
+	<classpathentry excluding="**" kind="src" output="target/classes" path="src/main/resources">
+		<attributes>
+			<attribute name="maven.pomderived" value="true"/>
+			<attribute name="optional" value="true"/>
+		</attributes>
+	</classpathentry>
+	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.8">
+		<attributes>
+			<attribute name="maven.pomderived" value="true"/>
+		</attributes>
+	</classpathentry>
 	<classpathentry kind="output" path="target/classes"/>
 </classpath>

SimpleAPI/.settings/org.eclipse.core.resources.prefs

@@ -1,4 +1,7 @@
 eclipse.preferences.version=1
 encoding//src/main/java=Cp1252
+encoding//src/main/resources=Cp1252
+encoding//src/test/java=Cp1252
+encoding//src/test/resources=Cp1252
 encoding/<project>=Cp1252
 encoding/src=Cp1252

SimpleAPI/.settings/org.eclipse.jdt.core.prefs

@@ -1,8 +1,8 @@
 eclipse.preferences.version=1
 org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=enabled
-org.eclipse.jdt.core.compiler.codegen.targetPlatform=17
+org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.8
 org.eclipse.jdt.core.compiler.codegen.unusedLocal=preserve
-org.eclipse.jdt.core.compiler.compliance=17
+org.eclipse.jdt.core.compiler.compliance=1.8
 org.eclipse.jdt.core.compiler.debug.lineNumber=generate
 org.eclipse.jdt.core.compiler.debug.localVariable=generate
 org.eclipse.jdt.core.compiler.debug.sourceFile=generate
@@ -11,5 +11,5 @@ org.eclipse.jdt.core.compiler.problem.enablePreviewFeatures=disabled
 org.eclipse.jdt.core.compiler.problem.enumIdentifier=error
 org.eclipse.jdt.core.compiler.problem.forbiddenReference=warning
 org.eclipse.jdt.core.compiler.problem.reportPreviewFeatures=ignore
-org.eclipse.jdt.core.compiler.release=disabled
-org.eclipse.jdt.core.compiler.source=17
+org.eclipse.jdt.core.compiler.release=enabled
+org.eclipse.jdt.core.compiler.source=1.8

SimpleAPI/src/main/java/com/bencodez/simpleapi/file/BungeeJsonFile.java

@@ -143,22 +143,20 @@ public List<String> getStringList(String path, List<String> def) {
 	public List<String> getKeys(String path) {
 		JsonObject parentNode = navigateToNode(path);
 		String lastPart = getLastPathPart(path);
-		JsonObject node = parentNode.getAsJsonObject(lastPart);
+		JsonObject node = null;
 		if (path.split("\\.").length == 1) {
-           node = parentNode;
+			node = parentNode;
+		} else if (parentNode != null) {
+			node = parentNode.getAsJsonObject(lastPart);
 		}
-		
+
 		if (node != null) {
-			//System.out.println(node.toString());
 			List<String> keys = new ArrayList<>();
 			for (Map.Entry<String, JsonElement> entry : node.entrySet()) {
 				keys.add(entry.getKey());
 			}
 			return keys;
 		}
-
-		// System.out.println("getKeys: No valid keys found for path = " + path + ".
-		// Make sure the path is correct.");
 		return new ArrayList<>();
 	}
 

SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/BungeeJsonFileTest.java

@@ -109,6 +109,13 @@ public void testGetKeysBase() {
 		assertTrue(keys.contains("two"));
 	}
 
+	@Test
+	public void testGetKeysInvalidPath() {
+		List<String> keys = bungeeJsonFile.getKeys("invalid.path");
+		assertNotNull(keys);
+		assertTrue(keys.isEmpty());
+	}
+
 	@Test
 	public void testGetNode() {
 		bungeeJsonFile.setString("test.node.key", "value");