SteamWar/BauSystem2.0
Archiviert
12
0

NoSchemSharing bug fix #180

Geschlossen
xSpw möchte 3 Commits von NoSchemSharingBugFix nach master mergen
Mitglied
https://discord.com/channels/690530484920385586/1047226353465966693/1047226638183698442
xSpw hat 1 Commit 2023-05-30 01:47:12 +02:00 hinzugefügt
NoSchemSharing bug fix
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
b7a27928d8
Lixfel hat 2023-05-31 12:50:44 +02:00 Änderungen angefragt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Ich weiß nicht, ob es mit /reset nicht noch einen anderen Fall gibt, in dem das hier auftreten kann.

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
Besitzer

& ist falsch, es muss ein § sein. "NoSchemSharing ban" klingt jetzt auch nicht so gut, evtl. eher "You currently cannot share schems"

& ist falsch, es muss ein § sein. "NoSchemSharing ban" klingt jetzt auch nicht so gut, evtl. eher "You currently cannot share schems"
Besitzer

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.

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
Besitzer

Orthographie. "aktuel", "bann". Und auch hier klingt NoSchemSharing nicht gerade toll.

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 ->
Besitzer

Ich glaube, das sollte nur greifen, wenn der Spieler nicht auf seinem eigenen Bau ist?

Ich glaube, das sollte nur greifen, wenn der Spieler nicht auf seinem eigenen Bau ist?
Besitzer

Hast du recht auf dem eigenen bau sollte es noch gehen

Hast du recht auf dem eigenen bau sollte es noch gehen
Autor
Mitglied

Aber wieso?

Aber wieso?
Besitzer

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.

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.
Besitzer

Ich weiß nicht, ob es mit /reset nicht noch einen anderen Fall gibt, in dem das hier auftreten kann.

Ja auch /reset ist hiervon betroffen

> Ich weiß nicht, ob es mit /reset nicht noch einen anderen Fall gibt, in dem das hier auftreten kann. Ja auch /reset ist hiervon betroffen
zOnlyKroks hat 1 Commit 2023-07-25 16:11:20 +02:00 hinzugefügt
Merge branch 'master' into NoSchemSharingBugFix
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
e6e1387b9f
zOnlyKroks hat 1 Commit 2023-07-26 12:13:16 +02:00 hinzugefügt
Merge branch 'master' into NoSchemSharingBugFix
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
9b10d3d870
Besitzer

@zOnlyKroks /reset braucht die Sache auch noch.

@zOnlyKroks /reset braucht die Sache auch noch.
Mitglied

@zOnlyKroks /reset braucht die Sache auch noch.

mache das auf einem anderen branch und auch pr, das hier kann also geschlossen werden

> @zOnlyKroks /reset braucht die Sache auch noch. mache das auf einem anderen branch und auch pr, das hier kann also geschlossen werden
YoyoNow hat diesen Pull-Request 2023-07-26 20:44:05 +02:00 geschlossen
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.