search

BedrockStreaming / RedisMock

A simple PHP Redis mock
http://tech.m6web.fr/redismock-qui-a-bouchonne-mon-redis.html
MIT License
81 stars 60 forks source link

lrem() function - parameters order fixed #90

Closed alex-scott closed 4 years ago

alex-scott commented 4 years ago

Seems predis has different parameters order than REDIS server. Just fixed function to match PHP extension

Oliboy50 commented 4 years ago

thanks @alex-scott :wave:

could you link some documentation about this, please?

alex-scott commented 4 years ago

thanks @alex-scott

could you link some documentation about this, please?

Sure https://github.com/phpredis/phpredis#lrem-lremove

Oliboy50 commented 4 years ago

thanks 👌

indeed, it seems to be a "predis only" stuff:

Note: The argument order is not the same as in the Redis documentation. This difference is kept for compatibility reasons

but since we've put the following statement in our documentation:

Work for now only with predis

maybe we should follow up with the predis parameters order too

Oliboy50 commented 4 years ago

thanks @alex-scott => https://github.com/M6Web/RedisMock/releases/tag/v5.0.0