mirror of
https://github.com/Kelsidavis/WoWee.git
synced 2026-08-03 11:29:22 -04:00
d32667192f
castSpell dismounts before casting, so crafting and the like work while mounted. For a mount spell that meant the button dismounted the player and then immediately put them back on the same mount, which reads as the button doing nothing at all. Retail dismounts and stops there. Recognise the case and return after the dismount. Pre-WotLK mounts are items rather than spells, so dispatchUseItem gets the same rule — it already resolves the item's on-use spell, and both bag and backpack use funnel through it, so a mount used from a bag behaves the same as one on the bar. This needs the mount's spell id to be right, and it was only a guess: on mounting, the client scanned for an indefinite self-cast aura and kept the last match, which is as likely to be a racial or a tracking buff. It now prefers the spell the player just cast or the item they just used, falling back to the old scan when neither matches.