Use "hasKey" instead of checking the value

If the check is negative it would depending on the used cache store the value as an empty string. When reading the value this check would thus return "false" even if a value exists.
This commit is contained in:
Lukas Reschke 2016-03-08 23:24:44 +01:00
parent 81f694d83e
commit ddf56b84d4
1 changed files with 2 additions and 2 deletions

View File

@ -253,8 +253,8 @@ class Storage extends DAV implements ISharedStorage {
*/ */
private function testRemoteUrl($url) { private function testRemoteUrl($url) {
$cache = $this->memcacheFactory->create('files_sharing_remote_url'); $cache = $this->memcacheFactory->create('files_sharing_remote_url');
if($result = $cache->get($url)) { if($cache->hasKey($url)) {
return (bool)$result; return (bool)$cache->get($url);
} }
$result = file_get_contents($url); $result = file_get_contents($url);