fix(Core/Scripts): Fix Ragnaros combat evade after Wrath of Ragnaros knockback#25326
Open
blinkysc wants to merge 1 commit intoazerothcore:masterfrom
Open
fix(Core/Scripts): Fix Ragnaros combat evade after Wrath of Ragnaros knockback#25326blinkysc wants to merge 1 commit intoazerothcore:masterfrom
blinkysc wants to merge 1 commit intoazerothcore:masterfrom
Conversation
…knockback Adapt Magma Blast to a regular repeating timer (matching TrinityCore), removing the CanAIAttack melee restriction and custom EnterEvadeMode that were incompatible with the new threat system. Co-Authored-By: offl <[email protected]>
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 xyz 838.31 -831.47 -232.19 409.instance setbossstate 8 3.npc add temp 12018.npc set faction 1080.npc set flag 1Known Issues and TODO List:
Description
The old Ragnaros script used
CanAIAttackto restrict valid targets to melee range, then interceptedEnterEvadeModeto cast Magma Blast when no melee targets existed. This design became incompatible with the new threat system port, which usesCanAIAttackto determine threat entry online/offline status. When all players were knocked back:CanAIAttackreturned false for everyone → all threat entries marked offlineIsThreatListEmpty()(default: excludes offline) returned true → Ragnaros fully evadedSelectTarget/DoCastRandomTargetskipped offline entries → Magma Blast couldn't find targetsThe fix adapts TrinityCore's approach: Magma Blast is now a regular repeating event (4s) that checks
IsWithinMeleeRangebefore casting.CanAIAttackand the customEnterEvadeModeare removed entirely, so the threat system operates normally.