From 5d3c8bfaa4d3483bef297241de284ff4916fd79d Mon Sep 17 00:00:00 2001
From: Wesley Wolfe <weswolf@aol.com>
Date: Thu, 24 May 2012 23:02:02 -0500
Subject: [PATCH] Removed AnimalTamer cache, fixes BUKKIT-1628

---
 .../entity/CraftTameableAnimal.java           | 25 ++++++-------------
 1 file changed, 7 insertions(+), 18 deletions(-)

diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftTameableAnimal.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftTameableAnimal.java
index 5226c392f8..04520be3ee 100644
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftTameableAnimal.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftTameableAnimal.java
@@ -4,12 +4,9 @@ import net.minecraft.server.EntityTameableAnimal;
 import org.bukkit.craftbukkit.CraftServer;
 import org.bukkit.entity.AnimalTamer;
 import org.bukkit.entity.Creature;
-import org.bukkit.entity.Player;
 import org.bukkit.entity.Tameable;
 
 public class CraftTameableAnimal extends CraftAnimals implements Tameable, Creature {
-    private AnimalTamer owner;
-
     public CraftTameableAnimal(CraftServer server, EntityTameableAnimal entity) {
         super(server, entity);
     }
@@ -20,12 +17,11 @@ public class CraftTameableAnimal extends CraftAnimals implements Tameable, Creat
     }
 
     public AnimalTamer getOwner() {
-        if (owner == null && !("").equals(getOwnerName())) {
-            owner = getServer().getPlayer(getOwnerName());
+        if (("").equals(getOwnerName())) return null;
 
-            if (owner == null) {
-                owner = getServer().getOfflinePlayer(getOwnerName());
-            }
+        AnimalTamer owner = getServer().getPlayerExact(getOwnerName());
+        if (owner == null) {
+            owner = getServer().getOfflinePlayer(getOwnerName());
         }
 
         return owner;
@@ -40,17 +36,10 @@ public class CraftTameableAnimal extends CraftAnimals implements Tameable, Creat
     }
 
     public void setOwner(AnimalTamer tamer) {
-        owner = tamer;
-
-        if (owner != null) {
+        if (tamer != null) {
             setTamed(true);
             getHandle().setPathEntity(null);
-
-            if (owner instanceof Player) {
-                setOwnerName(((Player) owner).getName());
-            } else {
-                setOwnerName("");
-            }
+            setOwnerName(tamer.getName());
         } else {
             setTamed(false);
             setOwnerName("");
@@ -58,7 +47,7 @@ public class CraftTameableAnimal extends CraftAnimals implements Tameable, Creat
     }
 
     public void setOwnerName(String ownerName) {
-        getHandle().setOwnerName(ownerName);
+        getHandle().setOwnerName(ownerName == null ? "" : ownerName);
     }
 
     public void setTamed(boolean tame) {