Core/AI: No longer do strange things when told to enter evade mode while there's no need to do so, typically by terrible legacy code.

src/common/Utilities/FuzzyFind.h

@@ -18,6 +18,10 @@
 #ifndef TRINITY_FUZZYFIND_H
 #define TRINITY_FUZZYFIND_H
 
+#include "advstd.h"
+#include <map>
+#include <string>
+
 namespace Trinity
 {
     namespace Containers

src/server/game/AI/CreatureAI.cpp

@@ -285,7 +285,7 @@ void CreatureAI::EngagementOver()
 
 bool CreatureAI::_EnterEvadeMode(EvadeReason /*why*/)
 {
-    if (me->IsInEvadeMode())
+    if (!IsEngaged())
         return false;
 
     if (!me->IsAlive())

src/server/game/Grids/ObjectGridLoader.cpp

@@ -250,10 +250,7 @@ void ObjectGridStoper::Visit(CreatureMapType &m)
     {
         iter->GetSource()->RemoveAllDynObjects();
         if (iter->GetSource()->IsInCombat())
-        {
             iter->GetSource()->CombatStop();
-            iter->GetSource()->AI()->EnterEvadeMode();
-        }
     }
 }