From 095ca86b341de6e8bee26f829fe915027a686b18 Mon Sep 17 00:00:00 2001
From: Albert Santoni <albert.santoni@sourcefabric.org>
Date: Fri, 10 Apr 2015 15:03:25 -0400
Subject: [PATCH] Open session for writing when we need it (fixes login/logout)

---
 airtime_mvc/application/controllers/LoginController.php       | 2 ++
 airtime_mvc/application/controllers/ShowbuilderController.php | 4 +++-
 .../application/controllers/UsersettingsController.php        | 3 +++
 3 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/airtime_mvc/application/controllers/LoginController.php b/airtime_mvc/application/controllers/LoginController.php
index 9d2ef2f9d..920a258cc 100644
--- a/airtime_mvc/application/controllers/LoginController.php
+++ b/airtime_mvc/application/controllers/LoginController.php
@@ -8,6 +8,8 @@ class LoginController extends Zend_Controller_Action
 
     public function init()
     {
+        //Open the session for writing, because we close it for writing by default in Bootstrap.php as an optimization.
+        session_start();
     }
 
     public function indexAction()
diff --git a/airtime_mvc/application/controllers/ShowbuilderController.php b/airtime_mvc/application/controllers/ShowbuilderController.php
index 3bb30abf4..9dadb6b71 100644
--- a/airtime_mvc/application/controllers/ShowbuilderController.php
+++ b/airtime_mvc/application/controllers/ShowbuilderController.php
@@ -67,7 +67,8 @@ class ShowbuilderController extends Zend_Controller_Action
                 if (isset($values["Privacy"])) {
                     Application_Model_Preference::SetPrivacyPolicyCheck($values["Privacy"]);
                 }
-                // unset session
+                // unset referrer
+                session_start();  //open session for writing again
                 Zend_Session::namespaceUnset('referrer');
             } elseif ($values["Publicise"] == '1' && $form->isValid($values)) {
                 Application_Model_Preference::SetHeadTitle($values["stnName"], $this->view);
@@ -89,6 +90,7 @@ class ShowbuilderController extends Zend_Controller_Action
                     Application_Model_Preference::SetPrivacyPolicyCheck($values["Privacy"]);
                 }
                 // unset session
+                session_start();  //open session for writing again
                 Zend_Session::namespaceUnset('referrer');
             } else {
                 $logo = Application_Model_Preference::GetStationLogo();
diff --git a/airtime_mvc/application/controllers/UsersettingsController.php b/airtime_mvc/application/controllers/UsersettingsController.php
index 86b366fc8..35fbb19e1 100644
--- a/airtime_mvc/application/controllers/UsersettingsController.php
+++ b/airtime_mvc/application/controllers/UsersettingsController.php
@@ -70,12 +70,14 @@ class UsersettingsController extends Zend_Controller_Action
     public function remindmeAction()
     {
         // unset session
+        session_start();  //open session for writing again
         Zend_Session::namespaceUnset('referrer');
         Application_Model_Preference::SetRemindMeDate();
     }
     
     public function remindmeNeverAction()
     {
+        session_start();  //open session for writing again
         Zend_Session::namespaceUnset('referrer');
         //pass in true to indicate 'Remind me never' was clicked
         Application_Model_Preference::SetRemindMeDate(true);
@@ -84,6 +86,7 @@ class UsersettingsController extends Zend_Controller_Action
     public function donotshowregistrationpopupAction()
     {
         // unset session
+        session_start();  //open session for writing again
         Zend_Session::namespaceUnset('referrer');
     }