forked from SteamWar/SteamWar
Replace RegionDataStore.saveRegion to save
Replace RegionDataStore.loadRegion to load Replace RegionDataStore.deleteRegion to delete
This commit is contained in:
@@ -63,7 +63,7 @@ public interface RegionBackups {
|
||||
|
||||
@SuppressWarnings("java:S3038") // This forces everybody to implement 'deleteRegion' for Backups!
|
||||
@Override
|
||||
public abstract void deleteRegion();
|
||||
public abstract void delete();
|
||||
}
|
||||
|
||||
@CheckReturnValue
|
||||
|
||||
@@ -39,12 +39,12 @@ public abstract class RegionData {
|
||||
protected RegionData(RegionDataStore store) {
|
||||
this.store = store;
|
||||
initialize();
|
||||
store.loadRegion(this);
|
||||
store.load(this);
|
||||
}
|
||||
|
||||
public final void setStore(RegionDataStore store) {
|
||||
this.store = store;
|
||||
store.saveRegion();
|
||||
store.save();
|
||||
}
|
||||
|
||||
protected void initialize() {
|
||||
@@ -66,11 +66,11 @@ public abstract class RegionData {
|
||||
public final <T extends Enum<T> & Flag.Value<T>> boolean set(@NonNull Flag<T> flag, @NonNull T value) {
|
||||
if (has(flag).isWritable()) {
|
||||
boolean needsSave = flagMap.put(flag, value) != value;
|
||||
if (needsSave) store.saveRegion();
|
||||
if (needsSave) store.save();
|
||||
|
||||
connectedRegions().forEach(region -> {
|
||||
if (region.getRegionData().flagMap.put(flag, value) != value) {
|
||||
region.saveRegion();
|
||||
region.save();
|
||||
}
|
||||
});
|
||||
return needsSave;
|
||||
@@ -93,13 +93,13 @@ public abstract class RegionData {
|
||||
}
|
||||
initialize();
|
||||
properties.forEach(property -> property.set(null));
|
||||
store.saveRegion();
|
||||
store.save();
|
||||
|
||||
connectedRegions().forEach(region -> {
|
||||
region.getRegionData().flagMap.keySet().removeAll(remove);
|
||||
region.getRegionData().initialize();
|
||||
region.getRegionData().properties.forEach(property -> property.set(null));
|
||||
region.saveRegion();
|
||||
region.save();
|
||||
});
|
||||
}
|
||||
|
||||
@@ -117,7 +117,7 @@ public abstract class RegionData {
|
||||
|
||||
public final void setTestblockSchematic(SchematicNode schematic) {
|
||||
testblockSchematic.set(schematic);
|
||||
store.saveRegion();
|
||||
store.save();
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -20,8 +20,8 @@
|
||||
package de.steamwar.bausystem.region;
|
||||
|
||||
public interface RegionDataStore {
|
||||
void saveRegion();
|
||||
void loadRegion(RegionData regionData);
|
||||
default void deleteRegion() {
|
||||
void save();
|
||||
void load(RegionData regionData);
|
||||
default void delete() {
|
||||
}
|
||||
}
|
||||
|
||||
+1
-1
@@ -65,7 +65,7 @@ public class DynamicRegionCommand extends SWCommand {
|
||||
public void deleteRegion(Player player) {
|
||||
Region region = DynamicRegionSystem.INSTANCE.get(player.getLocation());
|
||||
if (!region.getType().isDeletable()) return;
|
||||
region.deleteRegion();
|
||||
region.delete();
|
||||
}
|
||||
|
||||
@Mapper("regionType")
|
||||
|
||||
+2
-2
@@ -48,7 +48,7 @@ public abstract class DynamicRegion implements Region {
|
||||
this.minZ = minZ;
|
||||
init();
|
||||
DynamicRegionSystem.INSTANCE.add(this);
|
||||
saveRegion();
|
||||
save();
|
||||
}
|
||||
|
||||
public abstract void init();
|
||||
@@ -95,7 +95,7 @@ public abstract class DynamicRegion implements Region {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void deleteRegion() {
|
||||
public void delete() {
|
||||
if (!getType().isDeletable()) return;
|
||||
DynamicRegionSystem.INSTANCE.remove(this);
|
||||
DynamicRegionRepository.deleteRegion(this);
|
||||
|
||||
+2
-2
@@ -135,12 +135,12 @@ public class GlobalRegion implements Region {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void saveRegion() {
|
||||
public void save() {
|
||||
DynamicRegionRepository.saveRegion(this);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void loadRegion(RegionData regionData) {
|
||||
public void load(RegionData regionData) {
|
||||
DynamicRegionRepository.loadRegionData(this, regionData);
|
||||
}
|
||||
}
|
||||
|
||||
+2
-2
@@ -115,12 +115,12 @@ public class PathRegion extends DynamicRegion {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void saveRegion() {
|
||||
public void save() {
|
||||
DynamicRegionRepository.saveRegion(this);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void loadRegion(RegionData regionData) {
|
||||
public void load(RegionData regionData) {
|
||||
DynamicRegionRepository.loadRegionData(this, regionData);
|
||||
}
|
||||
|
||||
|
||||
+2
-2
@@ -92,12 +92,12 @@ public class DryRegion extends DynamicRegion {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void saveRegion() {
|
||||
public void save() {
|
||||
DynamicRegionRepository.saveRegion(this);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void loadRegion(RegionData regionData) {
|
||||
public void load(RegionData regionData) {
|
||||
DynamicRegionRepository.loadRegionData(this, regionData);
|
||||
}
|
||||
}
|
||||
|
||||
+2
-2
@@ -91,12 +91,12 @@ public class WetRegion extends DynamicRegion {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void saveRegion() {
|
||||
public void save() {
|
||||
DynamicRegionRepository.saveRegion(this);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void loadRegion(RegionData regionData) {
|
||||
public void load(RegionData regionData) {
|
||||
DynamicRegionRepository.loadRegionData(this, regionData);
|
||||
}
|
||||
}
|
||||
|
||||
+2
-2
@@ -142,12 +142,12 @@ public final class FixedGlobalRegion implements Region {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void saveRegion() {
|
||||
public void save() {
|
||||
FixedRegionDataUtils.saveRegionData("global", FLAG_STORAGE);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void loadRegion(RegionData regionData) {
|
||||
public void load(RegionData regionData) {
|
||||
FixedRegionDataUtils.loadRegionData("global", FLAG_STORAGE);
|
||||
}
|
||||
}
|
||||
|
||||
+5
-5
@@ -139,15 +139,15 @@ public class FixedRegion implements Region {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void saveRegion() {
|
||||
public void save() {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void loadRegion(RegionData regionData) {
|
||||
public void load(RegionData regionData) {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void deleteRegion() {
|
||||
public void delete() {
|
||||
file.delete();
|
||||
}
|
||||
}
|
||||
@@ -402,12 +402,12 @@ public class FixedRegion implements Region {
|
||||
}
|
||||
|
||||
@Override
|
||||
public void saveRegion() {
|
||||
public void save() {
|
||||
FixedRegionDataUtils.saveRegionData(name, flagStorage);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void loadRegion(RegionData regionData) {
|
||||
public void load(RegionData regionData) {
|
||||
FixedRegionDataUtils.loadRegionData(name, flagStorage);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user