Skip to content

Commit d863d7a

Browse files
committed
improved kick command
1 parent bee07c8 commit d863d7a

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

src/main/java/pro/cloudnode/smp/smpcore/command/NationCommand.java

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -181,10 +181,7 @@ public boolean kickMember(
181181
final @NotNull String label,
182182
final @NotNull String @NotNull [] args
183183
) {
184-
if (!sender.hasPermission(Permission.NATION_CITIZENS_KICK_OTHER) && (
185-
member == null || !nation.id.equals(member.nationID)
186-
|| !sender.hasPermission(Permission.NATION_CITIZENS_KICK)
187-
))
184+
if (!sender.hasPermission(Permission.NATION_CITIZENS_KICK))
188185
return sendMessage(sender, SMPCore.messages().errorNoPermission());
189186

190187
if (args.length == 0)
@@ -196,7 +193,7 @@ public boolean kickMember(
196193
return sendMessage(sender, SMPCore.messages().errorNotMember(target));
197194

198195
final @NotNull Member targetMember = targetMemberOptional.get();
199-
if (targetMember.nationID == null || !targetMember.nationID.equals(member.nationID))
196+
if (targetMember.nationID == null || !targetMember.nationID.equals(nation.id) || (member != null && !nation.id.equals(member.nationID)))
200197
return sendMessage(sender, SMPCore.messages().errorNotCitizen(targetMember));
201198
if (targetMember.uuid.equals(nation.leaderUUID) || targetMember.uuid.equals(nation.viceLeaderUUID))
202199
return sendMessage(sender, SMPCore.messages().errorKickLeadership());

0 commit comments

Comments
 (0)