Shauren

0a34290484f3e90a2a642986de682a8b0c7e3815

No, druid Dash is special (shapeshift requirements imply losing the aura when you shift out of the form, Dash doesn't do that and doesnt have a shapeshift requirement in spell...

Don't use std::extent anymore, replace with `std::size(path)`

I think this one is wrong, what about being rooted while in air?

No, spells are not fixed by randomly finding a similar named spell that does similar things. Heroic Leap does not need a spellscript at all

TODO: All hits made against a creature that cannot reach its target should end in "EVADE" message.

@LordUsagi CanReachTarget is complete garbage and does not check if creature can actually path to target

What are all those changes that put the source in target arg if there is no proper target?

https://github.com/TrinityCore/TrinityCore/blob/c3455595fcf498975b1b5c0b38e5aa467dbfdcbb/src/server/game/Spells/Spell.cpp#L5798-L5832

This one is without a doubt mariadb incompatibility