Fix world not loading when the world name has . in it

Author: benwoo1110

src/main/java/org/mvplugins/multiverse/core/world/WorldsConfigManager.java

@@ -121,10 +121,15 @@ private void migrateRemoveOldConfigSerializable() {
      * @return A tuple containing a list of the new WorldConfigs added and a list of the worlds removed from the config.
      */
     private NewAndRemovedWorlds parseNewAndRemovedWorlds() {
-        Set<String> allWorldsInConfig = worldsConfig.getKeys(false);
+        List<String> allWorldsInConfig = worldsConfig.getKeys(false)
+                .stream()
+                .map(this::decodeWorldName)
+                .toList();
+
         List<WorldConfig> newWorldsAdded = new ArrayList<>();
 
         for (String worldName : allWorldsInConfig) {
+            Logging.warning("Loading world: %s", worldName);
             getWorldConfig(worldName)
                     .peek(config -> config.load(getWorldConfigSection(worldName)))
                     .orElse(() -> {
@@ -174,7 +179,7 @@ public Try<Void> save() {
                 worldConfig.save().onFailure(e -> {
                     throw new RuntimeException("Failed to save world config: " + worldName, e);
                 });
-                worldsConfig.set(worldName, worldConfig.getConfigurationSection());
+                worldsConfig.set(encodeWorldName(worldName), worldConfig.getConfigurationSection());
             });
             worldsConfig.save(worldConfigFile);
         }).onFailure(e -> {
@@ -214,7 +219,7 @@ public Try<Void> save() {
      */
     public void deleteWorldConfig(@NotNull String worldName) {
         worldConfigMap.remove(worldName);
-        worldsConfig.set(worldName, null);
+        worldsConfig.set(encodeWorldName(worldName), null);
     }
 
     /**
@@ -225,11 +230,23 @@ public void deleteWorldConfig(@NotNull String worldName) {
      * @return The {@link ConfigurationSection} for the given world.
      */
     private ConfigurationSection getWorldConfigSection(String worldName) {
+        worldName = encodeWorldName(worldName);
         return worldsConfig.isConfigurationSection(worldName)
                 ? worldsConfig.getConfigurationSection(worldName)
                 : worldsConfig.createSection(worldName);
     }
 
+    /**
+     * Dot is a special character in YAML that causes sub-path issues.
+     */
+    private String encodeWorldName(String worldName) {
+        return worldName.replace(".", "[dot]");
+    }
+
+    private String decodeWorldName(String worldName) {
+        return worldName.replace("[dot]", ".");
+    }
+
     private static final class ConfigMigratedException extends RuntimeException {
         private ConfigMigratedException() {
             super("Config migrated");