SteamWar/BungeeCore
Archiviert
13
2

Rework of the Codesite Group System #139

Manuell gemergt
Lixfel hat 11 Commits von change-color nach master 2020-10-31 10:47:43 +01:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
Chaoscaot hat den Titel von WIP: Rework of the Codesiede Group System zu WIP: Rework of the Codeside Group System 2020-10-28 19:27:27 +01:00 geändert
Chaoscaot hat den Titel von WIP: Rework of the Codeside Group System zu WIP: Rework of the Codesite Group System 2020-10-28 19:31:24 +01:00 geändert
Chaoscaot hat den Titel von WIP: Rework of the Codesite Group System zu Rework of the Codesite Group System 2020-10-28 22:35:41 +01:00 geändert
Lixfel hat 2020-10-29 08:34:56 +01:00 Änderungen angefragt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Finde ich gut, dass du das mal aufräumst :) , dann würde ich ggf. aber auch bevorzugen, gleich komplett die Sachen, die noch über das Bungee-Permissionsystem laufen, gleich noch auf UserGroup umzustellen.

Finde ich gut, dass du das mal aufräumst :) , dann würde ich ggf. aber auch bevorzugen, gleich komplett die Sachen, die noch über das Bungee-Permissionsystem laufen, gleich noch auf UserGroup umzustellen.
@ -255,2 +255,4 @@
}
}
public enum UserGroup {
Besitzer

Ich glaube, als separate Datei wäre das ganze angenehmer.

Ich glaube, als separate Datei wäre das ganze angenehmer.
@ -257,0 +265,4 @@
YOUTUBER("Youtuber", "§5", false, false, false),
MEMBER("Member", "§7", false, false, false);
public static final UserGroup STANDARD_GROUP = UserGroup.MEMBER;
Besitzer

Man kann immer Member als Standardgroup annehmen (kein separater Wert nötig), da es schlicht und ergreifend derzeit unvorstellbar ist, dass sich das mal ändert.

Man kann immer Member als Standardgroup annehmen (kein separater Wert nötig), da es schlicht und ergreifend derzeit unvorstellbar ist, dass sich das mal ändert.
@ -257,0 +267,4 @@
public static final UserGroup STANDARD_GROUP = UserGroup.MEMBER;
private final String displayName;
Besitzer

Statt DisplayName separat zu haben, wäre es da nicht angenehmer, auf .name() (also den internen Namen) zurückzugreifen und diesen dann genauso entsprechend mit Groß- und Kleinschreibung anzupassen/zu setzen?

Statt DisplayName separat zu haben, wäre es da nicht angenehmer, auf .name() (also den internen Namen) zurückzugreifen und diesen dann genauso entsprechend mit Groß- und Kleinschreibung anzupassen/zu setzen?
@ -257,0 +301,4 @@
return checkSchematics;
}
public static UserGroup getUserGroupByPermission(ProxiedPlayer p) {
Besitzer

Wenn man den ProxiedPlayer hat, sollte der Weg eigentlich über SteamwarUser.get(p).getUserGroup() laufen, und nicht über das BungeeCord-Permissionsystem.

Wenn man den ProxiedPlayer hat, sollte der Weg eigentlich über SteamwarUser.get(p).getUserGroup() laufen, und nicht über das BungeeCord-Permissionsystem.
Lixfel hat 2020-10-30 21:44:27 +01:00 Änderungen angefragt
@ -47,2 +47,4 @@
return;
SteamwarUser p = SteamwarUser.get(sender.getName());
if(p.getUserGroup().isHigherThen(target.getUserGroup())) {
Besitzer

Damit fangen wir gar nicht erst an.

Damit fangen wir gar nicht erst an.
@ -43,2 +43,4 @@
return;
SteamwarUser p = SteamwarUser.get(sender.getName());
if(p.getUserGroup().isHigherThen(target.getUserGroup())) {
Besitzer

Damit fangen wir gar nicht erst an.

Damit fangen wir gar nicht erst an.
YoyoNow hat die Änderungen 2020-10-30 22:19:05 +01:00 genehmigt
Lixfel hat 2020-10-30 22:21:20 +01:00 Änderungen angefragt
@ -69,3 +69,3 @@
uuid = UUID.fromString(rs.getString("UUID"));
userName = rs.getString("UserName");
userGroup = rs.getString("UserGroup");
userGroup = UserGroup.valueOf(rs.getString("UserGroup").toUpperCase());
Besitzer

Das gibt NPEs.

Das gibt NPEs.
@ -91,3 +91,3 @@
}
}else{
SQL.update("INSERT INTO UserData (UUID, UserName, UserGroup) VALUES (?, ?, 'Member')", connection.getUniqueId().toString(), connection.getName());
SQL.update("INSERT INTO UserData (UUID, UserName, UserGroup) VALUES (?, ?, ?)", connection.getUniqueId().toString(), connection.getName(), UserGroup.Member.name());
Besitzer

Warum kann die Zeile nicht einfach bei der alten Zeile bleiben?

Warum kann die Zeile nicht einfach bei der alten Zeile bleiben?
YoyoNow hat die Änderungen 2020-10-31 09:37:41 +01:00 genehmigt
Lixfel hat die Änderungen 2020-10-31 10:47:38 +01:00 genehmigt
Lixfel hat diesen Pull-Request 2020-10-31 10:47:43 +01:00 geschlossen
Lixfel löschte die Branch change-color 2020-10-31 10:48:31 +01:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.