Merge pull request #89 from clementtalleu/fix/predis_client

fix choice of redis client
clementtalleu · Sep 22, 2024 · 62b5827 · 62b5827
2 parents c722233 + 5761acb
commit 62b5827
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Om/Persister/AbstractPersister.php b/src/Om/Persister/AbstractPersister.php
@@ -16,7 +16,7 @@ public function __construct(
         private ?KeyGenerator $keyGenerator = null,
         protected ?RedisClientInterface $redis = null
     ) {
-        $this->redis = $redis ?? getenv('REDIS_CLIENT') === 'predis' ? new PredisClient() : new RedisClient();
+        $this->redis = $redis ?? (getenv('REDIS_CLIENT') === 'predis' ? new PredisClient() : new RedisClient());
 
         $this->keyGenerator = $keyGenerator ?? new KeyGenerator();
     }

diff --git a/src/Om/RedisObjectManager.php b/src/Om/RedisObjectManager.php
@@ -31,7 +31,7 @@ final class RedisObjectManager implements RedisObjectManagerInterface
     public function __construct(
         private ?RedisClientInterface $redisClient = null,
     ) {
-        $this->redisClient = getenv('REDIS_CLIENT') === 'predis' ? new PredisClient() : new RedisClient();
+        $this->redisClient = $redisClient ?? (getenv('REDIS_CLIENT') === 'predis' ? new PredisClient() : new RedisClient());
 
         $this->keyGenerator = new KeyGenerator();
     }