[Patch][8956] Implement aura SPELL_AURA_ALLOW_ONLY_ABILITY

[Auntie Mangos]

Quoting again my post because you guys maybe don't understand... the first effect says which immunities it should grant, implementing the aura will fix them, is the same as creatures immunities.

If is proven there should be some additional effect immunity like fear, only that needs to be hardcoded.

If it has to be hardcoded I can try... but I don't understand why it is that way when the effects that would need to be hardcoded exist and could be used like the charm one.

EDIT: done http://getmangos.eu/community/viewtopic.php?id=6940.

[Maxxie]

What bug does the patch fix? What features does the patch add?

This patch implements the SPELL_AURA_ALLOW_ONLY_ABILITY which should deny to its target to cast any spell or attack for its duration.

For which repository revision was the patch created?

7288

Who has been writing this patch?

Me

Git: http://github.com/dragonshard/mangos/commit/f4d1e058fb3faa645b20f6457e57f2da8ba3e199

[Sephiroth1983]

So this should fix Bladestorm from using all spells while affected by it?

No more Mortal Strikes while being hitted by 3 consecutive whirlwind, good.

[aerione]

That's primarily the spell thats gonna be fixed by this, it uses Aura 263 SPELL_AURA_ALLOW_ONLY_ABILITY.

Sounds great, warriors were quite overpowered without this fix.

[Maxxie]

Wish I could fix the crow control immunity too, but for now the result is it makes the warrior immune to damage from that spell too. For example it makes the player immune to the frostbolt's snare as it should but it has the problem that prevents the damage too. It is the same problem that there was with totems before... So I guess it shouldn't be too difficult

[Lightguard]

It would fix other warrior talents too:) It would be great

[Ebrithil]

*Bump*

Very important patch,

would be great if some dev could take a look at this.

[LickedLurk]

Would be great to have this in git... nice job maxxie

[Gommes]

bump

its very important to nerf these warriors

[Sephiroth1983]

bump

its very important to nerf these warriors

Nice by you, since you are one of them good.

About the fix, makes sense to me, implementing this aura will not allow using other abilities - now only need a Dev to say if the fix is proper.

[LickedLurk]

Any opinion from Dev ?

[oc_redfox]

BuMp...?

A much wanted fix for the Overpowered warrior

[Maxxie]

I was told that it shouldn't prevent targets from auto-attacking...

Can someone confirm that?

[jorooo]

I think so.

Source: http://www.wowhead.com/?spell=46924#comments (search for "Auto")

[Maxxie]

In this case... Here there is a corrected version of the patch: http://pastebin.com/f2f627196

Thank you jorooo

[Sephiroth1983]

About bladestorm if someone wants to fix the first effect

Apply Aura: Add Creature Immunity (1733)

This should mean immunity to Charm, Disarm, Root, Pacify, Sleep and Snare effects.

[Maxxie]

How about fear? I tried to fix that too but I was told it makes immune to fear too, so I'm not sure if they are the correct effects

[Gommes]

it make you immune to ALL EFFECTS and only death can stop u.

so u are disarm also to fear and so on

[Tassader2]

it make you immune to ALL EFFECTS and only death can stop u.

so u are disarm also to fear and so on

All? Including School Damage effect?

[jorooo]

CC and Slow Effects only

[Gommes]

it should work like BEstial Wrath for hunter

[Tassader2]

it should work like BEstial Wrath for hunter

Not exactly.

According to wowhead comments bladestorm gives immunity to CC and disarm, but does not break such effects already on the target.

EDIT: oh I see: Patch 3.0.3 (11/4/08): Now breaks all snares and roots on the warrior when activated.

BW breaks CC as well as making target immune to new. And AFAIK BW does not provide disarm immunity.

[Gommes]

well u cant get disarm while Bladestorming. If u are disarmed already u acnt use bladestorm.

And Bladestorm removes ROOTS and Immobilize effects

Just tested on blizz.

[Maxxie]

According to wowhead comments bladestorm gives immunity to CC and disarm, but does not break such effects already on the target.

From 3.0.3 it should...

And AFAIK BW does not provide disarm immunity.

It does, it can't be cast if you are disarmed but makes you immune to it

[Sephiroth1983]

About bladestorm if someone wants to fix the first effect

Apply Aura: Add Creature Immunity (1733)

This should mean immunity to Charm, Disarm, Root, Pacify, Sleep and Snare effects.

Quoting again my post because you guys maybe don't understand... the first effect says which immunities it should grant, implementing the aura will fix them, is the same as creatures immunities.

If is proven there should be some additional effect immunity like fear, only that needs to be hardcoded.