NetSky

ok i see what you are pointing at but on the other hand this is hacky too i suggest. btw i already told you i am working on a generic solution for additional spell/healing boni i know its hacky

this is my suggestion how you should do this: why are you using new spell id defines? this is not neccesary for this case because stealth form has not any other spell id applied than 0 which could also be deleted beacause there is no redefinition needed. spellids get initialized with 0. for talents like master shapeshifter you need extra spellid defines because spell ids are already in use at shapeshift. for additional damage calculation you can lookup my master shapeshifter patch(dire bear or bear form) check http://github.com/NetSky/mangos/commit/0c1acaddb6c627bd9f37743ad625c1b0cab95d51 as an example but take care whitespaces got fixed in a later commit). spell and heal damage boni are still to hacky this way thats why i am working on an generic methods with new unit stat applied adding percantage spell damage done by several spells or whole class spell categories related to caster. edit: + if(((Player*)this)->HasAura(48412)) + { + if(((Player*)this)->HasAura(48422) && spellProto->Effect[effect_index] == SPELL_EFFECT_HEAL) + value = value * 1.04; + if(((Player*)this)->HasAura(48421) && spellProto->Effect[effect_index] == SPELL_EFFECT_SCHOOL_DAMAGE) + value = value * 1.04; + } + } has to be replaced by + if(((Player*)this)->HasAura(48412)) + { + if(((Player*)this)->HasAura(48422) && spellProto->Effect[effect_index] != SPELL_EFFECT_SCHOOL_DAMAGE) + value = value * 1.04; + if(((Player*)this)->HasAura(48421) && spellProto->Effect[effect_index] != SPELL_EFFECT_HEAL) + value = value * 1.04; + } + }