[HttpFoundation][Session] change possible replace() & set() for set only()

vicb · vicb · commit 72d21c66149f · 2012-05-08T10:05:47.000+02:00
diff --git a/src/Symfony/Component/HttpFoundation/Session/Storage/Handler/MemcacheSessionHandler.php b/src/Symfony/Component/HttpFoundation/Session/Storage/Handler/MemcacheSessionHandler.php
@@ -117,11 +117,7 @@ public function read($sessionId)
      */
     public function write($sessionId, $data)
     {
-        if (!$this->memcache->replace($this->prefix.$sessionId, $data, 0, $this->memcacheOptions['expiretime'])) {
-            return $this->memcache->set($this->prefix.$sessionId, $data, 0, $this->memcacheOptions['expiretime']);
-        }
-
-        return true;
+        return $this->memcache->set($this->prefix.$sessionId, $data, 0, $this->memcacheOptions['expiretime']);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -117,11 +117,7 @@ public function read($sessionId)`
`117`	`117`	`*/`
`118`	`118`	`public function write($sessionId, $data)`
`119`	`119`	`{`
`120`		`- if (!$this->memcache->replace($this->prefix.$sessionId, $data, 0, $this->memcacheOptions['expiretime'])) {`
`121`		`- return $this->memcache->set($this->prefix.$sessionId, $data, 0, $this->memcacheOptions['expiretime']);`
`122`		`- }`
`123`		`-`
`124`		`- return true;`
	`120`	`+ return $this->memcache->set($this->prefix.$sessionId, $data, 0, $this->memcacheOptions['expiretime']);`
`125`	`121`	`}`
`126`	`122`
`127`	`123`	`/**`