NoSchemSharing bug fix #180
Keine Reviewer
Label
Kein Label
1.15
1.19
1.20
Script-API
Bug
Codeverbesserung
Einsteiger Freundlich
Idee
In Arbeit
Neues Feature
Prio A
Security Breach
Überprüfung notwendig
Verbesserung
Zu Beobachten
Kein Meilenstein
Kein Projekt
Niemand zuständig
4 Beteiligte
Fällig am
Kein Fälligkeitsdatum gesetzt.
Abhängigkeiten
Keine Abhängigkeiten gesetzt.
Referenz: SteamWar/BauSystem2.0#180
Laden…
In neuem Issue referenzieren
Einen Benutzer sperren
Keine Beschreibung angegeben.
Branch "NoSchemSharingBugFix" 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?
https://discord.com/channels/690530484920385586/1047226353465966693/1047226638183698442
Ich weiß nicht, ob es mit /reset nicht noch einen anderen Fall gibt, in dem das hier auftreten kann.
@ -1118,6 +1118,7 @@ REGION_TB_DONE=§7Dummy reset
REGION_TB_ERROR=§cError resetting the dummy
REGION_TB_NO_PERMS=§cYou are not allowed to reset the dummy here
REGION_TB_NO_REGION=§cYou are currently not in any region
REGION_TB_NOSCHEMSHARING=&cYou currently have a NoSchemSharing ban
& ist falsch, es muss ein § sein. "NoSchemSharing ban" klingt jetzt auch nicht so gut, evtl. eher "You currently cannot share schems"
Könnte man so aus dem SpigotCore glaube kopieren und dann halt anpassen, weil keine Zeit drin steht oder wir lassen es das Punishment System selber senden, also plus Zeit uns alles, dann brauchen wir hier glaube die Nachrichten nicht. Dafür sollte es Methoden im SpigotCore geben.
@ -1086,6 +1086,7 @@ REGION_TB_DONE=§7Testblock zurückgesetzt
REGION_TB_ERROR=§cFehler beim Zurücksetzen des Testblocks
REGION_TB_NO_PERMS=§cDu darfst hier nicht den Testblock zurücksetzen
REGION_TB_NO_REGION=§cDu befindest dich derzeit in keiner Region
REGION_TB_NOSCHEMSHARING=§cDu hast aktuel einen NoSchemSharing bann
Orthographie. "aktuel", "bann". Und auch hier klingt NoSchemSharing nicht gerade toll.
@ -85,6 +86,10 @@ public class TestblockCommand extends SWCommand {
return;
}
if(Punishment.isPunished(SteamwarUser.get(p.getUniqueId()), Punishment.PunishmentType.NoSchemSharing, punishment ->
Ich glaube, das sollte nur greifen, wenn der Spieler nicht auf seinem eigenen Bau ist?
Hast du recht auf dem eigenen bau sollte es noch gehen
Aber wieso?
Weil es bei der Bestrafung darum geht, dass du keine Schematics mit anderen teilen kannst und nicht darum, dass du keine Schematics mehr verwenden kannst.
Ja auch /reset ist hiervon betroffen
@zOnlyKroks /reset braucht die Sache auch noch.
mache das auf einem anderen branch und auch pr, das hier kann also geschlossen werden