Skip to content

Commit 9ed72d1

Browse files
author
Matias Melograno
committed
returning Item when an exception occurred
1 parent 24e38e3 commit 9ed72d1

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/SplitIO/Component/Cache/Storage/Adapter/SafeRedisWrapper.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,17 +25,17 @@ public function __construct(PRedis $cacheAdapter)
2525

2626
/**
2727
* @param string $key
28-
* @return \SplitIO\Component\Cache\Item|null
28+
* @return \SplitIO\Component\Cache\Item
2929
*/
3030
public function getItem($key)
3131
{
3232
try {
3333
return $this->cacheAdapter->getItem($key);
3434
} catch (\Exception $e) {
35-
Di::getLogger()->critical("An error occurred getting " . $key . "from redis.");
35+
Di::getLogger()->critical("An error occurred getting " . $key . " from redis.");
3636
Di::getLogger()->critical($e->getMessage());
3737
Di::getLogger()->critical($e->getTraceAsString());
38-
return null;
38+
return new Item($key);
3939
}
4040
}
4141

@@ -53,7 +53,7 @@ public function getItems(array $keys = array())
5353
try {
5454
return $this->cacheAdapter->getItems($keys);
5555
} catch (\Exception $e) {
56-
Di::getLogger()->critical("An error occurred getting " . $keys . "from redis.");
56+
Di::getLogger()->critical("An error occurred getting " . $keys . " from redis.");
5757
Di::getLogger()->critical($e->getMessage());
5858
Di::getLogger()->critical($e->getTraceAsString());
5959
return array();

0 commit comments

Comments
 (0)