Added chunk update messages when player config is empty.

Author: toddharrison

fake-block-api/src/main/java/com/briarcraft/fakeblock/api/data/Group.java

@@ -42,10 +42,16 @@ public class Group implements Comparable<Group>, ConfigurationSerializable {
     @SuppressWarnings("unused")
     public static @Nonnull Group deserialize(final @Nonnull Map<String, Object> args) {
         val name = (String) args.get(ARG_NAME);
-        val world = Bukkit.getWorld((String) args.get(ARG_WORLD));
-        val fakeBlocks = ((List<FakeBlock>) args.get(ARG_BLOCKS)).stream().collect(Collectors.toSet());
+        val worldName = (String) args.get(ARG_WORLD);
+        val world = Bukkit.getWorld(worldName);
+        val fakeBlocks = ((List<?>) args.get(ARG_BLOCKS)).stream()
+                .filter(FakeBlock.class::isInstance)
+                .map(FakeBlock.class::cast)
+                .collect(Collectors.toSet());
         val isShowDefault = (Boolean) args.get(ARG_SHOW_DEFAULT);
 
+        if (world == null) throw new IllegalStateException("World '" + worldName + "' was not found!");
+
         val group = new Group(name, world, fakeBlocks);
         group.setShownByDefault(isShowDefault);
         return group;

fake-block/build.gradle.kts

@@ -6,7 +6,7 @@ plugins {
     id("com.github.johnrengelman.shadow")
 }
 
-version = "2.0.0"
+version = "2.0.1"
 description = ""
 
 dependencies {

fake-block/src/main/java/com/briarcraft/fakeblock/FakeBlockCommand.java

@@ -13,7 +13,6 @@
 import net.kyori.adventure.text.format.NamedTextColor;
 import net.kyori.adventure.text.format.Style;
 import org.bukkit.Bukkit;
-import org.bukkit.Material;
 import org.bukkit.OfflinePlayer;
 import org.bukkit.command.*;
 import org.bukkit.entity.Player;

fake-block/src/main/java/com/briarcraft/fakeblock/data/PlayerGroup.java

@@ -7,6 +7,7 @@
 
 import javax.annotation.Nonnull;
 import java.util.*;
+import java.util.stream.Collectors;
 
 @SerializableAs("PlayerGroup")
 @Data
@@ -29,7 +30,10 @@ public class PlayerGroup implements Comparable<PlayerGroup>, ConfigurationSerial
     @SuppressWarnings("unused")
     public static @Nonnull PlayerGroup deserialize(final @Nonnull Map<String, Object> args) {
         val playerId = UUID.fromString((String) args.get(ARG_PLAYER_ID));
-        val groupNames = (Map<String, Boolean>) args.get(ARG_GROUPS);
+        val groupNames = ((Map<?, ?>) args.get(ARG_GROUPS)).entrySet().stream()
+                .filter(entry -> entry.getKey() instanceof String)
+                .filter(entry -> entry.getValue() instanceof Boolean)
+                .collect(Collectors.toMap(entry -> (String) entry.getKey(), entry -> (Boolean) entry.getValue()));
         return new PlayerGroup(playerId, groupNames);
     }
 

fake-block/src/main/java/com/briarcraft/fakeblock/service/PlayerGroupServiceImpl.java

@@ -87,6 +87,11 @@ public boolean showGroup(final @Nonnull String groupName, final @Nonnull Offline
                 newGroups.put(groupName, true);
                 playerGroups.put(playerId, newGroups);
                 playerGroupConfig.save(generatePlayerGroups());
+
+                if (offlinePlayer instanceof Player player) {
+                    groupService.getChunklets(groupName, player.getWorld())
+                            .forEach(chunklet -> protocolLibService.sendChunklet(player, chunklet));
+                }
                 return true;
             } else {
                 var isShown = false;
@@ -127,6 +132,11 @@ public boolean hideGroup(final @Nonnull String groupName, final @Nonnull Offline
                 newGroups.put(groupName, false);
                 playerGroups.put(playerId, newGroups);
                 playerGroupConfig.save(generatePlayerGroups());
+
+                if (offlinePlayer instanceof Player player) {
+                    groupService.getChunklets(groupName, player.getWorld())
+                            .forEach(chunklet -> protocolLibService.sendChunklet(player, chunklet, Material.AIR));
+                }
                 return true;
             } else {
                 var isHidden = false;