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

[Stun][Diminishing returns][Critical] #1916

Closed Humsas closed 11 years ago

Humsas commented 12 years ago

Theory:

Wowpedia:

Stuns cast by players will trigger diminishing returns. Consequently, after three stuns (not to be confused with disorients) within a short period of time, the target will become temporarily immune. For example, if the mage ability [Impact] is triggered more than three times in a short period of time, the target will show immunity towards a fourth Impact. There are however several diminishing returns categories, one for activated stuns, which include most stun effects, and one for proc stuns, which include the stuns from procs, as well as one specifically for Cheap Shot. Using stuns in one category will not affect the duration of stuns in the another category. Also, unlike many crowd control effects, most stuns suffer diminishing returns even in PvE encounters.

So there are 3 DR categories:

Spell and their categories list: http://www.wowpedia.org/index.php?title=Diminishing_returns&oldid=1580531 Also stuns from items like http://www.wowwiki.com/Deep_Thunder should be under "Proc stuns" category.


Tested these spells/items/talents and there are bugs that I found:

To test you will need 2x clients.

1 case: Current situation: Intimidation (Hunter pet) doesn't share DR with other stuns from Activated stuns category.

Create Tauren Hunter: .levelup 69 .l 12809 (Concussion Blow) .l 19577 (Intimidation) .l 1515 .m hp 9999999 .m rage 9999999 .t sw .go c ravager Capture ravager. (.npc tame) .t sw

Create Gnome Rogue: .levelup 69 .m hp 9999999 .t sw

With Hunter set pet to attack Rogue, then cast Concussion Blow on him and then Intimidation.

Result: Intimidation has full stun length. Desired result: Intimidation should share DR with other stuns from "Activated stuns" category.

2 case: Current situation: Weapon Deep Thunder stun doesn't share DR with other stuns from "Proc stuns" category.

Tauren Hunter: .l 15269 (Blackout stun effect) .l 34510 (Deep Thunder stun effect)

With Warrior cast Blackout on Rogue and then Deep Thunder stun(Stun).

Result: Deep Thunder stun has full stun length. Desired result: Deep Thunder should share DR with other stuns from "Proc stuns" category.

Humsas commented 12 years ago

1914

Mofoleone commented 12 years ago

DR works on Blackout (Priest talent), tested.

programuotojasgf commented 12 years ago

I think the testing methods, that used the spells directly, instead of them being triggered affected the results.

Either way:

Fixed, now, no matter by witch method, the DR will be correct

Humsas commented 12 years ago

Fix denied.

Intimidation casted by Hunter pet still does not share DR with other stuns from Activated stuns category. Other issue part is working.

Databasegm commented 11 years ago

Fix confirmed.

Intimidation casted by Hunter pet now shares DR with other stuns from Activated stuns category.

Mofoleone commented 11 years ago

Diminishing return does NOT apply atm for warrior mace stun, and it should. You can get stunlocked up to 10 seconds or more by any warrior.

Databasegm commented 11 years ago

This fix is not Live yet. Please refrain from further reports until it is.

Sideways commented 11 years ago

This has been applied to live in Patch 7.2.

Mofoleone commented 11 years ago

RE-open please.

Mace stun spec is not affected by DR.