SteamWar/BauSystem
SteamWar/BauSystem
Archiviert
13
0
Fork 0
Code Issues Pull-Requests Releases Wiki Aktivität

Fix Region with null testblock #162

Manuell gemergt
YoyoNow hat 2 Commits von FixTestblockNull nach master 2021-01-20 11:07:52 +01:00 zusammengeführt
Diskussion 0 Commits 2 Geänderte Dateien 2 +10 -2
2 geänderte Dateien mit 10 neuen und 2 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen

2
BauSystem_Main/src/de/steamwar/bausystem/commands/CommandTestblock.java
Datei anzeigen

@ -46,7 +46,7 @@ public class CommandTestblock implements CommandExecutor {
} }
for(Region region : Region.getRegions()){ for(Region region : Region.getRegions()){
if(region.inRegion(player.getLocation())){ if(region.inRegion(player.getLocation()) && region.hasTestblock()){
try { try {
Lixfel hat 2021-01-20 10:53:53 +01:00 kommentiert
Veraltet
Review
Link kopieren

Würde eher empfehlen, das if mit dem obrigen zu mergen: Man ist dann einfach in keiner passenden Region. Könnte ja sein, dass sich da mehrere Regionen überlappen, wobei die andere Region dann den TB hat.

Würde eher empfehlen, das if mit dem obrigen zu mergen: Man ist dann einfach in keiner passenden Region. Könnte ja sein, dass sich da mehrere Regionen überlappen, wobei die andere Region dann den TB hat.
YoyoNow hat 2021-01-20 11:05:43 +01:00 kommentiert
Review
Link kopieren

Ok sollte so nun behoben sein.

Ok sollte so nun behoben sein.
Schematic schem = null; Schematic schem = null;
if(args.length > 0){ if(args.length > 0){

10
BauSystem_Main/src/de/steamwar/bausystem/world/Region.java
Datei anzeigen

@ -88,6 +88,10 @@ public class Region {
prototype.reset(this, schem); prototype.reset(this, schem);
} }
public boolean hasTestblock() {
return prototype.hasTestblock();
}
public void resetTestblock(Schematic schem) throws IOException, NoClipboardException{ public void resetTestblock(Schematic schem) throws IOException, NoClipboardException{
prototype.resetTestblock(this, schem); prototype.resetTestblock(this, schem);
} }
@ -167,7 +171,7 @@ public class Region {
paste(schem.load(), x, y, z, rotate); paste(schem.load(), x, y, z, rotate);
} }
public boolean hasProtection(){ public boolean hasProtection() {
return protectSchematic != null; return protectSchematic != null;
} }
@ -181,6 +185,10 @@ public class Region {
paste(schem.load(), x, y, z, rotate); paste(schem.load(), x, y, z, rotate);
} }
public boolean hasTestblock() {
return testblock != null;
}
public void resetTestblock(Region region, Schematic schem) throws IOException, NoClipboardException { public void resetTestblock(Region region, Schematic schem) throws IOException, NoClipboardException {
testblock.reset(region, schem); testblock.reset(region, schem);
} }