Server/Entities: Expertise now affects ranged attacks.

Signed-off-by: AriDEV <aridev666@gmail.com>
ProjectSkyfire · Mar 6, 2019 · c7cf101 · c7cf101
1 parent 65ba8a8
commit c7cf101
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/server/game/Entities/Player/Player.cpp b/src/server/game/Entities/Player/Player.cpp
@@ -6088,6 +6088,8 @@ float Player::GetExpertiseDodgeOrParryReduction(WeaponAttackType attType) const
             return GetUInt32Value(PLAYER_FIELD_MAINHAND_EXPERTISE) / 4.0f;
         case OFF_ATTACK:
             return GetUInt32Value(PLAYER_FIELD_OFFHAND_EXPERTISE) / 4.0f;
+        case RANGED_ATTACK:
+            return GetUInt32Value(PLAYER_FIELD_RANGED_EXPERTISE) / 4.0f;
         default:
             break;
     }

diff --git a/src/server/game/Entities/Unit/StatSystem.cpp b/src/server/game/Entities/Unit/StatSystem.cpp
@@ -695,9 +695,6 @@ void Player::UpdateAllSpellCritChances()
 
 void Player::UpdateExpertise(WeaponAttackType attack)
 {
-    if (attack == RANGED_ATTACK)
-        return;
-
     int32 expertise = int32(GetRatingBonusValue(CR_EXPERTISE));
 
     Item* weapon = GetWeaponForAttack(attack, true);
@@ -718,8 +715,9 @@ void Player::UpdateExpertise(WeaponAttackType attack)
 
     switch (attack)
     {
-        case BASE_ATTACK: SetUInt32Value(PLAYER_FIELD_MAINHAND_EXPERTISE, expertise);         break;
+        case BASE_ATTACK: SetUInt32Value(PLAYER_FIELD_MAINHAND_EXPERTISE, expertise); break;
         case OFF_ATTACK:  SetUInt32Value(PLAYER_FIELD_OFFHAND_EXPERTISE, expertise); break;
+        case RANGED_ATTACK: SetUInt32Value(PLAYER_FIELD_RANGED_EXPERTISE, expertise); break;
         default: break;
     }
 }