refactor(Core): remove unused imports (#17094)

src/server/game/AI/CoreAI/TotemAI.cpp

@@ -20,11 +20,16 @@
 #include "Creature.h"
 #include "DBCStores.h"
 #include "GridNotifiers.h"
-#include "GridNotifiersImpl.h"
 #include "ObjectAccessor.h"
 #include "SpellMgr.h"
 #include "Totem.h"
 
+/// @todo: this import is not necessary for compilation and marked as unused by the IDE
+//  however, for some reasons removing it would cause a damn linking issue
+//  there is probably some underlying problem with imports which should properly addressed
+//  see: https://github.com/azerothcore/azerothcore-wotlk/issues/9766
+#include "GridNotifiersImpl.h"
+
 int32 TotemAI::Permissible(Creature const* creature)
 {
     if (creature->IsTotem())

src/server/game/AI/CoreAI/UnitAI.cpp

@@ -21,7 +21,6 @@
 #include "Player.h"
 #include "Spell.h"
 #include "SpellAuraEffects.h"
-#include "SpellAuras.h"
 #include "SpellInfo.h"
 #include "SpellMgr.h"
 

src/server/game/AI/CreatureAI.cpp

@@ -26,7 +26,6 @@
 #include "Player.h"
 #include "Vehicle.h"
 #include "ScriptMgr.h"
-#include "Language.h"
 #include "ZoneScript.h"
 
 //Disable CreatureAI when charmed

src/server/game/AI/CreatureAISelector.cpp

@@ -22,7 +22,6 @@
 #include "MovementGenerator.h"
 
 #include "GameObject.h"
-#include "GameObjectAIFactory.h"
 
 #include "ScriptMgr.h"
 

src/server/game/AI/ScriptedAI/ScriptedCreature.cpp

@@ -15,17 +15,21 @@
  * with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "AreaBoundary.h"
 #include "ScriptedCreature.h"
 #include "Cell.h"
 #include "CellImpl.h"
 #include "GameTime.h"
 #include "GridNotifiers.h"
-#include "GridNotifiersImpl.h"
 #include "ObjectMgr.h"
 #include "Spell.h"
 #include "TemporarySummon.h"
 
+/// @todo: this import is not necessary for compilation and marked as unused by the IDE
+//  however, for some reasons removing it would cause a damn linking issue
+//  there is probably some underlying problem with imports which should properly addressed
+//  see: https://github.com/azerothcore/azerothcore-wotlk/issues/9766
+#include "GridNotifiersImpl.h"
+
 // Spell summary for ScriptedAI::SelectSpell
 struct TSpellSummary
 {

src/server/game/AI/SmartScripts/SmartAI.cpp

@@ -19,7 +19,6 @@
 #include "CellImpl.h"
 #include "GridDefines.h"
 #include "GridNotifiers.h"
-#include "GridNotifiersImpl.h"
 #include "Group.h"
 #include "ObjectDefines.h"
 #include "ObjectMgr.h"

src/server/game/AI/SmartScripts/SmartScript.cpp

@@ -24,7 +24,6 @@
 #include "GossipDef.h"
 #include "GridDefines.h"
 #include "GridNotifiers.h"
-#include "GridNotifiersImpl.h"
 #include "Group.h"
 #include "InstanceScript.h"
 #include "Language.h"
@@ -37,6 +36,12 @@
 #include "SpellMgr.h"
 #include "Vehicle.h"
 
+/// @todo: this import is not necessary for compilation and marked as unused by the IDE
+//  however, for some reasons removing it would cause a damn linking issue
+//  there is probably some underlying problem with imports which should properly addressed
+//  see: https://github.com/azerothcore/azerothcore-wotlk/issues/9766
+#include "GridNotifiersImpl.h"
+
 SmartScript::SmartScript()
 {
     go = nullptr;

src/server/game/AI/SmartScripts/SmartScriptMgr.cpp

@@ -21,8 +21,6 @@
 #include "DatabaseEnv.h"
 #include "GameEventMgr.h"
 #include "GridDefines.h"
-#include "GridNotifiers.h"
-#include "GridNotifiersImpl.h"
 #include "InstanceScript.h"
 #include "ObjectDefines.h"
 #include "ObjectMgr.h"