SteamWar/BauSystem2.0
SteamWar/BauSystem2.0
11
0
Fork 0
Code Issues 32 Pull-Requests Projekte Releases Wiki Aktivität

NoSchemSharing bug fix #180

Geschlossen
xSpw möchte 3 Commits von NoSchemSharingBugFix nach master mergen
pullen von: NoSchemSharingBugFix
Ziel: SteamWar:master
Diskussion 3 Commits 3 Geänderte Dateien 3 +8
xSpw hat 2023-05-30 01:47:11 +02:00 kommentiert
Mitglied
Link kopieren

https://discord.com/channels/690530484920385586/1047226353465966693/1047226638183698442

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
Details
b7a27928d8
Lixfel hat 2023-05-31 12:50:44 +02:00 Änderungen angefragt
Lixfel hat einen Kommentar hinterlassen
Besitzer
Link kopieren

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.
BauSystem_Main/src/BauSystem.properties Veraltet
@ -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
Lixfel hat 2023-05-31 12:49:34 +02:00 kommentiert
Besitzer
Link kopieren

& 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"
YoyoNow hat 2023-05-31 12:55:09 +02:00 kommentiert
Besitzer
Link kopieren

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.
BauSystem_Main/src/BauSystem_de.properties Veraltet
@ -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
Lixfel hat 2023-05-31 12:50:10 +02:00 kommentiert
Besitzer
Link kopieren

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

Orthographie. "aktuel", "bann". Und auch hier klingt NoSchemSharing nicht gerade toll.
BauSystem_Main/src/de/steamwar/bausystem/features/region/TestblockCommand.java
@ -85,6 +86,10 @@ public class TestblockCommand extends SWCommand {
return;
}
if(Punishment.isPunished(SteamwarUser.get(p.getUniqueId()), Punishment.PunishmentType.NoSchemSharing, punishment ->
Lixfel hat 2023-05-31 12:48:35 +02:00 kommentiert
Besitzer
Link kopieren

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?
YoyoNow hat 2023-05-31 12:55:24 +02:00 kommentiert
Besitzer
Link kopieren

Hast du recht auf dem eigenen bau sollte es noch gehen

Hast du recht auf dem eigenen bau sollte es noch gehen
xSpw hat 2023-06-10 18:54:24 +02:00 kommentiert
Autor
Mitglied
Link kopieren

Aber wieso?

Aber wieso?
Lixfel hat 2023-06-10 19:02:56 +02:00 kommentiert
Besitzer
Link kopieren

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.
YoyoNow hat 2023-05-31 12:55:45 +02:00 kommentiert
Besitzer
Link kopieren

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
Details
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
Details
9b10d3d870
YoyoNow hat 2023-07-26 12:47:40 +02:00 kommentiert
Besitzer
Link kopieren

@zOnlyKroks /reset braucht die Sache auch noch.

@zOnlyKroks /reset braucht die Sache auch noch.
zOnlyKroks hat 2023-07-26 13:50:50 +02:00 kommentiert
Mitglied
Link kopieren

@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
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
Erforderlich
Details

Pull-Request geschlossen

Bitte diesen Pull-Request wieder öffnen, um zu mergen.
Anmelden, um an der Diskussion teilzunehmen.
Reviewer
Keine Reviewer
Lixfel
Label
Label entfernen   
1.15

   
1.19

   
1.20

   
Script-API
  
Bug

Etwas funktioniert nicht oder ist fehlerhaft.

  
Codeverbesserung

Ein Codeverbesserungsvorschlag

  
Einsteiger Freundlich

Ist für Leute welche noch nicht so eingearbeitet sind ein guter Start.

  
Idee

Bisher unausgereifte Idee für ein neues Feature

  
In Arbeit

Naja hier dran wird gearbeitet

  
Neues Feature

Das ist ein neues Feature

  
Prio A

Oberste Priorität

  
Security Breach

Die Sicherheit der User ist beeinträchtigt!

  
Überprüfung notwendig

Naja hier muss noch was überprüft werden?

  
Verbesserung

Ein Verbesserungsvorschlag

  
Zu Beobachten

Sollte gefixt sein bzw. kann bislang nicht bestätigt werden

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
Meilenstein
Meilenstein entfernen
Keine Einträge
Kein Meilenstein
Projekte
Projekte löschen
Kein Projekt
Zuständig
Zuständige entfernen
Niemand zuständig
4 Beteiligte
Nachrichten
Fällig am
Das Fälligkeitsdatum ist ungültig oder außerhalb des zulässigen Bereichs. Bitte verwende das Format „jjjj-mm-tt“.

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: SteamWar/BauSystem2.0#180
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?