[Crash] Bugfix for zone crash caused by #flymode -1

[ghost]

Bugfix for zone crash caused by #flymode when invalid values are passed. Also, when #flymode is typed without parameters, the correct syntax is displayed.

[ghost]

Should just atoi the separator argument and check if >= 0 and <= 5.

[ghost]

Should just atoi the separator argument and check if >= 0 and <= 5.

I have tried to find the smallest possible change to fix the issue.

[mackal]

Should just atoi the separator argument and check if >= 0 and <= 5.

I agree with this

Should just atoi the separator argument and check if >= 0 and <= 5.

I have tried to find the smallest possible change to fix the issue.

I think just checking IsNumber, atoi, etc is a cleaner fix. Which is better.

[ghost]

Should just atoi the separator argument and check if >= 0 and <= 5.

I have tried to find the smallest possible change to fix the issue.

Edit: You'll also need to flip the logic, put the message as the else and this as the part that sends the command.

if (strlen(sep->arg[1]) == 1 && sep.IsNumber(1) && atoi(sep->arg[1]) >= 0 && atoi(sep->arg[1]) <= 5))

[ghost]

^ is the best way to make that code smaller the original is pretty awful.

Understood and changed as requested. The PR is tested on my server and works for me. I still have some questions but I will ask them in Discord.

[ghost]

Looks good to me. 👍