From 7cde5b755e7b501842be3227c2ed4d75231b6ac6 Mon Sep 17 00:00:00 2001
From: libraryaddict <libraryaddict115@yahoo.co.nz>
Date: Thu, 5 Jun 2014 00:30:12 +1200
Subject: [PATCH] Read desc

Added boolean to disguise which tells you simply if the disguise is in
use or not.
In use means that the plugin stores a reference, doesn't nessercerily
mean there is a entity.

Added method startDisguise() to disguise and cleaned up setEntity().
Helps counter the idiots who hate DisguiseAPI
---
 .../libraryaddict/disguise/DisguiseAPI.java   |  9 +----
 .../disguise/disguisetypes/Disguise.java      | 38 +++++++++++++++++--
 2 files changed, 36 insertions(+), 11 deletions(-)

diff --git a/src/me/libraryaddict/disguise/DisguiseAPI.java b/src/me/libraryaddict/disguise/DisguiseAPI.java
index 590833a1..b5475994 100644
--- a/src/me/libraryaddict/disguise/DisguiseAPI.java
+++ b/src/me/libraryaddict/disguise/DisguiseAPI.java
@@ -167,12 +167,7 @@ public class DisguiseAPI {
             // Set the disguise's entity
             disguise.setEntity(entity);
         }
-        // Stick the disguise in the disguises bin
-        DisguiseUtilities.addDisguise(entity.getUniqueId(), (TargetedDisguise) disguise);
-        // Resend the disguised entity's packet
-        DisguiseUtilities.refreshTrackers((TargetedDisguise) disguise);
-        // If he is a player, then self disguise himself
-        DisguiseUtilities.setupFakeDisguise(disguise);
+        disguise.startDisguise();
     }
 
     public static void disguiseIgnorePlayers(Entity entity, Disguise disguise, Collection playersToNotSeeDisguise) {
@@ -326,7 +321,7 @@ public class DisguiseAPI {
     }
 
     public static boolean isDisguiseInUse(Disguise disguise) {
-        return DisguiseUtilities.isDisguiseInUse(disguise);
+        return disguise.isDisguiseInUse();
     }
 
     /**
diff --git a/src/me/libraryaddict/disguise/disguisetypes/Disguise.java b/src/me/libraryaddict/disguise/disguisetypes/Disguise.java
index 9e5c2d3e..ecb75f51 100644
--- a/src/me/libraryaddict/disguise/disguisetypes/Disguise.java
+++ b/src/me/libraryaddict/disguise/disguisetypes/Disguise.java
@@ -33,6 +33,7 @@ import com.comphenix.protocol.reflect.StructureModifier;
 
 public abstract class Disguise {
     private static JavaPlugin plugin;
+    private boolean disguiseInUse;
     private DisguiseType disguiseType;
     private Entity entity;
     private boolean hearSelfDisguise = DisguiseConfig.isSelfDisguisesSoundsReplaced();
@@ -326,6 +327,14 @@ public abstract class Disguise {
         return watcher;
     }
 
+    /**
+     * In use doesn't mean that this disguise is active. It means that Lib's Disguises still stores a reference to the disguise.
+     * getEntity() can still return null if this disguise is active after despawn, logout, etc.
+     */
+    public boolean isDisguiseInUse() {
+        return disguiseInUse;
+    }
+
     public boolean isHidingArmorFromSelf() {
         return hideArmorFromSelf;
     }
@@ -398,9 +407,12 @@ public abstract class Disguise {
      * Removes the disguise and undisguises the entity if its using this disguise. This doesn't fire a UndisguiseEvent
      */
     public void removeDisguise() {
-        if (taskId != -1) {
-            Bukkit.getScheduler().cancelTask(taskId);
-            taskId = -1;
+        if (disguiseInUse) {
+            disguiseInUse = false;
+            if (taskId != -1) {
+                Bukkit.getScheduler().cancelTask(taskId);
+                taskId = -1;
+            }
             HashMap<UUID, HashSet<TargetedDisguise>> disguises = DisguiseUtilities.getDisguises();
             // If this disguise has a entity set
             if (getEntity() != null) {
@@ -461,7 +473,6 @@ public abstract class Disguise {
         }
         this.entity = entity;
         setupWatcher();
-        taskId = Bukkit.getScheduler().scheduleSyncRepeatingTask(plugin, velocityRunnable, 1, 1);
     }
 
     public void setHearSelfDisguise(boolean hearSelfDisguise) {
@@ -640,5 +651,24 @@ public abstract class Disguise {
                     + getType().getWatcherClass().getSimpleName() + " for DisguiseType " + getType().name());
         }
         watcher = newWatcher;
+        if (getEntity() != null) {
+            setupWatcher();
+        }
+    }
+
+    public void startDisguise() {
+        if (!isDisguiseInUse()) {
+            if (getEntity() == null) {
+                throw new RuntimeException("No entity is assigned to this disguise!");
+            }
+            disguiseInUse = true;
+            taskId = Bukkit.getScheduler().scheduleSyncRepeatingTask(plugin, velocityRunnable, 1, 1);
+            // Stick the disguise in the disguises bin
+            DisguiseUtilities.addDisguise(entity.getUniqueId(), (TargetedDisguise) this);
+            // Resend the disguised entity's packet
+            DisguiseUtilities.refreshTrackers((TargetedDisguise) this);
+            // If he is a player, then self disguise himself
+            DisguiseUtilities.setupFakeDisguise(this);
+        }
     }
 }
\ No newline at end of file