Define prefix strategy for ZMSCORE command

[mohorev]

Description:

When using Predis, if a client prefix is set and then a ZMSCORE command is executed, the client prefix is ignored. This leads to incorrect results, as the command operates on keys without the applied prefix.

Steps to Reproduce:

• Set a prefix for the Predis client.
• Use the ZMSCORE command to retrieve the score from a sorted set.

Expected Behavior:

The ZMSCORE command should consider the client prefix when retrieving the score from the sorted set, ensuring that the correct key is used.

Actual Behavior:

The ZMSCORE command retrieves the score from the sorted set without considering the client prefix. As a result, incorrect keys are accessed, leading to unexpected behavior.

Additional Information:

Predis Version: v2.2.2

Similar issue with prefix problems in ZPOPMIN & ZPOPMAX commands #1432 already reported.

Sample Code:

<?php

require __DIR__ . '/vendor/autoload.php';

// Set up Predis client with a prefix
$redis = new Predis\Client('tcp://localhost:6383', ['prefix' => 'test_']);

// Add values to a sorted set
$redis->zadd('mykey', ['foo' => 10, 'bar' => 20]);

// ZRANGE works correct, return: ['foo' => '10', 'bar' => '20']
var_dump($redis->zrange('mykey', 0, -1, ['WITHSCORES' => true]));

// ZMSCORE ignores the prefix. Outputs incorrect scores: [null, null]
var_dump($redis->zmscore('mykey', 'foo', 'bar'));

[coveralls]

coverage: 80.258% (+0.01%) from 80.247%
when pulling 74ddd24 on mohorev:zmscore-prefix
into 45b18eb on predis:v2.x.

[mohorev]

@vladvildanov @tillkruss Hey team, can we release with these changes? This code is my salvation in production. I'd be praying for you in church if you help speed up this release 😺

[mohorev]

To kill two birds with one stone, I decided to go further, and within the scope of this pull request, I also fixed the prefix issue with the ZPOPMAX and ZPOPMIN commands, which was mentioned in a neighboring pull request. However, unlike that solution, I added unit tests here that verify the functionality of the prefixes.

[tillkruss]

Please undo all the styling changes.

[mohorev]

@tillkruss Done. As per your request, I've reverted the commit regarding the styling formatting.

[vladvildanov]

@mohorev Thanks for your input! I added a PR to fix codestyle issues and bind a version of php-cs-fixer package. After it will be merged please rebase your branch and we will release both of this changes

#1445

[mohorev]

Hi, @vladvildanov @tillkruss

Just wanted to check in to see if there are any updates or progress on this enhancement?

[tillkruss]

I've merged this in #1451 without the test changes.

[mohorev]

 Thanks!