diff --git a/src/main/java/com/drtshock/playervaults/Listeners.java b/src/main/java/com/drtshock/playervaults/Listeners.java index 4cfec11..14de215 100644 --- a/src/main/java/com/drtshock/playervaults/Listeners.java +++ b/src/main/java/com/drtshock/playervaults/Listeners.java @@ -2,7 +2,6 @@ package com.drtshock.playervaults; import java.io.IOException; -import org.bukkit.Bukkit; import org.bukkit.ChatColor; import org.bukkit.Location; import org.bukkit.Material; @@ -24,7 +23,6 @@ import org.bukkit.event.player.PlayerInteractEvent; import org.bukkit.event.player.PlayerJoinEvent; import org.bukkit.event.player.PlayerQuitEvent; import org.bukkit.event.player.PlayerTeleportEvent; -import org.bukkit.event.server.PluginDisableEvent; import org.bukkit.inventory.Inventory; import com.drtshock.playervaults.commands.VaultViewInfo; @@ -45,7 +43,7 @@ public class Listeners implements Listener { /** * Save a players vault. Sends to method in VaultManager class. * - * @param Player p + * @param Player The player of whose vault to save */ public void saveVault(Player p) { if (PlayerVaults.IN_VAULT.containsKey(p.getName())) { @@ -62,21 +60,6 @@ public class Listeners implements Listener { } } - /** - * Save vaults when the plugin disables. Fixes a dupe on server shutdown - * with open vaults. - * - * @param event - */ - @EventHandler - public void onDisableEvent(PluginDisableEvent event) { - for(Player p:Bukkit.getOnlinePlayers()) { - if (PlayerVaults.IN_VAULT.containsKey(p.getName())) { - p.closeInventory(); - } - } - } - @EventHandler public void onTeleport(PlayerTeleportEvent event) { saveVault(event.getPlayer()); @@ -118,8 +101,6 @@ public class Listeners implements Listener { /** * Check if a player is trying to do something while in a vault. Don't let * them open up another chest. - * - * @param PlayerInteractEvent */ @EventHandler public void onInteract(PlayerInteractEvent event) { diff --git a/src/main/java/com/drtshock/playervaults/PlayerVaults.java b/src/main/java/com/drtshock/playervaults/PlayerVaults.java index db7ea53..055e17f 100644 --- a/src/main/java/com/drtshock/playervaults/PlayerVaults.java +++ b/src/main/java/com/drtshock/playervaults/PlayerVaults.java @@ -9,7 +9,9 @@ import java.util.logging.Logger; import net.milkbowl.vault.economy.Economy; +import org.bukkit.Bukkit; import org.bukkit.configuration.file.YamlConfiguration; +import org.bukkit.entity.Player; import org.bukkit.inventory.Inventory; import org.bukkit.plugin.RegisteredServiceProvider; import org.bukkit.plugin.java.JavaPlugin; @@ -25,11 +27,11 @@ import com.drtshock.playervaults.util.VaultManager; public class PlayerVaults extends JavaPlugin { public static PlayerVaults PLUGIN; - public Logger log; + public static Logger log; public static boolean UPDATE = false; public static String NEWVERSION = ""; public static String LINK = ""; - Commands commands; + public static Commands commands; public static HashMap SET_SIGN = new HashMap(); public static HashMap IN_VAULT = new HashMap(); public static HashMap OPENINVENTORIES = new HashMap(); @@ -43,12 +45,13 @@ public class PlayerVaults extends JavaPlugin { public static File SIGNS_FILE; public static String DIRECTORY = "plugins" + File.separator + "PlayerVaults" + File.separator + "vaults"; public static VaultManager VM; + public static Listeners listener; @Override public void onEnable() { loadLang(); log = getServer().getLogger(); - getServer().getPluginManager().registerEvents(new Listeners(this), this); + getServer().getPluginManager().registerEvents(listener = new Listeners(this), this); loadConfig(); loadSigns(); startMetrics(); @@ -81,6 +84,15 @@ public class PlayerVaults extends JavaPlugin { VM = new VaultManager(this); } + @Override + public void onDisable() { + for(Player p:Bukkit.getOnlinePlayers()) { + if (IN_VAULT.containsKey(p.getName())) { + p.closeInventory(); + } + } + } + /** * Start metrics */