Moving names around, and the big fix.... Weirdness with the session handler, adding ttl to time was causing weird stuff to happen. Solves issue #13

Author: cryptiklemur

src/Aequasi/Bundle/CacheBundle/DependencyInjection/Builder/ServiceBuilder.php

@@ -74,14 +74,14 @@ private function buildService($typeId, $name, array $instance)
     private function prepareCacheClass(Definition $service, $name, array $instance)
     {
         $type  = $instance['type'];
-        $id    = sprintf("%s.instance.%s.%s_instance", $this->getAlias(), $name, $type);
+        $id    = sprintf("%s.instance.%s.cache_instance", $this->getAlias(), $name);
         $cache = null;
 
         switch ($type) {
             case 'memcache':
                 if (empty($instance['id'])) {
                     $cache = new Definition('Memcache');
-                    $cache->setPublic(false);
+                    //$cache->setPublic(false);
                     foreach ($instance['hosts'] as $config) {
                         $host    = empty($config['host']) ? 'localhost' : $config['host'];
                         $port    = empty($config['port']) ? 11211 : $config['port'];
@@ -99,11 +99,12 @@ private function prepareCacheClass(Definition $service, $name, array $instance)
             case 'memcached':
                 if (empty($instance['id'])) {
                     $cache = new Definition('Memcached');
+                    //$cache->setPublic(false);
+
                     if ($instance['persistent']) {
                         $cache->setArguments(array(serialize($instance['hosts'])));
                     }
 
-                    $cache->setPublic(false);
                     foreach ($instance['hosts'] as $config) {
                         $host   = is_null($config['host']) ? 'localhost' : $config['host'];
                         $port   = is_null($config['port']) ? 11211 : $config['port'];
@@ -121,7 +122,8 @@ private function prepareCacheClass(Definition $service, $name, array $instance)
             case 'redis':
                 if (empty($instance['id'])) {
                     $cache = new Definition('Redis');
-                    $cache->setPublic(false);
+                    //$cache->setPublic(false);
+
                     foreach ($instance['hosts'] as $config) {
                         $host    = empty($config['host']) ? 'localhost' : $config['host'];
                         $port    = empty($config['port']) ? 6379 : $config['port'];

src/Aequasi/Bundle/CacheBundle/DependencyInjection/Compiler/SessionSupportCompilerPass.php

@@ -66,6 +66,6 @@ private function enableSessionSupport(array $config)
         $this->container->setDefinition(sprintf('%s.session_handler', $this->getAlias()), $definition);
         $this->container->setAlias('cache.session_handler', sprintf('%s.session_handler', $this->getAlias()));
 
-        $this->container->setAlias('session.handler', $this->getAlias() . '.session_handler');
+        $this->container->setAlias('session.handler', 'cache.session_handler');
     }
 }

src/Aequasi/Bundle/CacheBundle/Service/CacheService.php

@@ -100,7 +100,7 @@ private function timeCall( $name, $arguments )
      *
      * @return mixed The cached data or FALSE, if no cache entry exists for the given id.
      */
-    function fetch( $id )
+    public function fetch( $id )
     {
         if ($this->isLogging()) {
             $call         = $this->timeCall( 'fetch', array( $id ) );
@@ -122,7 +122,7 @@ function fetch( $id )
      *
      * @return boolean TRUE if a cache entry exists for the given cache id, FALSE otherwise.
      */
-    function contains( $id )
+    public function contains( $id )
     {
         if ($this->isLogging()) {
             $call           = $this->timeCall( 'contains', array( $id ) );
@@ -144,7 +144,7 @@ function contains( $id )
      *
      * @return boolean TRUE if the entry was successfully stored in the cache, FALSE otherwise.
      */
-    function save( $id, $data, $lifeTime = self::NO_EXPIRE )
+    public function save( $id, $data, $lifeTime = self::NO_EXPIRE )
     {
         if ($this->isLogging()) {
             $call            = $this->timeCall( 'save', array( $id, $data, $lifeTime ) );
@@ -164,7 +164,7 @@ function save( $id, $data, $lifeTime = self::NO_EXPIRE )
      *
      * @return boolean TRUE if the cache entry was successfully deleted, FALSE otherwise.
      */
-    function delete( $id )
+    public function delete( $id )
     {
         if ($this->isLogging()) {
             $call           = $this->timeCall( 'delete', array( $id ) );
@@ -248,7 +248,7 @@ private function getDataFromPayload( $payload )
      *
      * @return array|null An associative array with server's statistics if available, NULL otherwise.
      */
-    function getStats()
+    public function getStats()
     {
         return $this->cache->getStats();
     }
@@ -292,4 +292,4 @@ public function getCalls()
     {
         return $this->calls;
     }
-}
+}

src/Aequasi/Bundle/CacheBundle/Session/SessionHandler.php

@@ -49,7 +49,7 @@ public function __construct( Cache $cache, array $options = array() )
         $this->cache = $cache;
 
         $this->ttl    = isset( $options[ 'cookie_lifetime' ] ) ? (int)$options[ 'cookie_lifetime' ] : 86400;
-        $this->prefix = isset( $options[ 'prefix' ] ) ? $options[ 'prefix' ] : 'sf2s';
+        $this->prefix = isset( $options[ 'prefix' ] ) ? $options[ 'prefix' ] : 'sf2ses_';
     }
 
     /**
@@ -81,7 +81,7 @@ public function read( $sessionId )
      */
     public function write( $sessionId, $data )
     {
-        return $this->cache->save( $this->prefix . $sessionId, $data, time() + $this->ttl );
+        return $this->cache->save( $this->prefix . $sessionId, $data, $this->ttl );
     }
 
     /**