[dev]SPELL_AURA_MOD_HEALING_DONE

[virusav]

Есть спеллы, у которых есть эффект ауры 135 (SPELL_AURA_MOD_HEALING_DONE), но изменение доп. хила не наблюдается визуально в статах игрока, при касте спеллов на лечение бонус также не ощущается.

Просмотрел все спеллы с аурой 135, у которых должна быть подобная ситуация (разбирал на примере 67684):

26276
37436
55637
60565
60566
60567
60568
60569
60570
63283
67684
70244

В Unit.cpp в int32 Unit::SpellBaseHealingBonusDone(SpellSchoolMask schoolMask) есть условие:

if(((*i)->GetModifier()->m_miscvalue & schoolMask) != 0)

но для этих спеллов:
(*i)->GetModifier()->m_miscvalue = 0
schoolMask = SPELL_SCHOOL_MASK_ALL // 127

В итоге 0 & 127 = 0, поэтому по условию бонус не рассчитывается.

Как мне кажется, надо изменить условие, чтобы бонусы подобных спеллов отрабатывали.

Варианты (на вскидку):

if(((*i)->GetModifier()->m_miscvalue & schoolMask) != 0 || (*i)->GetModifier()->m_miscvalue == 0)

if(((*i)->GetModifier()->m_miscvalue & schoolMask) != 0 || schoolMask == SPELL_SCHOOL_MASK_ALL)

У себя для тестов поставил

if(((*i)->GetModifier()->m_miscvalue & schoolMask) != 0 || ((*i)->GetModifier()->m_miscvalue == 0 && schoolMask == SPELL_SCHOOL_MASK_ALL))

С таким условием все указанные выше спеллы визуально дают бонус в статах игрока, который также ощущается при касте спеллов на лечение.

Проверил некоторые спеллы на выбор с аурой 135 не из этого списка - бонусы не изменились, т.к. в цикл заходит по старому условию.

Сегодня уже нет времени проверять дальше.
Какие будут отзывы и предложения?

[virusav]

Просмотрел все, откуда идет заход в SpellBaseHealingBonusDone, но нигде не нашел захода в блок с условием, которое менял, т.е. измененное условие не нарушает работу других спеллов.

Кроме того, перед циклом идет

AuraList const& mHealingDone = GetAurasByType(SPELL_AURA_MOD_HEALING_DONE);

Т.е. выбираются все с данным типом аур, т.е. другие сюда даже попасть не могут.