Skip to content
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

Core/Creature: Implement aggro grace period #30362

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mdX7
Copy link
Member

@mdX7 mdX7 commented Oct 18, 2024

Issues addressed:
ref #26528
supercedes #30273

Tests performed:
creatures tested: 226909, 223799
tested db spawned & temp summoned creatures

  • run into aggro range of creature after heartbeat triggered => aggro
  • kill & respawn creature while camping at respawn spot (not moving) => aggro after grace period
  • kill & respawn creature while camping at respawn spot (moving) => aggro after grace period
  • kill & respawn creature while camping at respawn spot + attacking while grace period didnt expire yet => instant aggro

@Ovahlord
Copy link
Contributor

aggro grace period should also get reset when engaging (Unit::AtEngage) so when a player decides to pull right away, this behavior wont remain dangling around

@Nyr97
Copy link
Contributor

Nyr97 commented Oct 19, 2024

Does this force non-moving creatures that should be sparring with each other to aggro each other? When I was working on Silverpine, I had to force random movement from DB by 1.f for them to detect each other. Asking so that I might remove that small hackfix I had to use back 2 years ago.

@mdX7
Copy link
Member Author

mdX7 commented Oct 19, 2024

Does this force non-moving creatures that should be sparring with each other to aggro each other? When I was working on Silverpine, I had to force random movement from DB by 1.f for them to detect each other. Asking so that I might remove that small hackfix I had to use back 2 years ago.

Yes, MoveInLineOfSight triggers the aggro behavior and is manually triggered after the grace period in this pr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants