Return immutable map in DiscordLinkService getAllLinkedPlayers

EssentialsX · Aug 9, 2024 · a4a4464 · a4a4464
1 parent 51dc5a4
commit a4a4464
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/...ordLink/src/main/java/net/essentialsx/api/v2/services/discordlink/DiscordLinkService.java b/...ordLink/src/main/java/net/essentialsx/api/v2/services/discordlink/DiscordLinkService.java
@@ -86,7 +86,9 @@ default boolean isLinked(final String discordId) {
 
     /**
      * Gets a map of all linked players, where the key is the Discord ID and the value is the Minecraft UUID.
-     * @return a map of all linked players.
+     * The returned map is immutable and cannot be modified.
+     *
+     * @return an immutable map of all linked players.
      */
     Map<String, String> getAllLinkedPlayers();
 }
diff --git a/EssentialsDiscordLink/src/main/java/net/essentialsx/discordlink/AccountLinkManager.java b/EssentialsDiscordLink/src/main/java/net/essentialsx/discordlink/AccountLinkManager.java
@@ -8,6 +8,7 @@
 import net.essentialsx.api.v2.services.discordlink.DiscordLinkService;
 import net.essentialsx.discordlink.rolesync.RoleSyncManager;
 
+import java.util.Collections;
 import java.util.Map;
 import java.util.Optional;
 import java.util.Random;
@@ -165,6 +166,6 @@ private String generateCode() {
 
     @Override
     public Map<String, String> getAllLinkedPlayers() {
-        return storage.getRawStorageMap();
+        return Collections.unmodifiableMap(storage.getRawStorageMap());
     }
 }