CleanUp (removed setinventory api method

src/main/java/fr/xephi/authme/Utils.java

@@ -160,7 +160,6 @@ public class Utils {
     }
 
     // TODO: remove if not needed
-    @SuppressWarnings("unused")
     public static void hasPermOnJoin(Player player) {
         if (plugin.permission == null)
             return;

src/main/java/fr/xephi/authme/api/NewAPI.java

@@ -6,11 +6,9 @@ import org.bukkit.Bukkit;
 import org.bukkit.Location;
 import org.bukkit.Server;
 import org.bukkit.entity.Player;
-import org.bukkit.inventory.ItemStack;
 import org.bukkit.plugin.Plugin;
 
 import fr.xephi.authme.AuthMe;
-import fr.xephi.authme.ConsoleLogger;
 import fr.xephi.authme.Utils;
 import fr.xephi.authme.cache.auth.PlayerAuth;
 import fr.xephi.authme.cache.auth.PlayerCache;
@@ -94,16 +92,6 @@ public class NewAPI {
         }
     }
 
-    public void setPlayerInventory(Player player, ItemStack[] content,
-            ItemStack[] armor) {
-        try {
-            player.getInventory().setContents(content);
-            player.getInventory().setArmorContents(armor);
-        } catch (Exception npe) {
-            ConsoleLogger.showError("Some error appear while trying to set inventory for " + player.getName());
-        }
-    }
-
     /**
      * 
      * @param playerName

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

@@ -1,7 +1,6 @@
 package fr.xephi.authme.events;
 
 import org.bukkit.entity.Player;
-import org.bukkit.inventory.ItemStack;
 
 /**
  * This event restore the inventory.
@@ -10,37 +9,15 @@ import org.bukkit.inventory.ItemStack;
  */
 public class RestoreInventoryEvent extends CustomEvent {
 
-    private ItemStack[] inventory;
-    private ItemStack[] armor;
     private Player player;
 
     public RestoreInventoryEvent(Player player) {
         this.player = player;
-        this.inventory = player.getInventory().getContents();
-        this.armor = player.getInventory().getArmorContents();
     }
 
     public RestoreInventoryEvent(Player player, boolean async) {
         super(async);
         this.player = player;
-        this.inventory = inventory;
-        this.armor = armor;
-    }
-
-    public ItemStack[] getInventory() {
-        return this.inventory;
-    }
-
-    public void setInventory(ItemStack[] inventory) {
-        this.inventory = inventory;
-    }
-
-    public ItemStack[] getArmor() {
-        return this.armor;
-    }
-
-    public void setArmor(ItemStack[] armor) {
-        this.armor = armor;
     }
 
     public Player getPlayer() {

src/main/java/fr/xephi/authme/process/login/ProcessSyncronousPlayerLogin.java

@@ -95,7 +95,6 @@ public class ProcessSyncronousPlayerLogin implements Runnable {
         RestoreInventoryEvent event = new RestoreInventoryEvent(player);
         Bukkit.getServer().getPluginManager().callEvent(event);
         if (!event.isCancelled()) {
-            plugin.api.setPlayerInventory(player, event.getInventory(), event.getArmor());
             plugin.inventoryProtector.sendInventoryPacket(player);
         }
     }

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

@@ -4,9 +4,7 @@ import org.bukkit.GameMode;
 import org.bukkit.entity.Player;
 
 import fr.xephi.authme.AuthMe;
-import fr.xephi.authme.events.RestoreInventoryEvent;
 import fr.xephi.authme.settings.Settings;
-import org.bukkit.Bukkit;
 
 public class ProcessSyncronousPlayerQuit implements Runnable {
 
@@ -28,11 +26,6 @@ public class ProcessSyncronousPlayerQuit implements Runnable {
 
     @Override
     public void run() {
-        RestoreInventoryEvent ev = new RestoreInventoryEvent(player);
-        Bukkit.getPluginManager().callEvent(ev);
-        if (!ev.isCancelled()) {
-            plugin.api.setPlayerInventory(player, ev.getInventory(), ev.getArmor());
-        }
 
         if (needToChange) {
             player.setOp(isOp);

src/main/java/fr/xephi/authme/process/register/ProcessSyncronousPasswordRegister.java

@@ -95,9 +95,7 @@ public class ProcessSyncronousPasswordRegister implements Runnable {
             if (Settings.protectInventoryBeforeLogInEnabled && plugin.inventoryProtector != null) {
                 RestoreInventoryEvent event = new RestoreInventoryEvent(player);
                 Bukkit.getPluginManager().callEvent(event);
-                if (!event.isCancelled() && event.getArmor() != null && event.getInventory() != null) {
-                    player.getInventory().setContents(event.getInventory());
-                    player.getInventory().setArmorContents(event.getArmor());
+                if (!event.isCancelled()) {
                     plugin.inventoryProtector.sendInventoryPacket(player);
                 }
             }