Skip to content

PHP7 - sInterStore no longer accept an Array of keys #748

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

Closed
giviz opened this issue Feb 7, 2016 · 2 comments
Closed

PHP7 - sInterStore no longer accept an Array of keys #748

giviz opened this issue Feb 7, 2016 · 2 comments

Comments

@giviz
Copy link

giviz commented Feb 7, 2016

With PHP 5.6 I'm able to do :

$redis->sInterStore(['key1', 'key2', 'key3']);

But with PHP 7.0, that generate an error : Wrong parameter count for Redis::sInterStore()

We are now forced to pass each key as a parameter.

Is this a deliberate choice or a bug ?
Could you correct this, or will that stay this way in the next future ?

We use arrays like this a lot, so we cannot move to PHP7 until we pass this (or correct our code if it's a deliberate choice of yours).

Thank you for your work !

@michael-grunder
Copy link
Member

Go ahead and try it now. I had the minimum argument count to sinterstore set to 2 arguments, but it could be one array with multiple arguments.

@giviz
Copy link
Author

giviz commented Jan 25, 2017

That issue as been reopened by an other commit.

Argument needs to be set from 2 to 1 again.

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

3 participants