Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ZADD: allowing multiple member-score pairs #40

Closed
nleroy opened this issue Sep 16, 2015 · 4 comments
Closed

ZADD: allowing multiple member-score pairs #40

nleroy opened this issue Sep 16, 2015 · 4 comments

Comments

@nleroy
Copy link

nleroy commented Sep 16, 2015

Hi

Right now I'm using Predis as redis library. The ZADD function is defined this way:
zadd($key, array $membersAndScoresDictionary)

When using RedisMock in my unit tests, I get an error stating that argument 3 is missing for zadd. (as ZADD is defined as zadd($key, $score, $member)).

Does it seem logical to add this change? Or do other PHP Redis library implment zadd differently?

@nleroy
Copy link
Author

nleroy commented Sep 16, 2015

By the way: I see SADD already allows multiple members.

@nleroy
Copy link
Author

nleroy commented Sep 17, 2015

I'm sorry, apparently it is possible to use zadd the same way in Predis as defined in RedisMock. I changed my code and it works.

@nleroy nleroy closed this as completed Sep 17, 2015
@omansour
Copy link
Contributor

no problem that's for the best :)

don't hesitate to report any problem in the future.

@nleroy
Copy link
Author

nleroy commented Sep 17, 2015

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants