Rewrote "player drops item" in ScriptEvent format.

src/main/java/net/aufdemrand/denizen/Denizen.java

@@ -644,6 +644,7 @@ public void onEnable() {
             ScriptEvent.registerScriptEvent(new PlayerClosesInvScriptEvent());
             ScriptEvent.registerScriptEvent(new PlayerConsumesScriptEvent());
             ScriptEvent.registerScriptEvent(new PlayerDamagesBlockScriptEvent());
+            ScriptEvent.registerScriptEvent(new PlayerDropsItemScriptEvent());
             ScriptEvent.registerScriptEvent(new PlayerEntersBedScriptEvent());
             ScriptEvent.registerScriptEvent(new PlayerJumpScriptEvent());
             ScriptEvent.registerScriptEvent(new PlayerLeavesBedScriptEvent());

src/main/java/net/aufdemrand/denizen/events/player/PlayerDropsItemScriptEvent.java

@@ -0,0 +1,114 @@
+package net.aufdemrand.denizen.events.player;
+
+import net.aufdemrand.denizen.BukkitScriptEntryData;
+import net.aufdemrand.denizen.events.BukkitScriptEvent;
+import net.aufdemrand.denizen.objects.dEntity;
+import net.aufdemrand.denizen.objects.dItem;
+import net.aufdemrand.denizen.objects.dLocation;
+import net.aufdemrand.denizen.utilities.DenizenAPI;
+import net.aufdemrand.denizencore.objects.dObject;
+import net.aufdemrand.denizencore.scripts.ScriptEntryData;
+import net.aufdemrand.denizencore.scripts.containers.ScriptContainer;
+import net.aufdemrand.denizencore.utilities.CoreUtilities;
+import org.bukkit.Bukkit;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.Listener;
+import org.bukkit.event.player.PlayerDropItemEvent;
+
+import java.util.HashMap;
+
+public class PlayerDropsItemScriptEvent extends BukkitScriptEvent implements Listener {
+
+    // <--[event]
+    // @Events
+    // player drops item (in <area>)
+    // player drops <item> (in <area>)
+    //
+    // @Cancellable true
+    //
+    // @Triggers when a player drops an item.
+    //
+    // @Context
+    // <context.item> returns the dItem.
+    // <context.entity> returns a dEntity of the item.
+    // <context.location> returns a dLocation of the item's location.
+    //
+    // -->
+
+    public PlayerDropsItemScriptEvent() {
+        instance = this;
+    }
+
+    public static PlayerDropsItemScriptEvent instance;
+    public dItem item;
+    public dEntity entity;
+    public dLocation location;
+    public PlayerDropItemEvent event;
+
+    @Override
+    public boolean couldMatch(ScriptContainer scriptContainer, String s) {
+        return CoreUtilities.toLowerCase(s).startsWith("player drops");
+    }
+
+    @Override
+    public boolean matches(ScriptContainer scriptContainer, String s) {
+        String lower = CoreUtilities.toLowerCase(s);
+
+        if (dEntity.isNPC(event.getPlayer())) {
+            return false;
+        }
+        String iCheck = CoreUtilities.getXthArg(2, lower);
+        if (!iCheck.equals("item") && !tryItem(item, iCheck)) {
+            return false;
+        }
+        if (!runInCheck(scriptContainer, s, lower, location)) {
+            return false;
+        }
+        return true;
+    }
+
+    @Override
+    public String getName() {
+        return "PlayerDropsItem";
+    }
+
+    @Override
+    public void init() {
+        Bukkit.getServer().getPluginManager().registerEvents(this, DenizenAPI.getCurrentInstance());
+    }
+
+    @Override
+    public void destroy() {
+        PlayerDropItemEvent.getHandlerList().unregister(this);
+    }
+
+    @Override
+    public boolean applyDetermination(ScriptContainer container, String determination) {
+        return super.applyDetermination(container, determination);
+    }
+
+    @Override
+    public ScriptEntryData getScriptEntryData() {
+        return new BukkitScriptEntryData(dEntity.isPlayer(event.getPlayer()) ? dEntity.getPlayerFrom(event.getPlayer()) : null, null);
+    }
+
+    @Override
+    public HashMap<String, dObject> getContext() {
+        HashMap<String, dObject> context = super.getContext();
+        context.put("item", item);
+        context.put("entity", entity);
+        context.put("location", location);
+        return context;
+    }
+
+    @EventHandler
+    public void onPlayerDropsItem(PlayerDropItemEvent event) {
+        location = new dLocation(event.getPlayer().getLocation());
+        item = new dItem(event.getItemDrop().getItemStack());
+        entity = new dEntity(event.getItemDrop());
+        cancelled = event.isCancelled();
+        this.event = event;
+        fire();
+        event.setCancelled(cancelled);
+    }
+}

src/main/java/net/aufdemrand/denizen/scripts/containers/core/BukkitWorldScriptHelper.java

@@ -738,44 +738,6 @@ public void playerAnimation(PlayerAnimationEvent event) {
             event.setCancelled(true);
     }
 
-    // <--[event]
-    // @Events
-    // player drops item
-    // player drops <item>
-    //
-    // @Triggers when a player drops an item.
-    // @Context
-    // <context.item> returns the dItem.
-    // <context.entity> returns a dEntity of the item.
-    // <context.location> returns a dLocation of the item's location.
-    //
-    // @Determine
-    // "CANCELLED" to stop the item from being dropped.
-    //
-    // -->
-    @EventHandler
-    public void playerDropItem(PlayerDropItemEvent event) {
-
-        if (dEntity.isNPC(event.getPlayer()))
-            return;
-
-        Map<String, dObject> context = new HashMap<String, dObject>();
-        dItem item = new dItem(event.getItemDrop().getItemStack());
-        context.put("item", item);
-        context.put("entity", new dEntity(event.getItemDrop()));
-        context.put("location", new dLocation(event.getItemDrop().getLocation()));
-
-        List<String> events = new ArrayList<String>();
-
-        events.add("player drops item");
-        events.add("player drops " + item.identifySimple());
-
-        String determination = doEvents(events, null, dEntity.getPlayerFrom(event.getPlayer()), context, true);
-
-        if (determination.toUpperCase().startsWith("CANCELLED"))
-            event.setCancelled(true);
-    }
-
     // <--[event]
     // @Events
     // player throws (hatching/non-hatching) egg