[icinga-checkins] icinga.org: icinga-web/master: AppKitDoctrineSessionStorage: Update for PHP 7 session interfaces

git at icinga.org git at icinga.org
Mon Jul 18 18:53:33 CEST 2016


Module: icinga-web
Branch: master
Commit: 08bd86f64a8835c106c84ce38801fa617afd8e88
URL:    https://git.icinga.org/?p=icinga-web.git;a=commit;h=08bd86f64a8835c106c84ce38801fa617afd8e88

Author: Markus Frosch <lazyfrosch at icinga.org>
Date:   Mon Jul 18 15:56:51 2016 +0200

AppKitDoctrineSessionStorage: Update for PHP 7 session interfaces

fixes #12173

---

 .../AppKit/lib/storage/AppKitDoctrineSessionStorage.class.php |    9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/app/modules/AppKit/lib/storage/AppKitDoctrineSessionStorage.class.php b/app/modules/AppKit/lib/storage/AppKitDoctrineSessionStorage.class.php
index c3725d8..eecf90c 100644
--- a/app/modules/AppKit/lib/storage/AppKitDoctrineSessionStorage.class.php
+++ b/app/modules/AppKit/lib/storage/AppKitDoctrineSessionStorage.class.php
@@ -53,7 +53,7 @@ class AppKitDoctrineSessionStorage extends AgaviSessionStorage {
 
     public function sessionClose() {
         // Hm, the same as sessionOpen?!
-
+        return true;
     }
 
     /**
@@ -116,6 +116,7 @@ class AppKitDoctrineSessionStorage extends AgaviSessionStorage {
      */
     public function sessionOpen($path, $name) {
         // Hm should we do anything here?
+        return true;
     }
 
     /**
@@ -169,7 +170,7 @@ class AppKitDoctrineSessionStorage extends AgaviSessionStorage {
      * @param string $id
      * @param mixed $data
      */
-    public function sessionWrite($id, &$data) {
+    public function sessionWrite($id, $data) {
         $max = ini_get('session.gc_maxlifetime');
         $update = false;
 
@@ -185,7 +186,7 @@ class AppKitDoctrineSessionStorage extends AgaviSessionStorage {
         }
 
         if (! $update && $this->NsmSession->session_checksum === $m) {
-            return;
+            return true;
         }
 
         AppKitLogger::verbose("Writing new session information (checksum=%s)", $m);
@@ -199,6 +200,8 @@ class AppKitDoctrineSessionStorage extends AgaviSessionStorage {
         AppKitLogger::debug("Write session update: %s", $id);
 
         AppKitLogger::verbose("Writing new session information successful");
+
+        return true;
     }
 
 }



More information about the icinga-checkins mailing list