From 484f59706f28b556f1b8651a5f3aedc88cd18a1a Mon Sep 17 00:00:00 2001
From: Josef Moravec <josef.moravec@gmail.com>
Date: Mon, 21 Oct 2019 14:50:18 +0200
Subject: [PATCH] Fix Redis session handler destroy return value (#1474)
---
module/VuFind/src/VuFind/Session/Redis.php | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/module/VuFind/src/VuFind/Session/Redis.php b/module/VuFind/src/VuFind/Session/Redis.php
index 9e4dac5ca28..facf0e324ff 100644
--- a/module/VuFind/src/VuFind/Session/Redis.php
+++ b/module/VuFind/src/VuFind/Session/Redis.php
@@ -121,7 +121,7 @@ class Redis extends AbstractBase
*
* @param string $sess_id The session ID to destroy
*
- * @return void
+ * @return bool
*/
public function destroy($sess_id)
{
@@ -130,9 +130,10 @@ class Redis extends AbstractBase
// Perform Redis-specific cleanup
if ($this->redisVersion >= 4) {
- $this->getConnection()->unlink("vufind_sessions/{$sess_id}");
+ $return = $this->getConnection()->unlink("vufind_sessions/{$sess_id}");
} else {
- $this->getConnection()->del("vufind_sessions/{$sess_id}");
+ $return = $this->getConnection()->del("vufind_sessions/{$sess_id}");
}
+ return ($return > 0) ? true : false;
}
}
--
GitLab