Harmony/Lull temporary fix

[Angelox]

So what happened with this fix going into the official source? This is a good fix, and needs to be introduced.

[TheLieka]

Did anyone ever figure out a good way to make this work with AE lulls?

Dax

[Sadoke]

I think that the same problem behind the lull and harmony line is also behind the problems I have seen with the Enchanter Alliance spell.

When cast, it produces instant aggro from the target.

Something to add to the check list.

[realityincarnate]

The same basic fix also works for the Alliance line of spells (and bard song).
I'm new at this and haven't tested it extensively, but it seems to have things working the way they should.

In IsHarmonySpell(), spdat.cpp line ~202 replace:

if(sp.effectid[i] == SE_Lull || sp.effectid[i] == SE_Harmony)


with:

if(sp.effectid[i] == SE_Lull || sp.effectid[i] == SE_Harmony || sp.effectid[i] == SE_AddFaction)

[mixxit]

Still having this issue - is this in the current one?

[gaeorn]

Originally Posted by mixxit

Still having this issue - is this in the current one?

Could you be more specific? What problem are you having?

[mixxit]

A successful harmony causes aggro