New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[3.3.5][6.x]Core/Unit: Do not remove auras from Pets and Guardians after evading #16503
Conversation
Rushor
commented
Feb 8, 2016
- this adds free space for further adjustments
- this fixes [3.3.5][6.x] Guardians evade issue #14347
- originally work by @mik1893
this also fixes following bug:
result: ghoul will evade and blind will be removed. blind should never be removed. this pr fixes that issue |
@@ -259,7 +259,7 @@ bool CreatureAI::_EnterEvadeMode(EvadeReason /*why*/) | |||
|
|||
// don't remove vehicle auras, passengers aren't supposed to drop off the vehicle | |||
// don't remove clone caster on evade (to be verified) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now these comments possible should be moved to Unit.cpp
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
* this adds free space for further adjustments * this fixes #14347
@@ -4113,6 +4113,16 @@ void Unit::RemoveArenaAuras() | |||
} | |||
} | |||
|
|||
void Unit::RemoveAurasOnEvade() | |||
{ | |||
if (IsCharmedOwnedByPlayerOrPlayer()) // if it is a player owned creature it should not remove the aura |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any news on this? would you like to improve sth. here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me™. I am fairly sure @ccrs meant it should look like this.
@Rushor is this PR ready to be merged ? |
yep, i tested this since 2 months now - i didn't found any new issues with it |
merge it then please |
[3.3.5][6.x]Core/Unit: Do not remove auras from Pets and Guardians after evading (cherry picked from commit e869ccb)