don't allow inviting players already in party

Hexaoxide · Sep 30, 2023 · 89b1065 · 89b1065
1 parent 416be97
commit 89b1065
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/common/src/main/java/net/draycia/carbon/common/command/commands/PartyCommands.java b/common/src/main/java/net/draycia/carbon/common/command/commands/PartyCommands.java
@@ -172,6 +172,11 @@ private void invitePlayer(final CommandContext<Commander> ctx) {
             this.messages.mustBeInParty(player);
             return;
         }
+        final @Nullable Party recipientParty = recipient.party().join();
+        if (recipientParty != null && recipientParty.id().equals(party.id())) {
+            this.messages.alreadyInParty(player, recipient.displayName());
+            return;
+        }
         this.partyInvites.sendInvite(player.uuid(), recipient.uuid(), party.id());
         this.messages.receivedPartyInvite(recipient, player.displayName(), party.name());
     }

diff --git a/common/src/main/java/net/draycia/carbon/common/messages/CarbonMessages.java b/common/src/main/java/net/draycia/carbon/common/messages/CarbonMessages.java
@@ -469,6 +469,9 @@ void errorCommandCommandExecution(
     @Message("command.party.cannot_invite_self")
     void cannotInviteSelf(Audience audience);
 
+    @Message("command.party.already_in_party")
+    void alreadyInParty(Audience audience, Component displayName);
+
     @Message("command.party.description")
     Component partyDesc();
 

diff --git a/common/src/main/resources/locale/messages-en_US.properties b/common/src/main/resources/locale/messages-en_US.properties
@@ -80,6 +80,7 @@ command.party.create.description=Create a new party.
 command.party.invite.description=Invite a player to your party.
 command.party.accept.description=Accept party invites.
 command.party.leave.description=Leave your current party.
+command.party.already_in_party=<display_name><red> is already in your party.
 command.party.disband.description=Disband your current party.
 config.reload.failed=<red>Config failed to reload
 config.reload.success=<green>Config reloaded successfully