Random normal always random

[magralo]

Trying to solve #10164

This works at least for my usage!

You will need to select a target but the target will be reselected randomly in the backend,

Maybe not the best solution but this works for me... Hope this can merge with master

[DaWoblefet]

The way moves like Outrage currently work is like this:

When not locked into Outrage: send no target argument (i.e. /move outrage, other action).
When locked into Outrage: send either target as an argument (i.e. /move outrage +2, other action or /move outrage +1, other action).

The target ends up getting randomized no matter what here already. I don't think we should actually be doing it in the way you're doing so; you shouldn't need to send a target for randomNormal moves. The fix would be requiring that you don't supply a target, not that you do supply one.

[magralo]

Ok thanks!

... one question, though: Wouldnt it be better if the move never required a target?

[DaWoblefet]

Correct, that's what I'm saying. For randomNormal moves, we shouldn't accept a target argument. Right now, randomNormal moves are already being randomized - the target argument isn't being respected right now anyway, which is correct behavior. It should just throw an error if you try to provide one.