Skip to content

fix(Scripts/IcecrownCitadel): fix Valithria Dreamwalker portals not spawning correctly#25600

Open
mingzi120 wants to merge 2 commits intoazerothcore:masterfrom
mingzi120:fix-valithria-portals-clean
Open

fix(Scripts/IcecrownCitadel): fix Valithria Dreamwalker portals not spawning correctly#25600
mingzi120 wants to merge 2 commits intoazerothcore:masterfrom
mingzi120:fix-valithria-portals-clean

Conversation

@mingzi120
Copy link
Copy Markdown
Contributor

@mingzi120 mingzi120 commented Apr 26, 2026

Description

Fixes a regression introduced in PR #23477 where Valithria Dreamwalker fails to spawn portals correctly in all difficulties (10/25/Heroic/Normal).

Root cause:

  • Using false for CastSpell caused subsequent portals in the for loop to fail to cast due to the recently added strict cast validation in Core/Spells, preventing the remaining portals from spawning in 10-man mode, and completely breaking them in 25-man mode.

This fix sets the triggered flag to true to ensure all portals spawn correctly in a single loop, and relies on the core's spelldifficulty_dbc system to handle the spell ID automatically.

@github-actions github-actions Bot added Script Refers to C++ Scripts for the Core file-cpp Used to trigger the matrix build labels Apr 26, 2026
@Kitzunu
Copy link
Copy Markdown
Member

Kitzunu commented Apr 26, 2026

Use spelldifficulty, that eliminates the need to check for heroic

@mpfans
Copy link
Copy Markdown
Contributor

mpfans commented Apr 26, 2026

#25516
This PR has fixed the bug where the door does not refresh

@mingzi120 mingzi120 force-pushed the fix-valithria-portals-clean branch from 7a38c75 to 02e2031 Compare April 27, 2026 01:45
@mingzi120 mingzi120 force-pushed the fix-valithria-portals-clean branch from 02e2031 to 7ca2d43 Compare April 27, 2026 01:49
@mingzi120 mingzi120 changed the title fix(Scripts/IcecrownCitadel): fix Valithria Dreamwalker portals missing in Heroic and 25-man modes fix(Scripts/IcecrownCitadel): fix Valithria Dreamwalker portals not spawning correctly Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

file-cpp Used to trigger the matrix build Script Refers to C++ Scripts for the Core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants