diff --git a/EssentialsSpawn/src/com/earth2me/essentials/spawn/SpawnStorage.java b/EssentialsSpawn/src/com/earth2me/essentials/spawn/SpawnStorage.java index c8e12563c..36b6f9662 100644 --- a/EssentialsSpawn/src/com/earth2me/essentials/spawn/SpawnStorage.java +++ b/EssentialsSpawn/src/com/earth2me/essentials/spawn/SpawnStorage.java @@ -8,29 +8,18 @@ import java.io.File; import java.util.HashMap; import java.util.Locale; import java.util.Map; -import java.util.concurrent.ConcurrentHashMap; -import java.util.concurrent.ConcurrentMap; import org.bukkit.Location; import org.bukkit.World; public class SpawnStorage extends AsyncStorageObjectHolder implements IEssentialsModule { - private ConcurrentMap spawns; - public SpawnStorage(final IEssentials ess) { super(ess, Spawns.class); reloadConfig(); } - @Override - public final void reloadConfig() - { - super.reloadConfig(); - spawns = new ConcurrentHashMap(); - } - @Override public File getStorageFile() { @@ -63,8 +52,6 @@ public class SpawnStorage extends AsyncStorageObjectHolder implements IE unlock(); } - spawns.clear(); - if ("default".equalsIgnoreCase(group)) { loc.getWorld().setSpawnLocation(loc.getBlockX(), loc.getBlockY(), loc.getBlockZ()); @@ -73,43 +60,29 @@ public class SpawnStorage extends AsyncStorageObjectHolder implements IE public Location getSpawn(final String group) { - Location spawnLocation; - if (spawns.containsKey(group)) + acquireReadLock(); + try { - spawnLocation = spawns.get(group); + if (getData().getSpawns() == null || group == null) + { + return getWorldSpawn(); + } + final Map spawnMap = getData().getSpawns(); + String groupName = group.toLowerCase(Locale.ENGLISH); + if (!spawnMap.containsKey(groupName)) + { + groupName = "default"; + } + if (!spawnMap.containsKey(groupName)) + { + return getWorldSpawn(); + } + return spawnMap.get(groupName); } - else + finally { - acquireReadLock(); - try - { - if (getData().getSpawns() == null || group == null) - { - return getWorldSpawn(); - } - final Map spawnMap = getData().getSpawns(); - String groupName = group.toLowerCase(Locale.ENGLISH); - if (!spawnMap.containsKey(groupName)) - { - groupName = "default"; - } - if (!spawnMap.containsKey(groupName)) - { - spawnLocation = getWorldSpawn(); - } - else - { - spawnLocation = spawnMap.get(groupName); - } - } - finally - { - unlock(); - } - - spawns.put(group, spawnLocation); + unlock(); } - return spawnLocation; } private Location getWorldSpawn()