SteamWar/SpigotCore
SteamWar/SpigotCore
Archiviert
13
0
Fork 0
Code Issues 4 Pull-Requests 1 Releases Wiki Aktivität

CommandFramework3 #94

Manuell gemergt
Zeanon hat 71 Commits von CommandFramework3 nach master 2021-03-30 21:15:40 +02:00 zusammengeführt
Diskussion 3 Commits 71 Geänderte Dateien 6 +6 -7
1 geänderte Dateien mit 6 neuen und 7 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 1b10c96dc6 werden angezeigt - Alle Commits anzeigen

13
SpigotCore_Main/src/de/steamwar/command/SubCommand.java
Datei anzeigen

@ -21,7 +21,6 @@ package de.steamwar.command;
import org.bukkit.command.CommandSender;
import java.io.IOException;
import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
import java.lang.reflect.Parameter;
@ -107,16 +106,16 @@ class SubCommand {
List<String> tabComplete(CommandSender commandSender, String[] args) {
List<String> argsList = Arrays.asList(args);
Lixfel hat 2021-03-30 08:40:49 +02:00 kommentiert
Veraltet
Review
Link kopieren

Entweder du behandelst im SWCommand nicht den Fall null, oder du returnst immer eine leere Liste. Bitte nicht beides zeitgleich.

Entweder du behandelst im SWCommand nicht den Fall null, oder du returnst immer eine leere Liste. Bitte nicht beides zeitgleich.
YoyoNow hat 2021-03-30 09:22:42 +02:00 kommentiert
Veraltet
Review
Link kopieren

Denn fall null behandle ich eigentlich wegen unsauberen TabCompletern, die eigengeschrieben sind.

Denn fall null behandle ich eigentlich wegen unsauberen TabCompletern, die eigengeschrieben sind.
Lixfel hat 2021-03-30 10:06:40 +02:00 kommentiert
Veraltet
Review
Link kopieren

Dann nutze doch auch einfach immer null.

Dann nutze doch auch einfach immer null.
for (int i = 0; i < subCommand.length; i++) {
for (String value : subCommand) {
String s = argsList.remove(0);
if (argsList.isEmpty()) return Collections.singletonList(subCommand[i]);
if (!subCommand[i].equals(s)) return Collections.emptyList();
if (argsList.isEmpty()) return Collections.singletonList(value);
if (!value.equals(s)) return Collections.emptyList();
}
for (int i = 0; i < arguments.length; i++) {
for (TypeMapper<?> argument : arguments) {
String s = argsList.remove(0);
if (argsList.isEmpty()) return arguments[i].tabCompletes(s);
if (argsList.isEmpty()) return argument.tabCompletes(s);
try {
arguments[i].map(s);
argument.map(s);
} catch (Exception e) {
return Collections.emptyList();
}