feedc0de/LogBlock
1
0
Fork 0
forked from LogBlock/LogBlock
Code Pull Requests Activity

Player name auto-completion does more harm than good, so don't do that

Browse Source 
Fixes #388
This commit is contained in:
Brokkonaut
2018-08-01 00:38:49 +02:00
parent 1b0a575945
commit 4baa989e60
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/main/java/de/diddiz/LogBlock/QueryParams.java
View File

@ -585,11 +585,8 @@ public final class QueryParams implements Cloneable {
if (playerName.contains("\"")) { if (playerName.contains("\"")) {
players.add(playerName.replaceAll("[^a-zA-Z0-9_]", "")); players.add(playerName.replaceAll("[^a-zA-Z0-9_]", ""));
} else { } else {
final List<Player> matches = logblock.getServer().matchPlayer(playerName); final Player matches = logblock.getServer().getPlayerExact(playerName);
if (matches.size() > 1) { players.add(matches != null ? matches.getName() : playerName.replaceAll("[^a-zA-Z0-9_]", ""));
throw new IllegalArgumentException("Ambiguous playername '" + param + "'");
}
players.add(matches.size() == 1 ? matches.get(0).getName() : playerName.replaceAll("[^a-zA-Z0-9_]", ""));
} }
} }
} }