Core/Misc: moved WeaponAttackType definition to SharedDefines to avoid including Unit.h

(cherry picked from commit 6d991e9361)

src/server/game/Entities/Unit/Unit.h

@@ -430,14 +430,6 @@ enum UnitState
 TC_GAME_API extern float baseMoveSpeed[MAX_MOVE_TYPE];
 TC_GAME_API extern float playerBaseMoveSpeed[MAX_MOVE_TYPE];
 
-enum WeaponAttackType : uint8
-{
-    BASE_ATTACK   = 0,
-    OFF_ATTACK    = 1,
-    RANGED_ATTACK = 2,
-    MAX_ATTACK
-};
-
 enum CombatRating
 {
     CR_AMPLIFY                          = 0,

src/server/game/Miscellaneous/SharedDefines.h

@@ -5392,6 +5392,14 @@ enum DiminishingLevels
     DIMINISHING_LEVEL_TAUNT_IMMUNE  = 4
 };
 
+enum WeaponAttackType : uint8
+{
+    BASE_ATTACK   = 0,
+    OFF_ATTACK    = 1,
+    RANGED_ATTACK = 2,
+    MAX_ATTACK
+};
+
 enum TokenResult
 {
     TOKEN_RESULT_SUCCESS                        = 0,