Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Sort command, limit parameter #11

Closed
Mikushi opened this Issue · 1 comment

2 participants

@Mikushi

Hi,

i found a little issue with the Sort command today, involving the "limit" parameter.

When passed a String as either offset or count (array("0","4")), the function will ignore the limit parameter altogether. It's easy to reproduce, just run a sort command and pass it an offset or count as a String, while running MONITOR on Redis and you'll the the LIMIT param is missing from the command.

Is it an expected behavior of this function?

Thanks.

@nicolasff
Owner

Hi,

Thanks for this report; the case was simply not handled. Strings and integral numbers are now supported.

I have added a unit test as well.

Nicolas

@nicolasff nicolasff closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.