SteamWar/BungeeCore
Archiviert
13
2

Update BauMemberPermission #505

Zusammengeführt
Lixfel hat 5 Commits von PermissionUpdate nach master 2024-02-17 08:32:17 +01:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
YoyoNow hat 1 Commit 2023-12-22 17:07:58 +01:00 hinzugefügt
Update BauMemberPermission
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
3752f3f145
Lixfel hat 2024-01-10 18:00:21 +01:00 Änderungen angefragt
@ -131,0 +128,4 @@
boolean atLeastOneSupervisor = playersToSend.stream().anyMatch(player -> {
BauweltMember bauweltMember = BauweltMember.getBauMember(owner, player.getUniqueId());
if (bauweltMember == null) return false;
return bauweltMember.isSupervisor() || player.getUniqueId().equals(owner);
Besitzer

Ich glaube die Logik dieser Funktion ist falsch. Es kann an dieser Stelle davon ausgegangen werden, dass jeder Spieler, der auf den Bauserver gesendet werden soll, bereits entweder Member oder Owner ist. Dazu ist der Owner selbst kein Member (d.h. null!).

Ich glaube die Logik dieser Funktion ist falsch. Es kann an dieser Stelle davon ausgegangen werden, dass jeder Spieler, der auf den Bauserver gesendet werden soll, bereits entweder Member oder Owner ist. Dazu ist der Owner selbst kein Member (d.h. null!).
YoyoNow markierte diese Unterhaltung als gelöst
@ -132,3 +134,2 @@
if(subserver.getType() == Servertype.BAUSERVER && ((Bauserver)subserver).getOwner().equals(owner)) {
for(ProxiedPlayer p : playersToSend)
SubserverSystem.sendPlayer(subserver, p);
if (!subserver.getServer().getPlayers().isEmpty() || atLeastOneSupervisor) {
Besitzer

Auch hier musst du im Else-Fall jedem Spieler mitteilen, dass sie das nicht dürfen (wegen fehlendem Supervisor).

Auch hier musst du im Else-Fall jedem Spieler mitteilen, dass sie das nicht dürfen (wegen fehlendem Supervisor).
YoyoNow markierte diese Unterhaltung als gelöst
@ -234,2 +229,2 @@
BAU_MEMBER_TOGGLE_WORLD_EDIT = use WorldEdit
BAU_MEMBER_TOGGLE_WORLD = change Settings
BAU_MEMBER_SET_USAGE=§8/§7build {0} §8[§eplayer§8]
BAU_MEMBER_SET_TARGET=§aYou are now §a{1}§e on the world of §e{0}.
Besitzer

Die Farbformatierung scheint mir etwas fragwürdig. (auch beim Punkt hintendran). Evtl. auch das "a" der verschiedenen Typen hier in die Nachricht mitintegrieren, sodass nur noch "spectator", "builder" etc. Farbig (gelb) sind. Ich würde die Nachricht hellgrau statt hellgrün machen (ist schließlich eine Statusänderung, nicht eine positive Bestätigung.

Die Farbformatierung scheint mir etwas fragwürdig. (auch beim Punkt hintendran). Evtl. auch das "a" der verschiedenen Typen hier in die Nachricht mitintegrieren, sodass nur noch "spectator", "builder" etc. Farbig (gelb) sind. Ich würde die Nachricht hellgrau statt hellgrün machen (ist schließlich eine Statusänderung, nicht eine positive Bestätigung.
YoyoNow markierte diese Unterhaltung als gelöst
@ -217,2 +212,2 @@
BAU_MEMBER_TOGGLE_WORLD_EDIT = WorldEdit verwenden
BAU_MEMBER_TOGGLE_WORLD = Einstellungen vornehmen
BAU_MEMBER_SET_USAGE=§8/§7bau {0} §8[§eSpieler§8]
BAU_MEMBER_SET_TARGET=§aDu bist nun {1} auf der Welt von §e{0}§a.
Besitzer

Gleiches hier wie im Englischen.

Gleiches hier wie im Englischen.
YoyoNow markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2024-01-19 16:30:52 +01:00 hinzugefügt
Fix messages
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
443a1b29c9
Fix ServerStarter condition
YoyoNow hat 1 Commit 2024-01-19 16:39:10 +01:00 hinzugefügt
Merge branch 'master' into PermissionUpdate
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
760cf4b3b1
# Conflicts:
#	src/de/steamwar/bungeecore/ServerStarter.java
YoyoNow hat 1 Commit 2024-01-19 16:44:57 +01:00 hinzugefügt
Revert ServerStarter.builder startCondition
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
9688c38954
YoyoNow hat 1 Commit 2024-01-19 16:47:58 +01:00 hinzugefügt
Revert ServerStarter.builder startCondition
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
04ebed15d3
Fix CommonCore
Lixfel hat die Änderungen 2024-01-19 16:49:23 +01:00 genehmigt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Zeitgleich mit BauSystem-Merge.

Zeitgleich mit BauSystem-Merge.
Lixfel hat den Titel von WIP: Update BauMemberPermission zu Update BauMemberPermission 2024-02-17 08:14:15 +01:00 geändert
Lixfel hat Commit b3efb223af in master 2024-02-17 08:32:17 +01:00 gemerged
Lixfel löschte die Branch PermissionUpdate 2024-02-17 08:32:18 +01:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.