Relic shouldn't proc on Retaliation

[dowzer]

Checklist

• [x] I have searched for existing issues for issues like this one. The issue has not been posted.
• [x] I have provided reproducible steps. (No "as the title says" posts please. Please try to state facts only, and avoid assumptions/opinions.)
• [ ] I have provided my client version in the details. (type /ver into your game window)
• [x] I have provided proof of my claim (era videos, links to era wiki and/or forum posts) Lack of proof will slow down fixes and prioritization. Please provide details so that developers' research time is minimized.

Details

Unless mechanics have changed, it doesn't proc on Retaliation with the current lv99 version of Bravura. The variation in damage appears to be a normal hit and a critical hit.

https://www.youtube.com/watch?v=clV-ega-5WE

[KnowOne134]

has this occured to anyone yet, trace back of code i dont think its possible. uses CheckForDamageMultiplier() in attackutils.cpp with a bool of firstSwing of an attack round as only time for a proc.

[ConteAlmaviva]

Retaliation doesn't happen as part of a player's standard attack round, though, it's in response to an attack from a target. Would that cause it to be evaluated as a firstSwing?

[KnowOne134]

Someone sent me a screenshot of damage with a retaliation high dame seemingly to be double damage.

I was more asking if this was happening or not as I wasn't sure by the report.