[patch/dev] Anti-Magic zone (Зона антимагии)

[Insider42]

Суть проблемы:
Не учитывался бонус силы атаки для заклинания.

Предположительный фикс

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index 5668769..13ea8b9 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -7410,6 +7410,12 @@ void Aura::HandleSchoolAbsorb(bool apply, bool Real)
                         DoneActualBenefit = caster->SpellBaseHealingBonusDone(GetSpellSchoolMask(spellProto)) * 0.75f;
                     }
                     break;
+                case SPELLFAMILY_DEATHKNIGHT:
+                    // Anti-magic shield
+                    if (spellProto->Id == 50461)
+                        if (Unit *owner = caster->GetOwner())
+                            DoneActualBenefit += caster->GetOwner()->GetTotalAttackPowerValue(BASE_ATTACK) * 2;
+                    break;
                 default:
                     break;
             }

У меня смутное сомнение что код должен быть чуть ниже, т.к. мы считаем для него LevelPenalty, что по-моему неверно для спеллов берущих бонус АП

Возможно также потребуется проверка на то, игрок ли caster

Патч вообще не тестировался, жду комментариев

[zergtmn]

+                    // Anti-magic shield
+                    if (spellProto->Id == 50461)
+                        if (Unit *owner = caster->GetOwner())
+                            DoneActualBenefit += caster->GetOwner()->GetTotalAttackPowerValue(BASE_ATTACK) * 2;

На вовхеде формула: 10000 + 2 * AP. Базовые 10000 абсорба тоже надо учесть. Причем они хранятся как basepoints у спелла http://ru.wowhead.com/spell=51052

[Insider42]

всё там учитывается, между прочим там += стоит. Гляньте ниже по коду и поймёте о чем я. Как я уже говорил, вопрос в том, должен ли быть LevelPenalty для него?

[zergtmn]

Не нашел, где вычисляется базовый абсорб.
Идет:

float DoneActualBenefit = 0.0f;

потом свич по spellfamily, например:

                case SPELLFAMILY_WARLOCK:
                    // Shadow Ward
                    if (spellProto->SpellFamilyFlags2 & 0x00000040)
                        //+30% from +spell bonus
                        DoneActualBenefit = caster->SpellBaseDamageBonusDone(GetSpellSchoolMask(spellProto)) * 0.30f;
                    break;

И в конце:

m_modifier.m_amount += (int32)DoneActualBenefit;

Откуда возьмется в m_modifier.m_amount 10000, если спеллы разные? Или я что-то не понимаю?
10000 в данном случае хранится в спелле суммона.

[Insider42]

Вообще это дело нада в игре теститЬ, чего я сейчас сделать не могу. Игроки говорят что он абсорбит сейчас (без патча), но не учитывает АП. Т.е. сумма абсорба сейчас статична.

[Insider42]

Кто-нибудь уже протестил?