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

Move BauServerInfo to SpigotCore #160

Zusammengeführt
Lixfel hat 4 Commits von provider nach master 2022-01-28 15:59:29 +01:00 zusammengeführt
Diskussion 7 Commits 4 Geänderte Dateien 1 +47
Chaoscaot hat 2022-01-27 21:40:16 +01:00 kommentiert
Besitzer
Link kopieren
Keine Beschreibung angegeben.
Chaoscaot hat 1 Commit 2022-01-27 21:40:17 +01:00 hinzugefügt
Move Provider to SpigotCore
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Details
9ff498133d
Lixfel hat 2022-01-27 21:43:48 +01:00 Änderungen angefragt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Nope! Viel zu komplex und Querabhängigkeiten! Viel besserer Ansatz: Code zur Bestimmung des Owners vom BauSystem in den SpigotCore verlegen. (Ggf. den Code aus den BauSystem 1.0 bevorzugen, ich weiß nicht, ob das BauSystem2.0 UUID-Bauwelten (1.12) richtig handelt).

Nope! Viel zu komplex und Querabhängigkeiten! Viel besserer Ansatz: Code zur Bestimmung des Owners vom BauSystem in den SpigotCore verlegen. (Ggf. den Code aus den BauSystem 1.0 bevorzugen, ich weiß nicht, ob das BauSystem2.0 UUID-Bauwelten (1.12) richtig handelt).
YoyoNow hat 2022-01-27 22:29:27 +01:00 kommentiert
Besitzer
Link kopieren

Mehr dick gibt es nicht von dir Lixfel?

Mehr dick gibt es nicht von dir Lixfel?
Lixfel hat 2022-01-27 22:31:10 +01:00 kommentiert
Besitzer
Link kopieren

Der Code zur Bestimmung des BauweltOwners ist ~15 Zeilen lang, max. Und braucht auch keine Reflection etc... Und keine inverse Abhängigkeit...

Der Code zur Bestimmung des BauweltOwners ist ~15 Zeilen lang, max. Und braucht auch keine Reflection etc... Und keine inverse Abhängigkeit...
YoyoNow hat 2022-01-27 22:32:56 +01:00 kommentiert
Besitzer
Link kopieren

Das hättest du auch direkt im Discord schreiben können! Dann wäre hier der PR direkt anders da gewesen! So ist das einfach nur arschig!

Das hättest du auch direkt im Discord schreiben können! Dann wäre hier der PR direkt anders da gewesen! So ist das einfach nur arschig!
Lixfel hat 2022-01-27 22:37:41 +01:00 kommentiert
Besitzer
Link kopieren

Ich dachte, dass sich diese massive Vereinfachung einfach aus der Verschiebung ergibt. Sonst hat man ja nichts damit gewonnen, das in den SpigotCore zu verschieben. Tut mir leid, falls das nicht so verstanden wurde.

Ich dachte, dass sich diese massive Vereinfachung einfach aus der Verschiebung ergibt. Sonst hat man ja nichts damit gewonnen, das in den SpigotCore zu verschieben. Tut mir leid, falls das nicht so verstanden wurde.
YoyoNow hat 2022-01-27 22:43:52 +01:00 kommentiert
Besitzer
Link kopieren

Also in der 1.15 braucht man die UUID Sache nicht mehr, deswegen auch soweit ich weiß nicht unterstütz. Desweiteren finde ich die Indirektion schon gut, da das BauSystem sich um die Sache kümmert und nicht der SpigotCore, der kann es nur anderen bereit stellen, wenn die es brauchen und auf einem BauServer laufen!

Also in der 1.15 braucht man die UUID Sache nicht mehr, deswegen auch soweit ich weiß nicht unterstütz. Desweiteren finde ich die Indirektion schon gut, da das BauSystem sich um die Sache kümmert und nicht der SpigotCore, der kann es nur anderen bereit stellen, wenn die es brauchen **und** auf einem BauServer laufen!
Lixfel hat 2022-01-27 22:45:40 +01:00 kommentiert
Besitzer
Link kopieren

Anscheinend braucht auch zukünftig das SchematicSystem die Information über den Owner. Mehrere Spigotsysteme brauchen gemeinsamen Code => Spigotcore. Das Bausystem braucht ja auch nicht unbedingt die Fight-Tabelle, aber halt trotzdem Datenbankanbindung...

Anscheinend braucht auch zukünftig das SchematicSystem die Information über den Owner. Mehrere Spigotsysteme brauchen gemeinsamen Code => Spigotcore. Das Bausystem braucht ja auch nicht unbedingt die Fight-Tabelle, aber halt trotzdem Datenbankanbindung...
YoyoNow hat 2022-01-27 22:49:19 +01:00 kommentiert
Besitzer
Link kopieren

Weiß trotzdem nicht ob das so gut ist.

Weiß trotzdem nicht ob das so gut ist.
Chaoscaot hat 1 Commit 2022-01-28 15:37:59 +01:00 hinzugefügt
Change Providers
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Details
4e801ea4eb
Chaoscaot hat ein Review von Lixfel 2022-01-28 15:39:16 +01:00 angefragt
Chaoscaot hat den Titel von Move Provider to SpigotCore zu Move BauServerInfo to SpigotCore 2022-01-28 15:39:31 +01:00 geändert
Chaoscaot hat 1 Commit 2022-01-28 15:55:22 +01:00 hinzugefügt
Simplify
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Details
c593e23b80
Chaoscaot hat 1 Commit 2022-01-28 15:57:50 +01:00 hinzugefügt
Ignored
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Details
79d9a8b902
Lixfel hat die Änderungen 2022-01-28 15:59:12 +01:00 genehmigt
Lixfel hat Commit 8caaa01204 in master 2022-01-28 15:59:29 +01:00 gemerged
Lixfel löschte die Branch provider 2022-01-28 15:59:30 +01:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
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
3 Beteiligte
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#160
Keine Beschreibung angegeben.
Branch "provider" 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?