Fix npe when caching player

sandtechnology · sandtechnology · commit 522697f8ae5d · 2022-12-27T16:10:32.000+08:00
diff --git a/src/main/java/org/maxgamer/quickshop/util/PlayerFinder.java b/src/main/java/org/maxgamer/quickshop/util/PlayerFinder.java
@@ -84,7 +84,11 @@ public static void doLargeOfflineCachingWork(QuickShop quickShop, OfflinePlayer[
         quickShop.getLogger().log(Level.INFO, "Large server detected (offline player > 2000), start offline player caching...");
         isStashNeeded = true;
         for (OfflinePlayer offlinePlayer : quickShop.getServer().getOfflinePlayers()) {
-            string2UUIDStash.put(offlinePlayer.getName(), offlinePlayer.getUniqueId());
+            String name = offlinePlayer.getName();
+            UUID uuid = offlinePlayer.getUniqueId();
+            if (name != null && uuid != null) {
+                string2UUIDStash.put(offlinePlayer.getName(), offlinePlayer.getUniqueId());
+            }
         }
         quickShop.getLogger().log(Level.INFO, "Done! cached " + offlinePlayers.length + " players.");
     }

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,11 @@ public static void doLargeOfflineCachingWork(QuickShop quickShop, OfflinePlayer[`
`84`	`84`	`quickShop.getLogger().log(Level.INFO, "Large server detected (offline player > 2000), start offline player caching...");`
`85`	`85`	`isStashNeeded = true;`
`86`	`86`	`for (OfflinePlayer offlinePlayer : quickShop.getServer().getOfflinePlayers()) {`
`87`		`- string2UUIDStash.put(offlinePlayer.getName(), offlinePlayer.getUniqueId());`
	`87`	`+ String name = offlinePlayer.getName();`
	`88`	`+ UUID uuid = offlinePlayer.getUniqueId();`
	`89`	`+ if (name != null && uuid != null) {`
	`90`	`+ string2UUIDStash.put(offlinePlayer.getName(), offlinePlayer.getUniqueId());`
	`91`	`+ }`
`88`	`92`	`}`
`89`	`93`	`quickShop.getLogger().log(Level.INFO, "Done! cached " + offlinePlayers.length + " players.");`
`90`	`94`	`}`