Wouldn't it make more sense to calculate this as a Spell Bonus in
Mob::ApplySpellBonuses()?
Mob::CalcSpellBonuses() iterates through the spell slots (including any ones that have been added by AAs) already, so it seems redundant to do it again when calling
GetBuffSlotFromType() and then iterating through the spell's effects. It would also allow for stacking (if able) of the spell effects since
GetBuffSlotFromType() only gets the first slot.