Would there be a reason not to do this?

[Avocado]

I did this a while back after noticing and I’m curious why this wouldn’t work. Why would you need to put it before attack. You wouldn’t want stopAttack to stay triggered for any reason I can think of. I’m also not sure why we want to leave stopAttack on when you are running around.

I also found another bug (I thought this was the one he meant) where you can switch targets mid attack so you don’t move closer to the new one It’s referenced here too
https://www.udemy.com/course/unityrpg/learn/lecture/14035081#questions/8731306
Should I fix this or does it get fixed later?

[Brian_Trotter]

We don’t apply the fix mentioned later in the course.

In terms of the stopAttack, I actually go a different way…
In TriggerAttack, I use

anim.SetTrigger("Attack");
anim.ResetTrigger("StopAttack");

and in Cancel, I use

anim.SetTrigger("StopAttack");
anim.ResetTrigger("Attack");