Added a little delay on PlayerFishEventListener

Author: yL3oft

src/main/java/us/thezircon/play/autopickup/listeners/PlayerFishEventListener.java

@@ -2,7 +2,6 @@
 
 import org.bukkit.Bukkit;
 import org.bukkit.Location;
-import org.bukkit.Material;
 import org.bukkit.entity.Item;
 import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
@@ -21,7 +20,6 @@ public class PlayerFishEventListener implements Listener {
     @EventHandler
     public void onFish(PlayerFishEvent e) {
         Player player = e.getPlayer();
-
         if(!(e.getState().equals(PlayerFishEvent.State.CAUGHT_FISH))) return;
         if(!(e.getCaught() instanceof Item)) return;
 
@@ -47,36 +45,38 @@ public void run() {
             return;
         }
 
-        Item caught = (Item) e.getCaught();
+        Bukkit.getScheduler().runTask(PLUGIN, () -> {
+            Item caught = (Item) e.getCaught();
 
-        if (PLUGIN.getBlacklistConf().contains("BlacklistedFishing", true)) {
-            boolean doBlacklist = PLUGIN.getBlacklistConf().getBoolean("BlacklistedFishing");
-            List<String> blacklist = PLUGIN.getBlacklistConf().getStringList("BlacklistedFishing");
+            if (PLUGIN.getBlacklistConf().contains("BlacklistedFishing", true)) {
+                boolean doBlacklist = PLUGIN.getBlacklistConf().getBoolean("BlacklistedFishing");
+                List<String> blacklist = PLUGIN.getBlacklistConf().getStringList("BlacklistedFishing");
 
-            if (doBlacklist && blacklist.contains(caught.getItemStack().getType().toString())) {
-                return;
+                if (doBlacklist && blacklist.contains(caught.getItemStack().getType().toString())) {
+                    return;
+                }
             }
-        }
 
-        HashMap<Integer, ItemStack> leftOver = player.getInventory().addItem(caught.getItemStack());
-        caught.setItemStack(new ItemStack(Material.AIR));
-        if (!leftOver.isEmpty()) {
-            for (ItemStack item : leftOver.values()) {
-                player.getWorld().dropItemNaturally(loc, item);
-            }
-            if (doFullInvMSG) {
-                long secondsLeft;
-                long cooldown = 15000; // 15 sec
-                if (AutoPickup.lastInvFullNotification.containsKey(player.getUniqueId())) {
-                    secondsLeft = (AutoPickup.lastInvFullNotification.get(player.getUniqueId())/1000)+ cooldown/1000 - (System.currentTimeMillis()/1000);
-                } else {
-                    secondsLeft = 0;
+            HashMap<Integer, ItemStack> leftOver = player.getInventory().addItem(caught.getItemStack());
+            caught.remove();
+            if (!leftOver.isEmpty()) {
+                for (ItemStack item : leftOver.values()) {
+                    player.getWorld().dropItemNaturally(loc, item);
                 }
-                if (secondsLeft<=0) {
-                    player.sendMessage(PLUGIN.getMsg().getPrefix() + " " + PLUGIN.getMsg().getFullInventory());
-                    AutoPickup.lastInvFullNotification.put(player.getUniqueId(), System.currentTimeMillis());
+                if (doFullInvMSG) {
+                    long secondsLeft;
+                    long cooldown = 15000; // 15 sec
+                    if (AutoPickup.lastInvFullNotification.containsKey(player.getUniqueId())) {
+                        secondsLeft = (AutoPickup.lastInvFullNotification.get(player.getUniqueId()) / 1000) + cooldown / 1000 - (System.currentTimeMillis() / 1000);
+                    } else {
+                        secondsLeft = 0;
+                    }
+                    if (secondsLeft <= 0) {
+                        player.sendMessage(PLUGIN.getMsg().getPrefix() + " " + PLUGIN.getMsg().getFullInventory());
+                        AutoPickup.lastInvFullNotification.put(player.getUniqueId(), System.currentTimeMillis());
+                    }
                 }
             }
-        }
+        });
     }
 }