From 031326b902f39bb2dd4f27b2747d24180f4c4e1e Mon Sep 17 00:00:00 2001
From: Jon Stovell <jonstovell@gmail.com>
Date: Wed, 25 Dec 2024 13:35:05 -0700
Subject: [PATCH] Replaces uses of deprecated constant SID with calls to
 session_id()

Signed-off-by: Jon Stovell <jonstovell@gmail.com>
---
 Sources/QueryString.php | 14 +++++++-------
 Sources/Utils.php       | 10 +++++-----
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/Sources/QueryString.php b/Sources/QueryString.php
index afd1e1612f..7a113c46fe 100644
--- a/Sources/QueryString.php
+++ b/Sources/QueryString.php
@@ -414,14 +414,14 @@ public static function isFilteredRequest(array $value_list, string $var): bool
 	 */
 	public static function ob_sessrewrite(string $buffer): string
 	{
-		// If Config::$scripturl is set to nothing, or the SID is not defined (SSI?) just quit.
-		if (Config::$scripturl == '' || !defined('SID')) {
+		// If Config::$scripturl is set to nothing, or the session ID is not defined (SSI?) just quit.
+		if (Config::$scripturl == '' || session_id() === false) {
 			return $buffer;
 		}
 
 		// Do nothing if the session is cookied, or they are a crawler - guests are caught by redirectexit().
-		if (empty($_COOKIE) && SID != '' && !BrowserDetector::isBrowser('possibly_robot')) {
-			$buffer = preg_replace('/(?<!<link rel="canonical" href=)"' . preg_quote(Config::$scripturl, '/') . '(?!\?' . preg_quote(SID, '/') . ')\??/', '"' . Config::$scripturl . '?' . SID . '&amp;', $buffer);
+		if (empty($_COOKIE) && session_id() != '' && !BrowserDetector::isBrowser('possibly_robot')) {
+			$buffer = preg_replace('/(?<!<link rel="canonical" href=)"' . preg_quote(Config::$scripturl, '/') . '(?!\?' . preg_quote(session_id(), '/') . ')\??/', '"' . Config::$scripturl . '?' . session_id() . '&amp;', $buffer);
 		}
 		// Debugging templates, are we?
 		elseif (isset($_GET['debug'])) {
@@ -441,11 +441,11 @@ public static function ob_sessrewrite(string $buffer): string
 			)
 		) {
 			// Let's do something special for session ids!
-			if (defined('SID') && SID != '') {
+			if (session_id() != '') {
 				$buffer = preg_replace_callback(
-					'~"' . preg_quote(Config::$scripturl, '~') . '\?(?:' . SID . '(?:;|&|&amp;))((?:board|topic)=[^#"]+?)(#[^"]*?)?"~',
+					'~"' . preg_quote(Config::$scripturl, '~') . '\?(?:' . session_id() . '(?:;|&|&amp;))((?:board|topic)=[^#"]+?)(#[^"]*?)?"~',
 					function ($m) {
-						return '"' . Config::$scripturl . '/' . strtr("{$m[1]}", '&;=', '//,') . '.html?' . SID . ($m[2] ?? '') . '"';
+						return '"' . Config::$scripturl . '/' . strtr("{$m[1]}", '&;=', '//,') . '.html?' . session_id() . ($m[2] ?? '') . '"';
 					},
 					$buffer,
 				);
diff --git a/Sources/Utils.php b/Sources/Utils.php
index 33be0e7b25..5a3d84ca38 100644
--- a/Sources/Utils.php
+++ b/Sources/Utils.php
@@ -2237,8 +2237,8 @@ public static function redirectexit(string $setLocation = '', bool $refresh = fa
 		}
 
 		// Put the session ID in.
-		if (defined('SID') && SID != '') {
-			$setLocation = preg_replace('/^' . preg_quote(Config::$scripturl, '/') . '(?!\?' . preg_quote(SID, '/') . ')\??/', Config::$scripturl . '?' . SID . ';', $setLocation);
+		if (session_id() != '') {
+			$setLocation = preg_replace('/^' . preg_quote(Config::$scripturl, '/') . '(?!\?' . preg_quote(session_id(), '/') . ')\??/', Config::$scripturl . '?' . session_id() . ';', $setLocation);
 		}
 		// Keep that debug in their for template debugging!
 		elseif (isset($_GET['debug'])) {
@@ -2256,11 +2256,11 @@ public static function redirectexit(string $setLocation = '', bool $refresh = fa
 				Sapi::isSoftware([Sapi::SERVER_APACHE, Sapi::SERVER_LIGHTTPD, Sapi::SERVER_LITESPEED])
 			)
 		) {
-			if (defined('SID') && SID != '') {
+			if (session_id() != '') {
 				$setLocation = preg_replace_callback(
-					'~^' . preg_quote(Config::$scripturl, '~') . '\?(?:' . SID . '(?:;|&|&amp;))((?:board|topic)=[^#]+?)(#[^"]*?)?$~',
+					'~^' . preg_quote(Config::$scripturl, '~') . '\?(?:' . session_id() . '(?:;|&|&amp;))((?:board|topic)=[^#]+?)(#[^"]*?)?$~',
 					function ($m) {
-						return Config::$scripturl . '/' . strtr("{$m[1]}", '&;=', '//,') . '.html?' . SID . (isset($m[2]) ? "{$m[2]}" : '');
+						return Config::$scripturl . '/' . strtr("{$m[1]}", '&;=', '//,') . '.html?' . session_id() . (isset($m[2]) ? "{$m[2]}" : '');
 					},
 					$setLocation,
 				);