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

merged branch dlsniper/patch-1 (PR #3447)

Browse Source 
Commits
-------

eb58dd1 Removed useless parameter from Memcached::set()

Discussion
----------

Removed useless parameter from Memcached::set() which makes users unable to set session expiry time.

Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: -
Todo: -

The parameter count is wrong so it makes setting session expiration useless.

---------------------------------------------------------------------------

by stof at 2012-02-25T16:06:16Z

Already fixed in 15c6ba93f

---------------------------------------------------------------------------

by stof at 2012-02-25T16:06:46Z

ah sorry, it was the profiler storage
This commit is contained in:
Fabien Potencier 2012-02-27 10:05:08 +01:00
commit e7b17ce6c8
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/HttpFoundation/Session/Storage/MemcachedSessionStorage.php
View File

@ -91,7 +91,7 @@ class MemcachedSessionStorage extends AbstractSessionStorage implements \Session
*/
public function write($sessionId, $data)
{
return $this->memcached->set($sessionId, $data, false, $this->memcachedOptions['expiretime']);
return $this->memcached->set($sessionId, $data, $this->memcachedOptions['expiretime']);
}
/**