Add CommandTNT with extension explode message #189
Keine Reviewer
Label
Kein Label
Bug
Codeverbesserung
Einsteiger Freundlich
Idee
In Arbeit
Neues Feature
Prio A
Security Breach
Überprüfung notwendig
Verbesserung
Zu Beobachten
Kein Meilenstein
Niemand zuständig
2 Beteiligte
Kein Fälligkeitsdatum gesetzt.
Abhängigkeiten
Keine Abhängigkeiten gesetzt.
Referenz: SteamWar/BauSystem#189
Laden…
In neuem Issue referenzieren
Einen Benutzer sperren
Keine Beschreibung angegeben.
Branch "ExtensionTNT" 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?
Config passt schon.
@ -75,6 +75,10 @@ public class CommandTNT implements CommandExecutor, Listener {return "§cEine Explosion hätte Blöcke im Baubereich zerstört";}private String getDamageExtensionMessage() {Ich glaube nicht, dass es dafür eine Extra-Methode (mehr) braucht, aber ich weiß, dass du das schon weiter oben so gemacht hattest.
@ -289,2 +288,4 @@}public boolean inRegionExtension(Region region, Location l) {return inRange(l.getX(), region.minX + offsetX - extensionAxisX + 1, sizeX + extensionAxisX * 2 - 1) &&Das +1-1-Zeugs schaut für mich so ein bisschen Suspicios aus (müsste mal man die Ursache für diesen One-Off-Fehler bestimmen).
wegen deinem <= und + und so von inRange()
Dann sollte man das ggf. irgendwann mal auf ein < ändern, aber das muss nicht in diesem MR passieren
Ja klar. Mache ich aber werde erstmal reingucken, ob das nichts irgendwas anderes kaputt macht.