fix(Core/Unit): Notify creature pets when owner enters combat via assist#25327
Open
blinkysc wants to merge 1 commit intoazerothcore:masterfrom
Open
fix(Core/Unit): Notify creature pets when owner enters combat via assist#25327blinkysc wants to merge 1 commit intoazerothcore:masterfrom
blinkysc wants to merge 1 commit intoazerothcore:masterfrom
Conversation
When a creature enters combat via aggro chain or assist (not direct damage), its controlled creatures (guardians/pets) were not notified, causing them to stand idle while their owner fights. Closes chromiecraft/chromiecraft#9228
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes Proposed:
This PR proposes changes to:
AI-assisted Pull Requests
Important
While the use of AI tools when preparing pull requests is not prohibited, contributors must clearly disclose when such tools have been used and specify the model involved.
Contributors are also expected to fully understand the changes they are submitting and must be able to explain and justify those changes when requested by maintainers.
Issues Addressed:
SOURCE:
The changes have been validated through:
Tests Performed:
This PR has been:
How to Test the Changes:
.go creature id 11882— teleport to Twilight Stonecaller in SilithusKnown Issues and TODO List:
DealDamageandAttack)Description
When a creature enters combat via assist or aggro chain (as opposed to being directly attacked), its controlled creatures (guardians/pets summoned via spells like Summon Earth Elemental) were never notified. The existing
OwnerAttackedBynotification only fires fromDealDamage— meaning only the directly attacked creature's pets would assist.This adds an
OwnerAttackedBycall inCreature::AtEngageso that whenever a creature enters combat for any reason, its controlled creatures are informed about the engage target and can assist.