[patch/dev][8635] bonus damage handling

[The_Game_Master]

Any chance of seeing this patch in git by the end of 2009?

Retedted the patch, and the output is not quite as expected.

[ApoC]

Aplied in [8635]

Thank You.

[Abim]

I decided to remove the idea with SPELL_ATTR_EX5_WEAPON_DMG_MOD_ALL_DAMAGE for now, because it has nothing to with the original idea.

Correct me if i'm wrong (I tested newest mangos build, but haven't tested the talent, yet), but I suppose http://www.wowhead.com/?spell=20198 is still buggy ?

Spell school is physical but it should increase all damage done (this is the paladin version which increases all damage)

Overall patch looks very nice .

[przemratajczak]

i was using it for a long time, and now i'm very happy to see it in main source. Since there are more hands on developers board MaNGOS evolution accelerated (AV, now this patch) lests see what happens next

[pasdVn]

Correct me if i'm wrong (I tested newest mangos build, but haven't tested the talent, yet), but I suppose http://www.wowhead.com/?spell=20198 is still buggy ?

Spell school is physical but it should increase all damage done (this is the paladin version which increases all damage)

Overall patch looks very nice .

As I wrote I excluded this Idea for now. So yes, this spell does not work correct, but not because this patch is incorrect, but because thie feature is simply not yet implemented ;-)