Methods should be also renamed.

Author: Dviih

src/main/java/pro/cloudnode/smp/cloudnodemsg/command/MessageCommand.java

@@ -33,7 +33,7 @@ public boolean run(final @NotNull CommandSender sender, final @NotNull String la
         if (recipient.isEmpty() || (CloudnodeMSG.isVanished(recipient.get()) && !sender.hasPermission(Permission.SEND_VANISHED))) return new PlayerNotFoundError(args[0]).send(sender);
         if (sender instanceof final @NotNull Player player && recipient.get().getUniqueId().equals(player.getUniqueId()))
             return new MessageYourselfError().send(sender);
-        if (!Message.isIncomeEnabled(recipient.get()) && !sender.hasPermission(Permission.TOGGLE_BYPASS)) return new PlayerHasIncomingDisabledError(recipient.get().getName()).send(sender);
+        if (!Message.isIncomingEnabled(recipient.get()) && !sender.hasPermission(Permission.TOGGLE_BYPASS)) return new PlayerHasIncomingDisabledError(recipient.get().getName()).send(sender);
 
         try {
             new Message(Message.offlinePlayer(sender), recipient.get(), String.join(" ", Arrays.copyOfRange(args, 1, args.length))).send();

src/main/java/pro/cloudnode/smp/cloudnodemsg/command/ReplyCommand.java

@@ -27,7 +27,7 @@ public boolean run(final @NotNull CommandSender sender, final @NotNull String la
 
         final @NotNull Optional<@NotNull OfflinePlayer> recipient = Message.getReplyTo(Message.offlinePlayer(sender));
         if (recipient.isEmpty()) return new NobodyReplyError().send(sender);
-        if (!Message.isIncomeEnabled(Objects.requireNonNull(recipient.get().getPlayer())) && !sender.hasPermission(Permission.TOGGLE_BYPASS)) return new PlayerHasIncomingDisabledError(Objects.requireNonNull(recipient.get().getName())).send(sender);
+        if (!Message.isIncomingEnabled(Objects.requireNonNull(recipient.get().getPlayer())) && !sender.hasPermission(Permission.TOGGLE_BYPASS)) return new PlayerHasIncomingDisabledError(Objects.requireNonNull(recipient.get().getName())).send(sender);
         if (
             !recipient.get().getUniqueId().equals(Message.console.getUniqueId())
             && (

src/main/java/pro/cloudnode/smp/cloudnodemsg/command/ToggleMessageCommand.java

@@ -28,29 +28,29 @@ public boolean run(@NotNull CommandSender sender, @NotNull String label, @NotNul
             if (recipient.getPlayer() == null)
                 return new NeverJoinedError(Optional.ofNullable(recipient.getName()).orElse("Unknown Player")).send(sender);
 
-            if (Message.isIncomeEnabled(recipient.getPlayer())) {
-                Message.incomeDisable(recipient.getPlayer());
+            if (Message.isIncomingEnabled(recipient.getPlayer())) {
+                Message.incomingDisable(recipient.getPlayer());
                 sendMessage(sender, CloudnodeMSG.getInstance().config().toggleDisableOther(Optional.of(recipient.getPlayer().getName()).orElse("Unknown Player")));
 
                 return true;
             }
 
-            Message.incomeEnable(recipient.getPlayer());
+            Message.incomingEnable(recipient.getPlayer());
             sendMessage(sender, CloudnodeMSG.getInstance().config().toggleEnableOther(Optional.of(recipient.getPlayer().getName()).orElse("Unknown Player")));
 
             return true;
         }
         if (args.length > 1) return sendMessage(sender, CloudnodeMSG.getInstance().config().usage(label, usage));
         if (!(sender instanceof final @NotNull Player player)) return new NotPlayerError().send(sender);
 
-        if (Message.isIncomeEnabled(player)) {
-            Message.incomeDisable(player);
+        if (Message.isIncomingEnabled(player)) {
+            Message.incomingDisable(player);
             sendMessage(sender, CloudnodeMSG.getInstance().config().toggleDisable());
 
             return true;
         }
 
-        Message.incomeEnable(player);
+        Message.incomingEnable(player);
         sendMessage(sender, CloudnodeMSG.getInstance().config().toggleEnable());
 
         return true;

src/main/java/pro/cloudnode/smp/cloudnodemsg/message/Message.java

@@ -141,7 +141,7 @@ public static void unignore(final @NotNull Player player, final @NotNull Offline
      *
      * @param player The player
      */
-    public static void incomeEnable(final @NotNull Player player) {
+    public static void incomingEnable(final @NotNull Player player) {
         player.getPersistentDataContainer().set(INCOMING_ENABLED, PersistentDataType.BOOLEAN, true);
     }
 
@@ -150,7 +150,7 @@ public static void incomeEnable(final @NotNull Player player) {
      *
      * @param player The player
      */
-    public static void incomeDisable(final @NotNull Player player) {
+    public static void incomingDisable(final @NotNull Player player) {
         player.getPersistentDataContainer().set(INCOMING_ENABLED, PersistentDataType.BOOLEAN, false);
     }
 
@@ -159,7 +159,7 @@ public static void incomeDisable(final @NotNull Player player) {
      *
      * @param player The player
      */
-    public static boolean isIncomeEnabled(final @NotNull Player player) {
+    public static boolean isIncomingEnabled(final @NotNull Player player) {
         return player.getPersistentDataContainer().getOrDefault(INCOMING_ENABLED, PersistentDataType.BOOLEAN, true);
     }
 }