diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

minor #42065 [Messenger] clarify RedisStore::evaluate() (nicolas-grekas)

Browse Source 
This PR was merged into the 5.2 branch.

Discussion
----------

[Messenger] clarify RedisStore::evaluate()

| Q             | A
| ------------- | ---
| Branch?       | 5.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Commits
-------

450d33ff3b [Cache] clarify RedisStore::evaluate()
This commit is contained in:
Nicolas Grekas 2021-07-12 14:35:28 +02:00
commit d432821bf6
1 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/Symfony/Component/Lock/Store/RedisStore.php
View File

@ -281,19 +281,15 @@ class RedisStore implements SharedLockStoreInterface
}
return $result;
return $this->redis->_instance($this->redis->_target($resource))->eval($script, array_merge([$resource], $args), 1);
}
if ($this->redis instanceof \Predis\ClientInterface) {
try {
return $this->redis->eval(...array_merge([$script, 1, $resource], $args));
} catch (ServerException $e) {
throw new LockStorageException($e->getMessage(), $e->getCode(), $e);
}
}
\assert($this->redis instanceof \Predis\ClientInterface);
throw new InvalidArgumentException(sprintf('"%s()" expects being initialized with a Redis, RedisArray, RedisCluster or Predis\ClientInterface, "%s" given.', __METHOD__, get_debug_type($this->redis)));
try {
return $this->redis->eval(...array_merge([$script, 1, $resource], $args));
} catch (ServerException $e) {
throw new LockStorageException($e->getMessage(), $e->getCode(), $e);
}
}
private function getUniqueToken(Key $key): string