Yeah, they do ignore collision, and that's okay, not problem there, but any unit with Wind Walk buff can't go though Terrain, structures, doodads and similar pathing things, so do not air units and then isn't a basically air when you can't go over cliff.
Oh neat, I wasn't aware of this. I see, then yes that's a problem.
And I didn't understood you "Or perhaps you could add and remove "permanent invisibility" while using berserk for the movement speed.", so if you will to explain it.
permanent invisibility is a passive ability without an icon and without buff icon (at least iirc) that makes the unit invisible, it's normally found on Far Seer's summoned spirit wolves.
Because it's automatic and has no icons, the idea is that it would allow another ability like berserk to be used for both the buff and the movement speed (as well as being the visible activate-able ability on the unit's command card).
So when a unit uses your custom Berserk ability, you could simply add this ability to that unit.
And how will I remove bersek buff when I break invisibility? Unit - Attack / Finishes casting effect of an ability?
I didn't think much about it though since it was just an afterthought, so I didn't consider that neither berserk nor permanent invisibility would disappear when the unit attacks.
If the unit is a melee attacker, then using a damage detection system would work. But it wouldn't be as easy with a ranged attacker, so it might not be an option.
On
Producer Update: Natives List this event is listed:
JASS:
constant unitevent EVENT_UNIT_DAMAGE_RELEASE
//This triggers when a unit's attack is released, aka when a melee unit hits its target or a ranged unit launches a missile.
constant unitevent EVENT_UNIT_MOVES
So maybe it exists, but I checked with the world editor on latest patch, and it's also not listed here
common.j -- patch 1.30.4 | HIVE so presumably it either doesn't exist or it's on ptr or something.
But even if it doesn't exist, it might exist in a future patch.
As for if you're using a custom sorceress' "invisibility", that invisibility is broken when a unit attacks, so you only need to check for if the unit loses the invisibility buff and then also remove berserk's buff (check with periodic event).