From 058dc8c6112cbedaf6650f8752935d7e7cc906e8 Mon Sep 17 00:00:00 2001
From: Glitchfinder <glitchkey@gmail.com>
Date: Mon, 29 Oct 2012 14:27:22 -0700
Subject: [PATCH] Added support for newer mobs to fishing's shake ability.

---
 pom.xml                                       |  7 ++
 .../nossr50/skills/gathering/Fishing.java     | 91 ++++++++++++++++++-
 2 files changed, 93 insertions(+), 5 deletions(-)
 mode change 100644 => 100755 pom.xml
 mode change 100644 => 100755 src/main/java/com/gmail/nossr50/skills/gathering/Fishing.java

diff --git a/pom.xml b/pom.xml
old mode 100644
new mode 100755
index ba404472b..9bf4a8c55
--- a/pom.xml
+++ b/pom.xml
@@ -121,6 +121,13 @@
             <type>jar</type>
             <scope>compile</scope>
         </dependency>
+        <dependency>
+            <groupId>org.bukkit</groupId>
+            <artifactId>craftbukkit</artifactId>
+            <version>LATEST</version>
+            <type>jar</type>
+            <scope>compile</scope>
+        </dependency>
         <dependency>
             <groupId>org.getspout</groupId>
             <artifactId>spoutpluginapi</artifactId>
diff --git a/src/main/java/com/gmail/nossr50/skills/gathering/Fishing.java b/src/main/java/com/gmail/nossr50/skills/gathering/Fishing.java
old mode 100644
new mode 100755
index 2974b05b3..b8e17fea4
--- a/src/main/java/com/gmail/nossr50/skills/gathering/Fishing.java
+++ b/src/main/java/com/gmail/nossr50/skills/gathering/Fishing.java
@@ -17,6 +17,8 @@ import org.bukkit.event.player.PlayerFishEvent;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.material.Wool;
 
+import org.bukkit.craftbukkit.entity.CraftSkeleton;
+
 import com.gmail.nossr50.config.Config;
 import com.gmail.nossr50.config.TreasuresConfig;
 import com.gmail.nossr50.datatypes.PlayerProfile;
@@ -242,7 +244,12 @@ public class Fishing {
             break;
 
         case CREEPER:
-            Misc.dropItem(location, new ItemStack(Material.SULPHUR));
+            if (DROP_NUMBER > 99) {
+                Misc.dropItem(location, new ItemStack(Material.SKULL_ITEM, 1, (short) 4));
+            }
+            else {
+                Misc.dropItem(location, new ItemStack(Material.SULPHUR));
+            }
             break;
 
         case ENDERMAN:
@@ -258,6 +265,18 @@ public class Fishing {
             }
             break;
 
+        case IRON_GOLEM:
+            if (DROP_NUMBER > 99) {
+                Misc.dropItem(location, new ItemStack(Material.PUMPKIN));
+            }
+            else if (DROP_NUMBER > 90) {
+                Misc.dropItem(location, new ItemStack(Material.IRON_INGOT));
+            }
+            else {
+                Misc.dropItem(location, new ItemStack(Material.RED_ROSE));
+            }
+            break;
+
         case MAGMA_CUBE:
             Misc.dropItem(location, new ItemStack(Material.MAGMA_CREAM));
             break;
@@ -309,11 +328,27 @@ public class Fishing {
             break;
 
         case SKELETON:
-            if (DROP_NUMBER > 50) {
-                Misc.dropItem(location, new ItemStack(Material.BONE));
+            if(((CraftSkeleton) le).getHandle().getSkeletonType() == 1) {
+                if (DROP_NUMBER > 97) {
+                    Misc.dropItem(location, new ItemStack(Material.SKULL_ITEM, 1, (short) 1));
+                }	
+                else if (DROP_NUMBER > 50) {
+                    Misc.dropItem(location, new ItemStack(Material.BONE));
+                }
+                else {
+                    Misc.dropItems(location, new ItemStack(Material.COAL), 3);
+                }
             }
             else {
-                Misc.dropItems(location, new ItemStack(Material.ARROW), 3);
+                if (DROP_NUMBER > 99) {
+                    Misc.dropItem(location, new ItemStack(Material.SKULL_ITEM));
+                }	
+                else if (DROP_NUMBER > 50) {
+                    Misc.dropItem(location, new ItemStack(Material.BONE));
+                }
+                else {
+                    Misc.dropItems(location, new ItemStack(Material.ARROW), 3);
+                }
             }
             break;
 
@@ -343,8 +378,54 @@ public class Fishing {
             Misc.dropItem(location, new ItemStack(Material.INK_SACK, 1, (short) 0, (byte) 0x0));
             break;
 
+        case WITCH:
+            final int DROP_NUMBER_2 = random.nextInt(randomChance);
+            if (DROP_NUMBER > 97) {
+                if(DROP_NUMBER_2 > 66) {
+                    Misc.dropItem(location, new ItemStack(Material.POTION, 1, (short) 8197));
+                }
+                else if(DROP_NUMBER_2 > 33) {
+                    Misc.dropItem(location, new ItemStack(Material.POTION, 1, (short) 8195));
+                }
+                else {
+                    Misc.dropItem(location, new ItemStack(Material.POTION, 1, (short) 8194));
+                }
+            }
+            else {
+                if(DROP_NUMBER_2 > 88) {
+                    Misc.dropItem(location, new ItemStack(Material.GLASS_BOTTLE));
+                }
+                else if(DROP_NUMBER_2 > 75) {
+                    Misc.dropItem(location, new ItemStack(Material.GLOWSTONE_DUST));
+                }
+                else if(DROP_NUMBER_2 > 63) {
+                    Misc.dropItem(location, new ItemStack(Material.SULPHUR));
+                }
+                else if(DROP_NUMBER_2 > 50) {
+                    Misc.dropItem(location, new ItemStack(Material.REDSTONE));
+                }
+                else if(DROP_NUMBER_2 > 38) {
+                    Misc.dropItem(location, new ItemStack(Material.SPIDER_EYE));
+                }
+                else if(DROP_NUMBER_2 > 25) {
+                    Misc.dropItem(location, new ItemStack(Material.STICK));
+                }
+                else if(DROP_NUMBER_2 > 13) {
+                    Misc.dropItem(location, new ItemStack(Material.SUGAR));
+                }
+                else {
+                    Misc.dropItem(location, new ItemStack(Material.POTION));
+                }
+            }
+            break;
+
         case ZOMBIE:
-            Misc.dropItem(location, new ItemStack(Material.ROTTEN_FLESH));
+            if (DROP_NUMBER > 99) {
+                Misc.dropItem(location, new ItemStack(Material.SKULL_ITEM, 1, (short) 2));
+            }
+            else {
+                Misc.dropItem(location, new ItemStack(Material.ROTTEN_FLESH));
+            }
             break;
 
         default: