Skip to content

Commit

Permalink
Remove serializer option for now
Browse files Browse the repository at this point in the history
  • Loading branch information
driesvints committed Feb 10, 2020
1 parent 6088728 commit 9e20ec6
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 12 deletions.
16 changes: 4 additions & 12 deletions src/Illuminate/Redis/Connectors/PhpRedisConnector.php
Original file line number Diff line number Diff line change
Expand Up @@ -98,10 +98,6 @@ protected function createClient(array $config)
$client->setOption(Redis::OPT_READ_TIMEOUT, $config['read_timeout']);
}

if (! empty($config['serializer'])) {
$client->setOption(Redis::OPT_SERIALIZER, $config['serializer']);
}

if (! empty($config['scan'])) {
$client->setOption(Redis::OPT_SCAN, $config['scan']);
}
Expand Down Expand Up @@ -160,16 +156,12 @@ protected function createRedisClusterInstance(array $servers, array $options)
$client->setOption(RedisCluster::OPT_PREFIX, $options['prefix']);
}

if (! empty($options['serializer'])) {
$client->setOption(RedisCluster::OPT_SERIALIZER, $options['serializer']);
}

if (! empty($config['scan'])) {
$client->setOption(RedisCluster::OPT_SCAN, $config['scan']);
if (! empty($options['scan'])) {
$client->setOption(RedisCluster::OPT_SCAN, $options['scan']);
}

if (! empty($config['failover'])) {
$client->setOption(RedisCluster::OPT_SLAVE_FAILOVER, $config['failover']);
if (! empty($options['failover'])) {
$client->setOption(RedisCluster::OPT_SLAVE_FAILOVER, $options['failover']);
}
});
}
Expand Down
1 change: 1 addition & 0 deletions tests/Redis/RedisConnectionTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,7 @@ public function testItAddsMembersToSortedSet()
$this->assertEquals(0, $redis->zadd('set', 'XX', 'CH', ['arya' => 11]));

$redis->zadd('set', ['mohamed' => 100]);
dump($redis->zscore('set', 'mohamed'));
$this->assertEquals(100, $redis->zscore('set', 'mohamed'));

$redis->flushall();
Expand Down

0 comments on commit 9e20ec6

Please sign in to comment.