Skip to content

Scripts/Spells: Toggle skyriding and steady flight from Switch Flight Style - #32066

Open
Oreocide wants to merge 1 commit into
TrinityCore:masterfrom
Oreocide:pr/switch-flight-style
Open

Oreocide wants to merge 1 commit into
TrinityCore:masterfrom
Oreocide:pr/switch-flight-style

Conversation

@Oreocide

@Oreocide Oreocide commented Oct 15, 2026 •

Copy link
Copy Markdown

Summary

  • 404471 Change Flight Style and 436854 Switch Flight Style swap auras 404464 (Skyriding) and 404468 (Steady Flight), then call UpdateMountCapability.
  • Bound in 2026_10_14_01_world.sql.

Still needed

  • SpellEffect.db2 for 404471 and 436854. The handler is effect 0, SPELL_EFFECT_DUMMY. The hotfixes spell_effect table on this core is empty, and the effect rows were not loaded from DB2 here.

Test plan

  • Cast 436854 while 404468 is active and confirm it becomes 404464, and the reverse.
  • Cast it while mounted and confirm the mount capability follows the new aura.

… Style.

Steady Flight was flagged not saveable and login always reapplied it, so switching to Skyriding did not stick. Apply 404468 only when neither style aura was loaded, and do not call UpdateMountCapability from the toggle.
@Oreocide

Oreocide commented Oct 15, 2026 •

Copy link
Copy Markdown
Author

Gah, the toggle doesn't survive a logout. 404468 is still flagged as not saveable, and login always puts that aura back on, so you end up in steady flight again even if you had switched to 404464.

I'll remove that spell fix and only add 404468 when neither aura was loaded:

// 404464 Skyriding and 404468 Steady Flight are the saved flight-style auras.
if (!HasAura(404464) && !HasAura(404468))
    AddAura(404468, this);

}

// Mount capability is chosen from the flight-style aura.
caster->UpdateMountCapability();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While you can cast this spell in a druid flight form, finishing the cast removes the form on retail making UpdateMountCapability pointless here (and you cannot cast it while mounted)

@Oreocide
Oreocide force-pushed the pr/switch-flight-style branch 3 times, most recently from 20e9944 to 7c22ab1 Compare October 15, 2026 14:15
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.

2 participants