FeenixServerProject / Archangel_2.4.3_Bugtracker

This is the official Issue tracker for the Archangel 2.4.3 realm https://www.wow-one.com
78 stars 74 forks source link

[Warrior][Second Wind] #1961

Closed Agarwen closed 11 years ago

Agarwen commented 12 years ago

Frostbite/entrapment is a immobilize effect like improved hamstring/charge and more.

Second Wind proc when warrior have stun/immobilize effect.

http://www.wowwiki.com/Second_Wind_%28warrior_talent%29

http://imageshack.us/photo/my-images/196/wowscrnshot102812215030.jpg/

Darel1 commented 12 years ago

I agree immobilize effects such as Entrapment, Frostbite, Improved Harmstring should give the warrior Second Wind

Humsas commented 12 years ago

Current situation: Warrior talent Second Wind does not proc from talents which adds rooting effects to skills: Improved Wing Clip, Improved Hamstring, Entrapment and maybe some more.

To test you will need 2x clients. Create Tauren Warrior: .levelup 69 .learn 29838 (Second Wind) .t sw

Create Night Elf Hunter: .levelup 69 .l 27633 (Wing Clip) .l 19233 (Improved Wing Clip) .t sw

Now use Wing Clip on Warrior until he gets rooted.

Result: Warrior will not get Second Wind buff. Desired result: Warrior should gain Second Wind talent buff if he gets rooted or stunned from proc roots or stuns.

programuotojasgf commented 12 years ago

Fixed, soon to be on PTR

Humsas commented 12 years ago

Fix denied.

Second Wind does not proc from Hunters Entrapment talent. It should work with all effects that immobilize Warrior.

programuotojasgf commented 12 years ago

Please list all those spells, that can trigger Second Wind

Databasegm commented 12 years ago

Some examples of ROOT spells: Frost Nova (id 122) Entangling Roots (id 339) Entrapment (id 19185) Feral Charge effect (id 45334) Frostbite (id 12494)

Some examples of STUN effects: Starfire Stun (id 16922) Bash (id 8983) Blackout (id 15269) Impact (id 12355) Pyroclasm (id 18093)

Humsas commented 12 years ago

When I tested it was not working with these talents: Improved Wing Clip, Improved Hamstring, Entrapment. (Will check if there are more)

programuotojasgf commented 12 years ago

Fixed for Improved Hamstring and Entrapment.

Database - did you check your root and stun spells before posting that? They work fine for me :/

Agarwen commented 12 years ago

Maim,Nature's Grasp,Frostbite.

Humsas commented 12 years ago

Fix denied.

Maim is not stun.

Nature's Grasp, Frostbite does not trigger Second Wind. Second Wind and Entangling Roots trigger Second Wind after each damage tick. Should only trigger when Warrior is affected by them for the first time.

Sideways commented 12 years ago

Why fix denied? It was not applied to the PTR yet...

Humsas commented 12 years ago

It is denied as it didn't fixed everything. Also Gediminas agreed with that.

programuotojasgf commented 11 years ago

Nature's grasp was working fine for me Fixed FrostBite "Second Wind and Entangling Roots trigger Second Wind after each damage tick. Should only trigger when Warrior is affected by them for the first time." this is a database issue

Once you confirm Nature's grasp and FrostBite, the database issue with procing on each entangling roots tick should be forwarded to someone working the database

warlee commented 11 years ago

Hello, I tested it 20-12-12 on PTR. This is the list of all the kinds of crowd control that Second Wind should proc from.

MAGE Frost Nova - CONFIRMED Frostbite - CONFIRMED Impact - CONFIRMED

DRUID Pounce - CONFIRMED Bash - CONFIRMED Feral Charge - CONFIRMED Celestial Focus (Starfire stun) - CONFIRMED Nature's Grasp (16689) - FIX DENIED; It procs on each damage tick. What is more, it doesnt proc when warrior is actually immobilized (so warrior has no second wind for few first moments, then when the damage procs he gains second wind) Entangling Roots (339) - FIX DENIED; It procs on each damage tick, warrior should gain the Second Wind when he is affected by roots immobilizing effect, not the dot.

WARLOCK Pyroclasm - CONFIRMED Intercept (felguard ability) - CONFIRMED Summon Infernal - CONFIRMED Shadowfury - CONFIRMED

HUNTER Freezing Trap(1499) - FIX DENIED Entrapment(19185) - FIX DENIED Improved Wing Clip - CONFIRMED Intimidation (Hunter pet) - CONFIRMED Improved Concussive Shot - Report MISSING (https://github.com/FeenixServerProject/Archangel_2.4.3_Bugtracker/issues/2197)

PRIEST Blackout - CONFIRMED Chastise - CONFIRMED

ROGUE Cheap Shot - CONFIRMED Kidney Shot - CONFIRMED Mace Specialization - CONFIRMED

SHAMAN Stoneclaw Totem - CONFIRMED

PALADIN Hammer of Justice - CONFIRMED Seal of Justice - CONFIRMED

WARRIOR Mace Stun Specialization(talent) - CONFIRMED Improved Hamstring(24428) - FIX DENIED Charge - CONFIRMED Intercept - CONFIRMED

OTHERS War Stomp(racial) - CONFIRMED Deep Thunder - CONFIRMED Stormherald - CONFIRMED Brutal Earthstorm Diamond(meta gem) - CONFIRMED

Anelith commented 11 years ago

Fix confirmed on PTR. Second Wind now procs from Improved Hamstring, Freezing Trap, Entrapment, Entangling Roots and Nature's Grasp