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

merged branch jmikola/memcached-profiler (PR #3358)

Browse Source 
Commits
-------

57968f8 [HttpKernel] Fix call to Memcached::set() in MemcachedProfilerStorage

Discussion
----------

[HttpKernel] Fix call to Memcached::set() in MemcachedProfilerStorage

The existing code seems to have been copied from MemcacheProfilerStorage. Memcache::set() includes a $flag argument, but Memcached::set() omits that. See:

 * http://php.net/manual/en/memcached.set.php
 * http://php.net/manual/en/memcache.set.php

----

I assume Travis-CI didn't catch the MemcachedProfilerStorageTest failures because those tests are skipped: http://travis-ci.org/#!/symfony/symfony/jobs/678889/L104
This commit is contained in:
Fabien Potencier 2012-02-15 10:59:57 +01:00
commit 2b384bb461
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Symfony/Component/HttpKernel/Profiler/MemcachedProfilerStorage.php
View File

@ -67,7 +67,7 @@ class MemcachedProfilerStorage extends BaseMemcacheProfilerStorage
*/ */
protected function setValue($key, $value, $expiration = 0) protected function setValue($key, $value, $expiration = 0)
{ {
return $this->getMemcached()->set($key, $value, false, $expiration); return $this->getMemcached()->set($key, $value, $expiration);
} }
/** /**