version bump and fix login check

Author: RhythmicSys

pom.xml

@@ -6,7 +6,7 @@
 
   <groupId>simplexity</groupId>
   <artifactId>SimpleFly</artifactId>
-  <version>0.0.1</version>
+  <version>1.0.0</version>
   <packaging>jar</packaging>
 
   <name>SimpleFly</name>

src/main/java/simplexity/simplefly/FlyListeners.java

@@ -35,7 +35,7 @@ public void onPlayerLogin(PlayerJoinEvent joinEvent) {
                 return;
             }
             if (flyEnabled && !player.hasPermission(Constants.FLY_PERMISSION)) {
-                FlyLogic.flyDisable(player);
+                FlyLogic.disablePDC(player);
             }
         }, 10);
     }

src/main/java/simplexity/simplefly/FlyLogic.java

@@ -41,6 +41,10 @@ public static void flyDisable(Player player){
         player.setFlying(false);
     }
 
+    public static void disablePDC(Player player){
+        player.getPersistentDataContainer().set(flyStatus, PersistentDataType.BOOLEAN, false);
+    }
+
     private static boolean willFall(Player player){
         Location location = player.getLocation();
         Block blockBelow = location.clone().add(0, -1, 0).getBlock();

src/main/java/simplexity/simplefly/commands/Fly.java

@@ -73,42 +73,25 @@ private static int executeOnOtherWithArg(CommandContext<CommandSourceStack> ctx,
             sender.sendMessage(getParsedComponent(shouldEnable, player, sender, LocaleMessage.FLY_SET_OTHER.getMessage()));
             return Command.SINGLE_SUCCESS;
         }
-        int modified = 0;
         for (Player player : targets) {
             FlyLogic.setFlyStatus(player, shouldEnable);
             player.sendMessage(getParsedComponent(shouldEnable, player, sender, LocaleMessage.FLY_SET_BY_OTHER.getMessage()));
-            modified++;
-        }
-        String enabledString;
-        if (shouldEnable) {
-            enabledString = LocaleMessage.ENABLED.getMessage();
-        } else {
-            enabledString = LocaleMessage.DISABLED.getMessage();
         }
+        String enabledString = shouldEnable ? LocaleMessage.ENABLED.getMessage() : LocaleMessage.DISABLED.getMessage();
         sender.sendRichMessage(LocaleMessage.FLY_SET_OTHER_MANY_ARG.getMessage(),
                 Placeholder.parsed("value", enabledString),
-                Placeholder.parsed("count", String.valueOf(modified)));
+                Placeholder.parsed("count", String.valueOf(targets.size())));
         return Command.SINGLE_SUCCESS;
     }
 
     private static void sendOwnMessage(boolean flyEnabled, Player player) {
-        String enabledString;
-        if (flyEnabled) {
-            enabledString = LocaleMessage.ENABLED.getMessage();
-        } else {
-            enabledString = LocaleMessage.DISABLED.getMessage();
-        }
+        String enabledString = flyEnabled ? LocaleMessage.ENABLED.getMessage() : LocaleMessage.DISABLED.getMessage();
         player.sendRichMessage(LocaleMessage.FLY_SET_OWN.getMessage(),
                 Placeholder.parsed("value", enabledString));
     }
 
     private static Component getParsedComponent(boolean flyEnabled, Player target, CommandSender source, String message) {
-        String enabledString;
-        if (flyEnabled) {
-            enabledString = LocaleMessage.ENABLED.getMessage();
-        } else {
-            enabledString = LocaleMessage.DISABLED.getMessage();
-        }
+        String enabledString = flyEnabled ? LocaleMessage.ENABLED.getMessage() : LocaleMessage.DISABLED.getMessage();
         Component targetName = target.displayName();
         Component senderName;
         if (source instanceof Player player) {