Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Commit

Permalink
[zendframework/zendframework#4444] Fix usage of normalizeLibOptionKey
Browse files Browse the repository at this point in the history
- normalizeLibOptionKey() uses pass-by-reference, but statement was
  assuming a return value.
- fixed method to use pass-by-reference
  • Loading branch information
weierophinney committed May 8, 2013
1 parent b5b0936 commit 2697e3e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/Storage/Adapter/MemcachedResourceManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -282,14 +282,14 @@ public function getLibOption($id, $key)
throw new Exception\RuntimeException("No resource with id '{$id}'");
}

$constValue = $this->normalizeLibOptionKey($key);
$this->normalizeLibOptionKey($key);
$resource = & $this->resources[$id];

if ($resource instanceof MemcachedResource) {
return $resource->getOption($constValue);
return $resource->getOption($key);
}

return isset($resource['lib_options'][$constValue]) ? $resource['lib_options'][$constValue] : null;
return isset($resource['lib_options'][$key]) ? $resource['lib_options'][$key] : null;
}

/**
Expand Down

0 comments on commit 2697e3e

Please sign in to comment.