Change some custom event to let them be async

src/main/java/fr/xephi/authme/commands/AdminCommand.java

@@ -47,8 +47,8 @@ public class AdminCommand implements CommandExecutor {
     }
 
     @Override
-    public boolean onCommand(CommandSender sender, Command cmnd, String label,
+    public boolean onCommand(final CommandSender sender, Command cmnd,
-            String[] args) {
+            String label, String[] args) {
         if (args.length == 0) {
             sender.sendMessage("Usage:");
             sender.sendMessage("/authme reload - Reload the config");
@@ -434,34 +434,43 @@ public class AdminCommand implements CommandExecutor {
                     return true;
                 }
             }
-            try {
+            final String name = args[1].toLowerCase();
-                String name = args[1].toLowerCase();
+            final String raw = args[2];
-                String hash = PasswordSecurity.getHash(Settings.getPasswordHash, args[2], name);
+            Bukkit.getScheduler().runTaskAsynchronously(plugin, new Runnable() {
-                PlayerAuth auth = null;
+
-                if (PlayerCache.getInstance().isAuthenticated(name)) {
+                @Override
-                    auth = PlayerCache.getInstance().getAuth(name);
+                public void run() {
-                } else if (plugin.database.isAuthAvailable(name)) {
+                    String hash;
-                    auth = plugin.database.getAuth(name);
+                    try {
+                        hash = PasswordSecurity.getHash(Settings.getPasswordHash, raw, name);
+                    } catch (NoSuchAlgorithmException e) {
+                        m.send(sender, "error");
+                        return;
+                    }
+                    PlayerAuth auth = null;
+                    if (PlayerCache.getInstance().isAuthenticated(name)) {
+                        auth = PlayerCache.getInstance().getAuth(name);
+                    } else if (plugin.database.isAuthAvailable(name)) {
+                        auth = plugin.database.getAuth(name);
+                    }
+                    if (auth == null) {
+                        m.send(sender, "unknown_user");
+                        return;
+                    }
+                    auth.setHash(hash);
+                    if (PasswordSecurity.userSalt.containsKey(name)) {
+                        auth.setSalt(PasswordSecurity.userSalt.get(name));
+                        plugin.database.updateSalt(auth);
+                    }
+                    if (!plugin.database.updatePassword(auth)) {
+                        m.send(sender, "error");
+                        return;
+                    }
+                    sender.sendMessage("pwd_changed");
+                    ConsoleLogger.info(name + "'s password changed");
                 }
-                if (auth == null) {
+
-                    m.send(sender, "unknown_user");
+            });
-                    return true;
-                }
-                auth.setHash(hash);
-                if (PasswordSecurity.userSalt.containsKey(name)) {
-                    auth.setSalt(PasswordSecurity.userSalt.get(name));
-                    plugin.database.updateSalt(auth);
-                }
-                if (!plugin.database.updatePassword(auth)) {
-                    m.send(sender, "error");
-                    return true;
-                }
-                sender.sendMessage("pwd_changed");
-                ConsoleLogger.info(args[1] + "'s password changed");
-            } catch (NoSuchAlgorithmException ex) {
-                ConsoleLogger.showError(ex.getMessage());
-                m.send(sender, "error");
-            }
             return true;
         } else if (args[0].equalsIgnoreCase("unregister") || args[0].equalsIgnoreCase("unreg") || args[0].equalsIgnoreCase("del")) {
             if (args.length != 2) {

src/main/java/fr/xephi/authme/events/CustomEvent.java

@@ -13,6 +13,14 @@ public class CustomEvent extends Event implements Cancellable {
     private boolean isCancelled;
     private static final HandlerList handlers = new HandlerList();
 
+    public CustomEvent() {
+        super(false);
+    }
+
+    public CustomEvent(boolean b) {
+        super(b);
+    }
+
     public HandlerList getHandlers() {
         return handlers;
     }

src/main/java/fr/xephi/authme/events/FirstSpawnTeleportEvent.java

@@ -16,6 +16,7 @@ public class FirstSpawnTeleportEvent extends CustomEvent {
     private Location from;
 
     public FirstSpawnTeleportEvent(Player player, Location from, Location to) {
+        super(true);
         this.player = player;
         this.from = from;
         this.to = to;

src/main/java/fr/xephi/authme/events/PasswordEncryptionEvent.java

@@ -22,6 +22,7 @@ public class PasswordEncryptionEvent extends Event {
     private String playerName = "";
 
     public PasswordEncryptionEvent(EncryptionMethod method, String playerName) {
+        super(true);
         this.method = method;
         this.playerName = playerName;
     }

src/main/java/fr/xephi/authme/events/ProtectInventoryEvent.java

@@ -20,6 +20,7 @@ public class ProtectInventoryEvent extends CustomEvent {
 
     public ProtectInventoryEvent(Player player, ItemStack[] storedinventory,
             ItemStack[] storedarmor) {
+        super(true);
         this.player = player;
         this.storedinventory = storedinventory;
         this.storedarmor = storedarmor;

src/main/java/fr/xephi/authme/events/ResetInventoryEvent.java

@@ -13,6 +13,7 @@ public class ResetInventoryEvent extends CustomEvent {
     private Player player;
 
     public ResetInventoryEvent(Player player) {
+        super(true);
         this.player = player;
     }
 

src/main/java/fr/xephi/authme/events/RestoreInventoryEvent.java

@@ -22,6 +22,14 @@ public class RestoreInventoryEvent extends CustomEvent {
         this.armor = armor;
     }
 
+    public RestoreInventoryEvent(Player player, ItemStack[] inventory,
+            ItemStack[] armor, boolean b) {
+        super(b);
+        this.player = player;
+        this.inventory = inventory;
+        this.armor = armor;
+    }
+
     public ItemStack[] getInventory() {
         return this.inventory;
     }

src/main/java/fr/xephi/authme/events/StoreInventoryEvent.java

@@ -28,8 +28,7 @@ public class StoreInventoryEvent extends CustomEvent {
         try {
             this.inventory = fileCache.readCache(player).getInventory();
             this.armor = fileCache.readCache(player).getArmour();
-        } catch (Exception e)
+        } catch (Exception e) {
-        {
             this.inventory = player.getInventory().getContents();
             this.armor = player.getInventory().getArmorContents();
         }

src/main/java/fr/xephi/authme/process/join/AsyncronousJoin.java

@@ -6,6 +6,7 @@ import org.bukkit.Location;
 import org.bukkit.Material;
 import org.bukkit.block.Block;
 import org.bukkit.entity.Player;
+import org.bukkit.inventory.ItemStack;
 import org.bukkit.potion.PotionEffect;
 import org.bukkit.potion.PotionEffectType;
 import org.bukkit.scheduler.BukkitScheduler;
@@ -176,25 +177,27 @@ public class AsyncronousJoin {
 
         }
         if (Settings.protectInventoryBeforeLogInEnabled) {
-            sched.scheduleSyncDelayedTask(plugin, new Runnable() {
+            try {
+                LimboPlayer limbo = LimboCache.getInstance().getLimboPlayer(player.getName().toLowerCase());
+                ProtectInventoryEvent ev = new ProtectInventoryEvent(player, limbo.getInventory(), limbo.getArmour());
+                plugin.getServer().getPluginManager().callEvent(ev);
+                if (ev.isCancelled()) {
+                    if (!Settings.noConsoleSpam)
+                        ConsoleLogger.info("ProtectInventoryEvent has been cancelled for " + player.getName() + " ...");
+                } else {
+                    final ItemStack[] inv = ev.getEmptyArmor();
+                    final ItemStack[] armor = ev.getEmptyArmor();
+                    sched.scheduleSyncDelayedTask(plugin, new Runnable() {
 
-                @Override
+                        @Override
-                public void run() {
+                        public void run() {
-                    try {
+                            plugin.api.setPlayerInventory(player, inv, armor);
-                        LimboPlayer limbo = LimboCache.getInstance().getLimboPlayer(player.getName().toLowerCase());
-                        ProtectInventoryEvent ev = new ProtectInventoryEvent(player, limbo.getInventory(), limbo.getArmour());
-                        plugin.getServer().getPluginManager().callEvent(ev);
-                        if (ev.isCancelled()) {
-                            if (!Settings.noConsoleSpam)
-                                ConsoleLogger.info("ProtectInventoryEvent has been cancelled for " + player.getName() + " ...");
-                        } else {
-                            plugin.api.setPlayerInventory(player, ev.getEmptyInventory(), ev.getEmptyArmor());
                         }
-                    } catch (NullPointerException ex) {
-                    }
-                }
 
-            });
+                    });
+                }
+            } catch (NullPointerException ex) {
+            }
         }
         String[] msg;
         if (Settings.emailRegistration) {
@@ -262,21 +265,21 @@ public class AsyncronousJoin {
         else {
             if (Spawn.getInstance().getFirstSpawn() == null || Spawn.getInstance().getFirstSpawn().getWorld() == null)
                 return false;
-            final Location loc = Spawn.getInstance().getFirstSpawn();
+            FirstSpawnTeleportEvent tpEvent = new FirstSpawnTeleportEvent(player, player.getLocation(), Spawn.getInstance().getFirstSpawn());
-            Bukkit.getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
+            plugin.getServer().getPluginManager().callEvent(tpEvent);
+            if (!tpEvent.isCancelled()) {
+                if (player.isOnline() && tpEvent.getTo() != null && tpEvent.getTo().getWorld() != null) {
+                    final Location fLoc = tpEvent.getTo();
+                    Bukkit.getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
 
-                @Override
+                        @Override
-                public void run() {
+                        public void run() {
-                    FirstSpawnTeleportEvent tpEvent = new FirstSpawnTeleportEvent(player, player.getLocation(), loc);
+                            player.teleport(fLoc);
-                    plugin.getServer().getPluginManager().callEvent(tpEvent);
-                    if (!tpEvent.isCancelled()) {
-                        if (player.isOnline() && tpEvent.getTo() != null && tpEvent.getTo().getWorld() != null) {
-                            player.teleport(tpEvent.getTo());
                         }
-                    }
-                }
 
-            });
+                    });
+                }
+            }
             return true;
         }
     }

src/main/java/fr/xephi/authme/process/logout/ProcessSyncronousPlayerLogout.java

@@ -56,13 +56,7 @@ public class ProcessSyncronousPlayerLogout implements Runnable {
             player.setFlying(true);
         }
         // Player is now logout... Time to fire event !
-        sched.scheduleSyncDelayedTask(plugin, new Runnable() {
+        Bukkit.getServer().getPluginManager().callEvent(new LogoutEvent(player));
-
-            @Override
-            public void run() {
-                Bukkit.getServer().getPluginManager().callEvent(new LogoutEvent(player));
-            }
-        });
         m.send(player, "logout");
         ConsoleLogger.info(player.getDisplayName() + " logged out");
     }

src/main/java/fr/xephi/authme/process/quit/AsyncronousQuit.java

@@ -13,6 +13,7 @@ import fr.xephi.authme.cache.auth.PlayerCache;
 import fr.xephi.authme.cache.limbo.LimboCache;
 import fr.xephi.authme.cache.limbo.LimboPlayer;
 import fr.xephi.authme.datasource.DataSource;
+import fr.xephi.authme.events.RestoreInventoryEvent;
 import fr.xephi.authme.listener.AuthMePlayerListener;
 import fr.xephi.authme.plugin.manager.CombatTagComunicator;
 import fr.xephi.authme.settings.Settings;
@@ -97,6 +98,15 @@ public class AsyncronousQuit {
         }
         AuthMePlayerListener.gameMode.remove(name);
         final Player p = player;
+        RestoreInventoryEvent ev = new RestoreInventoryEvent(player, inv, armor, true);
+        Bukkit.getPluginManager().callEvent(ev);
+        if (ev.isCancelled()) {
+            inv = null;
+            armor = null;
+        } else {
+            inv = ev.getInventory();
+            armor = ev.getArmor();
+        }
         Bukkit.getScheduler().scheduleSyncDelayedTask(plugin, new ProcessSyncronousPlayerQuit(plugin, p, inv, armor, isOp, isFlying, needToChange));
     }
 }

src/main/java/fr/xephi/authme/process/quit/ProcessSyncronousPlayerQuit.java

@@ -5,7 +5,6 @@ import org.bukkit.entity.Player;
 import org.bukkit.inventory.ItemStack;
 
 import fr.xephi.authme.AuthMe;
-import fr.xephi.authme.events.RestoreInventoryEvent;
 import fr.xephi.authme.settings.Settings;
 
 public class ProcessSyncronousPlayerQuit implements Runnable {
@@ -32,13 +31,8 @@ public class ProcessSyncronousPlayerQuit implements Runnable {
 
     @Override
     public void run() {
-        if (inv != null && armor != null) {
+        if (inv != null && armor != null)
-            RestoreInventoryEvent ev = new RestoreInventoryEvent(player, inv, armor);
+            plugin.api.setPlayerInventory(player, inv, armor);
-            player.getServer().getPluginManager().callEvent(ev);
-            if (!ev.isCancelled()) {
-                plugin.api.setPlayerInventory(player, ev.getInventory(), ev.getArmor());
-            }
-        }
         if (needToChange) {
             player.setOp(isOp);
             if (player.getGameMode() != GameMode.CREATIVE && !Settings.isMovementAllowed) {