AC/ACv2 negative affects don't affect stacking fixes #622

Unsure if these are the only SPAs that should act this way, but this
resolves the issues we were seeing
This commit is contained in:
Michael Cook (mackal) 2017-06-08 00:54:22 -04:00
parent decaa1f7b6
commit c25a5fa0ba

View File

@ -3059,6 +3059,12 @@ int Mob::CheckStackConflict(uint16 spellid1, int caster_level1, uint16 spellid2,
if (IsEffectIgnoredInStacking(effect1))
continue;
// negative AC affects are skipped. Ex. Sun's Corona and Glacier Breath should stack
// There may be more SPAs we need to add here ....
// The client does just check base rather than calculating the affect change value.
if ((effect1 == SE_ArmorClass || effect1 == SE_ACv2) && sp2.base[i] < 0)
continue;
/*
If target is a npc and caster1 and caster2 exist
If Caster1 isn't the same as Caster2 and the effect is a DoT then ignore it.