Core/Game: Include cleanup

* Mostly aimed at removing Log/DatabaseEnv includes from other headers
* Fix most packet headers including other packet headers - moved common structures such as ItemInstance to their own files
* Moved SAI function definitions to source files (massive or requiring many different dependencies)

1 files changed, 14 insertions, 2 deletions

diff --git a/src/server/game/Battlegrounds/Battleground.h b/src/server/game/Battlegrounds/Battleground.h
index 182278fb615..5c5fc5b8076 100644
--- a/src/server/game/Battlegrounds/Battleground.h
+++ b/src/server/game/Battlegrounds/Battleground.h
@@ -26,8 +26,6 @@
 #include "WorldPacket.h"
 #include "Object.h"
 #include "GameObject.h"
-#include "Packets/WorldStatePackets.h"
-#include "Packets/BattlegroundPackets.h"
 #include "EventMap.h"
 
 class Creature;
@@ -42,6 +40,20 @@ class BattlegroundMap;
 struct PvpDifficultyEntry;
 struct WorldSafeLocsEntry;
 
+namespace WorldPackets
+{
+    namespace Battleground
+    {
+        class PVPLogData;
+        struct BattlegroundPlayerPosition;
+    }
+
+    namespace WorldState
+    {
+        class InitWorldStates;
+    }
+}
+
 enum BattlegroundCriteriaId
 {
     BG_CRITERIA_CHECK_RESILIENT_VICTORY,