No Schem Sharing & No Schem Receiving #196

Geschlossen
zOnlyKroks möchte 1 Commits von noschem_fix nach master mergen
Mitglied

Logic fix

Logic fix
zOnlyKroks hat 1 Commit 2023-07-29 14:58:46 +02:00 hinzugefügt
Fix logic
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
68e73dc7e9
Lixfel hat 2023-07-29 16:59:38 +02:00 Änderungen angefragt
@ -72,3 +70,1 @@
return;
}
} else if (Punishment.isPunished(SteamwarUser.get(bauServer.getOwner()), Punishment.PunishmentType.NoSchemReceiving, punishment -> BauSystem.MESSAGE.parse("REGION_TB_NO_SCHEMRECEIVING", p, punishment.getEndTime()))) {
if (Punishment.isPunished(SteamwarUser.get(p.getUniqueId()), Punishment.PunishmentType.NoSchemSharing, punishment -> BauSystem.MESSAGE.parse("REGION_TB_NO_SCHEMSHARING", p, punishment.getEndTime()))) {
Besitzer

Testblock pasten sollte selbst mit Noschemsharing auf dem eigenen Bau noch gehen.

Testblock pasten sollte selbst mit Noschemsharing auf dem eigenen Bau noch gehen.
@ -97,3 +95,1 @@
return;
}
} else if (Punishment.isPunished(SteamwarUser.get(bauServer.getOwner()), Punishment.PunishmentType.NoSchemReceiving, punishment -> BauSystem.MESSAGE.parse("REGION_TB_NO_SCHEMRECEIVING", p, punishment.getEndTime()))) {
if (Punishment.isPunished(SteamwarUser.get(p.getUniqueId()), Punishment.PunishmentType.NoSchemSharing, punishment -> BauSystem.MESSAGE.parse("REGION_TB_NO_SCHEMSHARING", p, punishment.getEndTime()))) {
Besitzer

Soweit ich das sehe ist die gleiche Logik hier doppelt implementiert. Evtl. wäre es sinnvoll, diese in eine Methode auszulagern.

Soweit ich das sehe ist die gleiche Logik hier doppelt implementiert. Evtl. wäre es sinnvoll, diese in eine Methode auszulagern.
YoyoNow hat diesen Pull-Request 2023-07-30 11:56:33 +02:00 geschlossen
zOnlyKroks löschte die Branch noschem_fix 2023-10-07 15:04:25 +02:00
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Erforderlich
Details

Pull-Request geschlossen

Anmelden, um an der Diskussion teilzunehmen.
Keine Beschreibung angegeben.