Fix for Warning: ini_set(): A session is active. You cannot change the session module's ini settings at this time

Author: Sergey Matvienko

lib/config/config/factories.yml

@@ -29,7 +29,6 @@ default:
   user:
     class: myUser
     param:
-      timeout:         1800
       logging:         %SF_LOGGING_ENABLED%
       use_flash:       true
       default_culture: %SF_DEFAULT_CULTURE%
@@ -38,6 +37,7 @@ default:
     class: sfSessionStorage
     param:
       session_name: symfony
+      timeout:      1800
 
   view_cache_manager:
     class: sfViewCacheManager

lib/storage/sfSessionStorage.class.php

@@ -68,6 +68,11 @@ public function initialize($options = null)
     // initialize parent
     parent::initialize($options);
 
+    if (!array_key_exists('timeout', $this->options))
+    {
+      $this->options['timeout'] = 1800;
+    }
+
     // set session name
     $sessionName = $this->options['session_name'];
 
@@ -90,6 +95,12 @@ public function initialize($options = null)
       session_cache_limiter($this->options['session_cache_limiter']);
     }
 
+    // force the max lifetime for session garbage collector to be greater than timeout
+    if (ini_get('session.gc_maxlifetime') < $this->options['timeout'])
+    {
+      ini_set('session.gc_maxlifetime', $this->options['timeout']);
+    }
+
     if ($this->options['auto_start'] && !self::$sessionStarted)
     {
       session_start();

lib/user/sfBasicSecurityUser.class.php

@@ -4,7 +4,7 @@
  * This file is part of the symfony package.
  * (c) 2004-2006 Fabien Potencier <fabien.potencier@symfony-project.com>
  * (c) 2004-2006 Sean Kerr <sean@code-box.org>
- * 
+ *
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  */
@@ -42,7 +42,7 @@ public function clearCredentials()
 
   /**
    * Returns the current user's credentials.
-   * 
+   *
    * @return array
    */
   public function getCredentials()
@@ -248,13 +248,8 @@ public function initialize(sfEventDispatcher $dispatcher, sfStorage $storage, $o
 
     if (!array_key_exists('timeout', $this->options))
     {
-      $this->options['timeout'] = 1800;
-    }
-
-    // force the max lifetime for session garbage collector to be greater than timeout
-    if (ini_get('session.gc_maxlifetime') < $this->options['timeout'])
-    {
-      ini_set('session.gc_maxlifetime', $this->options['timeout']);
+      $storageOptions = $storage->getOptions();
+      $this->options['timeout'] = isset($storageOptions['timeout']) ? $storageOptions['timeout'] : 1800;
     }
 
     // read data from storage