Fixes #14. Locking does not work correctly when a serialization mode is enabled with the Redis extension.

Author: willemstuursma

classes/mutex/PHPRedisMutex.php

@@ -62,6 +62,17 @@ protected function add($redis, $key, $value, $expire)
     protected function evalScript($redis, $script, $numkeys, array $arguments)
     {
         /** @var Redis $redis */
+
+        /*
+         * If a serializion mode such as "php" or "igbinary" is enabled, the arguments must be serialized but the keys
+         * must not.
+         *
+         * @issue 14
+         */
+        for ($i = $numkeys, $iMax = \count($arguments); $i < $iMax; $i++) {
+            $arguments[$i] = $redis->_serialize($arguments[$i]);
+        }
+
         try {
             return $redis->eval($script, $arguments, $numkeys);
         } catch (RedisException $e) {

tests/mutex/PHPRedisMutexTest.php

@@ -77,4 +77,25 @@ public function testEvalScriptFails()
             $this->redis->close();
         });
     }
+
+    /**
+     * @param $serialization
+     * @dataProvider dpSerializationModes
+     */
+    public function testSyncronizedWorks($serialization)
+    {
+        $this->redis->setOption(Redis::OPT_SERIALIZER, $serialization);
+
+        $this->mutex->synchronized(function () {
+            $this->assertTrue(true);
+        });
+    }
+
+    public function dpSerializationModes() {
+        return [
+            [Redis::SERIALIZER_NONE],
+            [Redis::SERIALIZER_PHP],
+            [Redis::SERIALIZER_IGBINARY],
+        ];
+    }
 }