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

WIP: Update to Linkage System #212

Geschlossen
YoyoNow möchte 6 Commits von Linkage nach master mergen
pullen von: Linkage
Ziel: SteamWar:master
Diskussion 0 Commits 6 Geänderte Dateien 12 +43 -45
YoyoNow hat 2022-09-23 12:50:38 +02:00 kommentiert
Besitzer
Link kopieren
Keine Beschreibung angegeben.
YoyoNow hat 1 Commit 2022-09-23 12:50:39 +02:00 hinzugefügt
Update to Linkage System
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
Details
8f544c3fd0
YoyoNow hat 1 Commit 2022-09-23 12:52:45 +02:00 hinzugefügt
Merge branch 'master' into Linkage
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Details
b9acaf82ca
Lixfel hat 2022-09-24 11:09:10 +02:00 Änderungen angefragt
SpigotCore_Main/src/de/steamwar/core/Core.java Veraltet
@ -133,1 +126,3 @@
//new ServerDataHandler();
LinkageUtils.run(ListenerLink.class, EnableLink.class);
TinyProtocol.init();
LinkageUtils.run(Plain.class);
Lixfel hat 2022-09-24 11:06:34 +02:00 kommentiert
Besitzer
Link kopieren

Was ist der Unterschied zwischen EnableLink und Plain? Btw. kannst du TinyProtocol auch vor den anderen Sachen initialisieren.

Was ist der Unterschied zwischen EnableLink und Plain? Btw. kannst du TinyProtocol auch vor den anderen Sachen initialisieren.
YoyoNow hat 2022-09-24 11:11:08 +02:00 kommentiert
Autor
Besitzer
Link kopieren

EnableLink ruft noch die enable methode auf, während Plain nur das Objekt erzeugt.

EnableLink ruft noch die enable methode auf, während Plain nur das Objekt erzeugt.
Lixfel hat 2022-09-24 11:11:54 +02:00 kommentiert
Besitzer
Link kopieren

Und warum würde es nicht funktionieren, wenn bei EnableLink die enable-Methode der Konstruktor ist?

Und warum würde es nicht funktionieren, wenn bei EnableLink die enable-Methode der Konstruktor ist?
YoyoNow hat 2022-09-24 11:14:55 +02:00 kommentiert
Autor
Besitzer
Link kopieren

Weil ich an manchen stellen im BauSystem auf einem Objekt (also Singleton) mehrfach die enable methode brauche!

Weil ich an manchen stellen im BauSystem auf einem Objekt (also Singleton) mehrfach die enable methode brauche!
Lixfel hat 2022-09-24 11:17:03 +02:00 kommentiert
Besitzer
Link kopieren

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?

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?
YoyoNow hat 2022-09-24 11:19:22 +02:00 kommentiert
Autor
Besitzer
Link kopieren

Weil zum Teil zwischendurch noch field injection gemacht wird, welche in der enable methode nicht null sein dürfen. Auch hier im BauSystem wichtig.

Weil zum Teil zwischendurch noch field injection gemacht wird, welche in der enable methode nicht null sein dürfen. Auch hier im BauSystem wichtig.
YoyoNow markierte diese Unterhaltung als gelöst
SpigotCore_Main/src/de/steamwar/core/authlib/AuthlibInjector.java Veraltet
@ -26,2 +30,2 @@
public class AuthlibInjector {
private AuthlibInjector() {}
@Linked(EnableLink.class)
@MaxVersion(19)
Lixfel hat 2022-09-24 11:07:12 +02:00 kommentiert
Besitzer
Link kopieren

Inklusiv oder Exklusiv? Ich finde die exklusive Range hier verwirrend.

Inklusiv oder Exklusiv? Ich finde die exklusive Range hier verwirrend.
YoyoNow hat 2022-09-24 11:10:47 +02:00 kommentiert
Autor
Besitzer
Link kopieren

Da es bis jetzt diese checks immer exklusiv gemacht wurden, habe ich dies erstmal so übernommen, kann dies aber gerne ändern.

Da es bis jetzt diese checks immer exklusiv gemacht wurden, habe ich dies erstmal so übernommen, kann dies aber gerne ändern.
Lixfel hat 2022-09-24 11:11:22 +02:00 kommentiert
Besitzer
Link kopieren

Die Checks wurden immer mit einem Vergleich gemacht, das hier aber als "MaxVersion" zu betiteln, verwirrt.

Die Checks wurden immer mit einem Vergleich gemacht, das hier aber als "MaxVersion" zu betiteln, verwirrt.
YoyoNow hat 2022-09-24 11:11:45 +02:00 kommentiert
Autor
Besitzer
Link kopieren

Gut dann wird das umgebaut.

Gut dann wird das umgebaut.
YoyoNow markierte diese Unterhaltung als gelöst
SpigotCore_Main/src/de/steamwar/core/events/PartialChunkFixer.java Veraltet
@ -36,2 +40,4 @@
* This class can only be loaded on 1.9 to 1.15 with active ViaVersion.
**/
@Linked(Plain.class)
@MaxVersion(17)
Lixfel hat 2022-09-24 11:08:57 +02:00 kommentiert
Besitzer
Link kopieren

Genauso.

Genauso.
YoyoNow markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2022-09-24 11:23:51 +02:00 hinzugefügt
Fix PR stuff
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Details
7b98669679
YoyoNow hat 1 Commit 2022-09-24 11:24:46 +02:00 hinzugefügt
Fix PR stuff
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Details
293b5b9e4a
YoyoNow hat 2 Commits 2022-09-24 19:48:18 +02:00 hinzugefügt
Update CommonCore
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Details
70de18adf4 
Update handlers
Lixfel hat den Titel von Update to Linkage System zu WIP: Update to Linkage System 2023-06-13 22:30:59 +02:00 geändert
Lixfel hat diesen Pull-Request 2024-03-17 15:59:19 +01:00 geschlossen
Lixfel löschte die Branch Linkage 2024-03-17 15:59:23 +01:00
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Erforderlich
Details

Pull-Request geschlossen

Dieser Pull-Request kann nicht wieder geöffnet werden, da die Branch bereits gelöscht wurde.
Anmelden, um an der Diskussion teilzunehmen.
Reviewer
Keine Reviewer
Lixfel
Label
Label entfernen
  
Bug

Etwas funktioniert nicht oder ist fehlerhaft.

  
Codeverbesserung

Ein Codeverbesserungsvorschlag

  
Einsteiger Freundlich

Ist für Leute welche noch nicht so eingearbeitet sind ein guter Start.

  
Idee

Bisher unausgereifte Idee für ein neues Feature

  
In Arbeit

Naja hier dran wird gearbeitet

  
Neues Feature

Das ist ein neues Feature

  
Prio A

Oberste Priorität

  
Security Breach

Die Sicherheit der User ist beeinträchtigt!

  
Überprüfung notwendig

Naja hier muss noch was überprüft werden?

  
Verbesserung

Ein Verbesserungsvorschlag

  
Zu Beobachten

Sollte gefixt sein bzw. kann bislang nicht bestätigt werden

Kein Label
Bug
Codeverbesserung
Einsteiger Freundlich
Idee
In Arbeit
Neues Feature
Prio A
Security Breach
Überprüfung notwendig
Verbesserung
Zu Beobachten
Meilenstein
Meilenstein entfernen
Keine Einträge
Kein Meilenstein
Zuständig
Zuständige entfernen
Niemand zuständig
2 Beteiligte
Nachrichten
Fällig am
Das Fälligkeitsdatum ist ungültig oder außerhalb des zulässigen Bereichs. Bitte verwende das Format „jjjj-mm-tt“.

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: SteamWar/SpigotCore#212
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?