WIP: Update to Linkage System #212
Keine Reviewer
Lixfel
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
2 Beteiligte
Kein Fälligkeitsdatum gesetzt.
Abhängigkeiten
Keine Abhängigkeiten gesetzt.
Referenz: SteamWar/SpigotCore#212
Laden…
In neuem Issue referenzieren
Einen Benutzer sperren
Keine Beschreibung angegeben.
Branch "Linkage" 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?
@ -133,1 +126,3 @@//new ServerDataHandler();LinkageUtils.run(ListenerLink.class, EnableLink.class);TinyProtocol.init();LinkageUtils.run(Plain.class);Was ist der Unterschied zwischen EnableLink und Plain? Btw. kannst du TinyProtocol auch vor den anderen Sachen initialisieren.
EnableLink ruft noch die enable methode auf, während Plain nur das Objekt erzeugt.
Und warum würde es nicht funktionieren, wenn bei EnableLink die enable-Methode der Konstruktor ist?
Weil ich an manchen stellen im BauSystem auf einem Objekt (also Singleton) mehrfach die enable methode brauche!
Ok, und was hat dann die enable-Methode mit den LinkageUtils zu tun? Warum kann der Konstruktor bei solchen Elementen nicht einfach die enable()-Methode aufrufen?
Weil zum Teil zwischendurch noch field injection gemacht wird, welche in der enable methode nicht null sein dürfen. Auch hier im BauSystem wichtig.
@ -26,2 +30,2 @@public class AuthlibInjector {private AuthlibInjector() {}@Linked(EnableLink.class)@MaxVersion(19)Inklusiv oder Exklusiv? Ich finde die exklusive Range hier verwirrend.
Da es bis jetzt diese checks immer exklusiv gemacht wurden, habe ich dies erstmal so übernommen, kann dies aber gerne ändern.
Die Checks wurden immer mit einem Vergleich gemacht, das hier aber als "MaxVersion" zu betiteln, verwirrt.
Gut dann wird das umgebaut.
@ -36,2 +40,4 @@* This class can only be loaded on 1.9 to 1.15 with active ViaVersion.**/@Linked(Plain.class)@MaxVersion(17)Genauso.
Update to Linkage Systemzu WIP: Update to Linkage System