Fix sethome and hopefully add the ability for the config to update

Author: RhythmicSys

pom.xml

@@ -6,7 +6,7 @@
 
   <groupId>simplexity</groupId>
   <artifactId>SimpleHomes</artifactId>
-  <version>1.2.1</version>
+  <version>1.2.2</version>
   <packaging>jar</packaging>
 
   <name>SimpleHomes</name>

src/main/java/simplexity/simplehomes/SimpleHomes.java

@@ -26,13 +26,10 @@ public final class SimpleHomes extends JavaPlugin {
     @Override
     public void onEnable() {
         instance = this;
-        saveDefaultConfig();
-        ConfigHandler.getInstance().loadConfigValues();
-        LocaleHandler.getInstance().loadLocale();
+        handleConfig();
         SQLHandler.getInstance().init();
-        this.getServer().getPluginManager().registerEvents(new PlayerMoveListener(), this);
-        this.getServer().getPluginManager().registerEvents(new PlayerLeaveListener(), this);
         registerCommands();
+        registerListeners();
     }
 
     public static SimpleHomes getInstance() {
@@ -51,4 +48,17 @@ private void registerCommands() {
         Objects.requireNonNull(this.getCommand("homesreload")).setExecutor(new HomesReload());
         Objects.requireNonNull(this.getCommand("importhomes")).setExecutor(new ImportHomes());
     }
+
+    private void registerListeners(){
+        this.getServer().getPluginManager().registerEvents(new PlayerMoveListener(), this);
+        this.getServer().getPluginManager().registerEvents(new PlayerLeaveListener(), this);
+    }
+
+    private void handleConfig(){
+        saveDefaultConfig();
+        getConfig().options().copyDefaults(true);
+        saveConfig();
+        ConfigHandler.getInstance().loadConfigValues();
+        LocaleHandler.getInstance().loadLocale();
+    }
 }

src/main/java/simplexity/simplehomes/commands/SetHome.java

@@ -22,7 +22,11 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
             return false;
         }
         List<Home> playerHomes = SQLHandler.getInstance().getHomes(player.getUniqueId());
-
+        if (!canSetMoreHomes(player, playerHomes)) {
+            sender.sendRichMessage(LocaleHandler.getInstance().getCannotSetMoreHomes(),
+                    Placeholder.parsed("value", String.valueOf(CommandUtils.maxHomesPermission(player))));
+            return false;
+        }
         if (args.length == 0) {
             handleNoArgs(player, playerHomes);
             return true;
@@ -57,4 +61,10 @@ private void handleSetHome(Player player, String homeName) {
         SQLHandler.getInstance().setHome(player.getUniqueId(), player.getLocation(), homeName);
         player.sendRichMessage(LocaleHandler.getInstance().getHomeSet(), Placeholder.unparsed("name", homeName));
     }
+
+    private boolean canSetMoreHomes(Player player, List<Home> homesList) {
+        int maxHomes = CommandUtils.maxHomesPermission(player);
+        int currentHomes = homesList.size();
+        return currentHomes < maxHomes;
+    }
 }