Grasscutters/Grasscutter
3
0
Fork 0
mirror of https://github.com/Melledy/Grasscutter.git synced 2024-11-23 16:16:46 +00:00
Code Issues Projects Releases Wiki Activity

Fix logic error in remove command

Browse Source
This commit is contained in:
AnimeGitB 2022-05-15 17:26:54 +09:30 committed by Melledy
parent 9380e9cda4
commit 2564fefe02
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/main/java/emu/grasscutter/command/commands/RemoveCommand.java
View File

@ -8,7 +8,6 @@ import emu.grasscutter.server.packet.send.PacketChangeMpTeamAvatarRsp;
import java.util.ArrayList;
import java.util.Collections;
import java.util.Comparator;
import java.util.List;
import static emu.grasscutter.utils.Language.translate;
@ -28,7 +27,9 @@ public class RemoveCommand implements CommandHandler {
for (String arg : args) {
try {
int avatarIndex = Integer.parseInt(arg);
avatarIndexList.add(avatarIndex);
if (!avatarIndexList.contains(avatarIndex)) {
avatarIndexList.add(avatarIndex);
}
} catch (Exception ignored) {
ignored.printStackTrace();
CommandHandler.sendMessage(sender, translate("commands.remove.invalid_index"));
@ -36,7 +37,7 @@ public class RemoveCommand implements CommandHandler {
}
}
Collections.reverse(avatarIndexList);
Collections.sort(avatarIndexList, Collections.reverseOrder());
for (int i = 0; i < avatarIndexList.size(); i++) {
if (avatarIndexList.get(i) > targetPlayer.getTeamManager().getCurrentTeamInfo().getAvatars().size() || avatarIndexList.get(i) <= 0) {