Jump to content

[fix] Shield Slam additional damage cap

Auntie Mangos

Recommended Posts

PATCH: http://pastebin.com/pzPnMtJ4

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

- it adds two caps for additional dmg provided by player's shield block value. basic cap: 34.5 * players_level, extended cap is after using Shield Block, which doubles the BV and the cap.

- For which repository revision was the patch created?

- 9763

- Is there a thread in the bug report section or at lighthouse? If yes, please add a link to the thread.

- http://getmangos.eu/community/showthread.php?13853-Shield-Slam

- Who has been writing this patch? Please include either forum user names or email addresses.

- only me

It's my first patch so dont be harsh on me, pls ;p i dont even know if i've posted in the right place... :rolleyes:

EDIT: small correction in code, it should be fine now.

Link to comment
Share on other sites

  • 40 years later...

Ad.1 double calculation is when warrior uses Shield Block, which doubles the cap, thats why i add doubled cap to the output damage.

If you dont use Shield Block - the cap is 34.5*level

If you use Shield Block - the cap is 69*level

So in other words, there are 2 caps, which depend on Shield Block skill.

Ad.2 30*level is the value where Diminishing Returns start to occur, every next SSlam would hit for lower dmg. This patch fixes Hard Cap for additional dmg from Block Value. Maybe I will also check the diminishing returns and the formula for counting BV from bonuses/talents (which is absolutely wrong right now), but right now its only the cap patch.

Helpful links:



EDIT: I did some tests and it looks like Block Value is counted properly, sorry for that accusation ;) Testers say that Shield Slam dmg is far too high, so I am trying to find the issue, but now it looks like the cap is the only one... Maybe % bonuses to the output dmg are wrong?

EDIT2: (I found out that Shield Mastery and Shield Block should both modify the base SBV only, but now Shield Block multiplies SBV already modified by Shield MAstery and this is the issue.)

Link to comment
Share on other sites

This topic is now closed to further replies.
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. Privacy Policy Terms of Use