Add SWCommand.Register.Registeres #104
Label
Kein Label
Bug
Codeverbesserung
Einsteiger Freundlich
Idee
In Arbeit
Neues Feature
Prio A
Security Breach
Überprüfung notwendig
Verbesserung
Zu Beobachten
Kein Meilenstein
Niemand zuständig
3 Beteiligte
Fällig am
Kein Fälligkeitsdatum gesetzt.
Abhängigkeiten
Keine Abhängigkeiten gesetzt.
Referenz: SteamWar/SpigotCore#104
Laden…
In neuem Issue referenzieren
Einen Benutzer sperren
Keine Beschreibung angegeben.
Branch "Registeres" löschen
Das Löschen eines Branches ist permanent. Obwohl der Branch für eine kurze Zeit weiter existieren könnte, kann diese Aktion in den meisten Fällen NICHT rückgängig gemacht werden. Fortfahren?
Add SWCommand.Register.Registereszu WIP: Add SWCommand.Register.RegisteresWIP: Add SWCommand.Register.Registereszu Add SWCommand.Register.RegisteresBin nicht genügend im CommandSystem drin, um die logischen Auswirkungen der Änderung zu analysieren und oder zu beurteilen.
@ -36,4 +26,0 @@
super(cause);
}
public CommandParseException(String message, Throwable cause, boolean enableSuppression, boolean writableStackTrace) {
Diese Konstruktoren werden auch in keinem aufs CommandSystem-umgewandelten Systeme verwendet?
Nicht das ich wüsste, nicht das ich es verwendet habe. Da dies nur eine Code Flow Exception ist, sollte man dies auch nicht machen. Da die anderen Werte gar nicht ausgewertet werden.
@ -198,12 +168,29 @@ public abstract class SWCommand {
SWCommandUtils.commandMap.register("steamwar", this.command);
}
public void inject(Plugin plugin) {
Das wirkt mir nach einem sehr dreckigen workaround, geht das nicht schöner?
Dies ist eigentlich kein so dreckiger workdaround, dies ist dafür da, dass man die Befehle erst nachdem alle Plugins initialisiert sind geladen werden. Dies ist besonders interessant bei so sachen, wenn man WE Command oder so überschreibt, wenn diese später als das hier erstellt werden. Ansonsten hat Zeanon die Änderungen getestet und keine Fehler gefunden. Ich glaube sogar das es etwas besser läuft, was ich aber noch nicht nachgewiesen habe.
Das ist nicht nötig, dann soll man das zumindest als soft-dependend markieren, dann können die Befehle auch sofort überschrieben werden.
Wenn du meinst. Dann schmeiße ich es dir noch raus.