SteamWar/BauSystem2.0
SteamWar/BauSystem2.0
Archiviert
12
0
Fork 0
Code Issues 39 Pull-Requests Projekte Releases Wiki Aktivität

No Schem Sharing & No Schem Receiving #196

Geschlossen
zOnlyKroks möchte 1 Commits von noschem_fix nach master mergen
pullen von: noschem_fix
Ziel: SteamWar:master
Diskussion 0 Commits 1 Geänderte Dateien 2 +6 -10
2 geänderte Dateien mit 6 neuen und 10 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen

8
BauSystem_Main/src/de/steamwar/bausystem/features/region/ResetCommand.java
Datei anzeigen

@ -67,11 +67,9 @@ public class ResetCommand extends SWCommand {
Region region = regionCheck(p); Region region = regionCheck(p);
if (region == null) return; if (region == null) return;
if (bauServer.getOwner() != p.getUniqueId()) { if (Punishment.isPunished(SteamwarUser.get(p.getUniqueId()), Punishment.PunishmentType.NoSchemSharing, punishment -> BauSystem.MESSAGE.parse("REGION_TB_NO_SCHEMSHARING", p, punishment.getEndTime()))) {
Lixfel hat 2023-07-29 16:59:36 +02:00 kommentiert
Review
Link kopieren

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

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

8
BauSystem_Main/src/de/steamwar/bausystem/features/region/TestblockCommand.java
Datei anzeigen

@ -92,11 +92,9 @@ public class TestblockCommand extends SWCommand {
return; return;
} }
if (bauServer.getOwner() != p.getUniqueId()) { if (Punishment.isPunished(SteamwarUser.get(p.getUniqueId()), Punishment.PunishmentType.NoSchemSharing, punishment -> BauSystem.MESSAGE.parse("REGION_TB_NO_SCHEMSHARING", p, punishment.getEndTime()))) {
Lixfel hat 2023-07-29 16:58:39 +02:00 kommentiert
Review
Link kopieren

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.
if (Punishment.isPunished(SteamwarUser.get(p.getUniqueId()), Punishment.PunishmentType.NoSchemSharing, punishment -> BauSystem.MESSAGE.parse("REGION_TB_NO_SCHEMSHARING", p, punishment.getEndTime()))) { return;
return; } else if (p.getUniqueId() != bauServer.getOwner() && Punishment.isPunished(SteamwarUser.get(bauServer.getOwner()), Punishment.PunishmentType.NoSchemReceiving, punishment -> BauSystem.MESSAGE.parse("REGION_TB_NO_SCHEMRECEIVING", p, punishment.getEndTime()))) {
}
} else if (Punishment.isPunished(SteamwarUser.get(bauServer.getOwner()), Punishment.PunishmentType.NoSchemReceiving, punishment -> BauSystem.MESSAGE.parse("REGION_TB_NO_SCHEMRECEIVING", p, punishment.getEndTime()))) {
return; return;
} }