Harmony/Lull temporary fix

[gernblan]

This is based off of 1030 build?

Thanks for your efforts! This will help many classes, especially chanters, if it gets checked in.

[inkubus]

Tested on 1026 and 1030 build.

Thanks

Bolly

[inkubus]

Looks like this has a side effect on npc->npc aggro. Looking into this.

[inkubus]

Ok found the problem, looks like hate of 0 is needed so drop the change to aggro.cpp and instead add this to spells.cpp ~ 2292:

Replace:

if(spelltar->IsAIControlled())
{
spelltar->AddToHateList(this, 1);
}

With:
if(spelltar->IsAIControlled())
{
if (!IsHarmonySpell(spell_id))
{
spelltar->AddToHateList(this, 1);
}
}

[inkubus]

And then of course further down in spells.cpp at the AddToHateList() ~ 2365

Replace:

if (spelltar->IsAIControlled() && IsDetrimentalSpell(spell_id)
) {
int16 aggro_amount = CheckAggroAmount(spell_id);//*spelltar->CastToNPC()->AggroModifier();
mlog(SPELLS__CASTING, "Spell %d cast on %s generated %d hate", spell_id, spelltar->GetName(), aggro_amount);
spelltar->AddToHateList(this, aggro_amount);
}


with:

if (spelltar->IsAIControlled() && IsDetrimentalSpell(spell_id)
) {
int16 aggro_amount = CheckAggroAmount(spell_id);//*spelltar->CastToNPC()->AggroModifier();
mlog(SPELLS__CASTING, "Spell %d cast on %s generated %d hate", spell_id, spelltar->GetName(), aggro_amount);
if (!IsHarmonySpell(spell_id))
{
spelltar->AddToHateList(this, aggro_amount);
}
}

[inkubus]

Thanks KLS, just done that now. Appears to be working but will continue further testing!

[inkubus]

Ok this now works appart from one issue where if it is resisted it won't generate aggro. I will work on this and then post a final diff file.

[KLS]

AddToHateList should be able to accept 0 hate as an argument without us ignoring it. Sometimes we want to be added to something's hate list without actually generating any hate. This is how proximity aggro works.

Probably want to look for where we're calling AddToHateList for lull spells and catch it before it gets there.

Bah apparently you followed up while I was writing this... ingenious!