Skript: Integers in-between (reverse)

[ShaneBeee]

So checking numbers between a low and high value (between 0 and 2) works
Checking numbers between high and low value (between 2 and 0) do not work

Example:

command /test:
    trigger:
        if 1 is between 0 and 2:
            send "1 is between 0 and 2"
        if 1 is between 2 and 0:
            send "1 is between 2 and 0"

The first part (between 0 and 2) sends the message
The second part (between 2 and 0) does not

[iOshawott]

the same situation when we are talking about negative numbers, but now will work only
"-1 is between -2 and 0"

and it's completely logical and fine for me because the correct number order is -2,-1,0,1,2,3
and when we are reading there's no 1 between 2 and 0 because after 2 there isn't 1 and 0

[RoboBlawck]

I have been working on comparison functions for 2 hours...now I realize there were already there in the documentation...

[Nicofisi]

I mean, I can fix this but I need someone to confirm that this is supposed to work lol

[ShaneBeee]

@Snow-Pyon Felt that it should be fixed.... so I guess that is your confirmation ;)

[Nicofisi]

Sure then, right after @bensku merges one of the 'x times' PRs

[ShaneBeee]

Sure sure... no rush... id rather see a working version of 1.13 skript vs having a tiny little problem like this solved.... id say priority: lowest

[Nicofisi]

Priority low since I'm interested in fixing this myself, when a dev is assigned they are free to choose the priority as long as it makes any sense, after all they are supposed to take care of the thing themselves :) and also this should take only a moment. But you're right here in general

[ShaneBeee]

Well thank you for fixing it ... I do appreciate it :) 👍 for nico

[TheLimeGlass]

I fixed numbers between %number% and %number% to work in reverse, so you can use that until then, but there is a comparator or the ExprBetween for checking between two objects (like locations and such as well).

Negative numbers aren't suppose to work, it returns null. There was already a debate about adding support for negative numbers when I fixed the numbers between, I personally thought it would be a great addition, but the other developers disagreed for some reason, like always. Look where we are now, having people ask if negative numbers work... Some times man, Some times...

if numbers between 2 and 1 contains {_input}:
    #
else if numbers between 1 and 2 contains {_input}:

[Snow-Pyon]

Negative numbers should work with this condition already, since the check supports it.

[Romitou]

This issue has been resolved in #3248.

[APickledWalrus]

This issue has been resolved in #3248.

Yep. Missed then when I opened that PR.