Aura: Adjust masked SPELL_AURA_MOD_HIT_CHANCE applying to wrong weapo…

…n types Only designation exists to melee vs ranged so not adding redundant main hand vs offhand granularity
cmangos · Nov 26, 2023 · 517473c · 517473c
1 parent d2d9ec7
commit 517473c
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 5 deletions.
diff --git a/src/game/Entities/Player.cpp b/src/game/Entities/Player.cpp
@@ -10484,9 +10484,14 @@ Item* Player::EquipItem(uint16 pos, Item* pItem, bool update)
         ApplyEquipCooldown(pItem);
 
         if (slot == EQUIPMENT_SLOT_MAINHAND)
+        {
             UpdateExpertise(BASE_ATTACK);
+            UpdateMeleeHitChances();
+        }
         else if (slot == EQUIPMENT_SLOT_OFFHAND)
             UpdateExpertise(OFF_ATTACK);
+        else if (slot == EQUIPMENT_SLOT_RANGED)
+            UpdateRangedHitChances();
     }
     else
     {

diff --git a/src/game/Entities/StatSystem.cpp b/src/game/Entities/StatSystem.cpp
@@ -572,13 +572,39 @@ void Player::UpdateSpellCritChance(uint32 school)
 
 void Player::UpdateMeleeHitChances()
 {
-    m_modMeleeHitChance = GetTotalAuraModifier(SPELL_AURA_MOD_HIT_CHANCE);
+    int32 meleeHitChance = 0;
+    Item* weapon = GetWeaponForAttack(BASE_ATTACK);
+
+    AuraList const& hitAura = GetAurasByType(SPELL_AURA_MOD_HIT_CHANCE);
+    for (auto hitAura : hitAura)
+    {
+        // item neutral spell
+        if (hitAura->GetSpellProto()->EquippedItemClass == -1)
+            meleeHitChance += hitAura->GetModifier()->m_amount;
+        // item dependent spell
+        else if (weapon && weapon->IsFitToSpellRequirements(hitAura->GetSpellProto()))
+            meleeHitChance += hitAura->GetModifier()->m_amount;
+    }
+    m_modMeleeHitChance = meleeHitChance;
     m_modMeleeHitChance +=  GetRatingBonusValue(CR_HIT_MELEE);
 }
 
 void Player::UpdateRangedHitChances()
 {
-    m_modRangedHitChance = GetTotalAuraModifier(SPELL_AURA_MOD_HIT_CHANCE);
+    int32 rangedHitChance = 0;
+    Item* weapon = GetWeaponForAttack(RANGED_ATTACK);
+
+    AuraList const& hitAura = GetAurasByType(SPELL_AURA_MOD_HIT_CHANCE);
+    for (auto hitAura : hitAura)
+    {
+        // item neutral spell
+        if (hitAura->GetSpellProto()->EquippedItemClass == -1)
+            rangedHitChance += hitAura->GetModifier()->m_amount;
+        // item dependent spell
+        else if (weapon && weapon->IsFitToSpellRequirements(hitAura->GetSpellProto()))
+            rangedHitChance += hitAura->GetModifier()->m_amount;
+    }
+    m_modMeleeHitChance = rangedHitChance;
     m_modRangedHitChance += GetRatingBonusValue(CR_HIT_RANGED);
 }
 

diff --git a/src/game/Spells/SpellAuras.cpp b/src/game/Spells/SpellAuras.cpp
@@ -5863,10 +5863,10 @@ void Aura::HandleModHitChance(bool apply, bool /*Real*/)
 {
     Unit* target = GetTarget();
 
-    if (target->GetTypeId() == TYPEID_PLAYER)
+    if (target->IsPlayer())
     {
-        ((Player*)target)->UpdateMeleeHitChances();
-        ((Player*)target)->UpdateRangedHitChances();
+        static_cast<Player*>(target)->UpdateMeleeHitChances();
+        static_cast<Player*>(target)->UpdateRangedHitChances();
     }
     else
     {