From c155fd4e4afe64a3c21850327ec749c0b5778850 Mon Sep 17 00:00:00 2001 From: Aaron Schulz Date: Fri, 21 Nov 2008 09:55:13 +0000 Subject: [PATCH] Revert r43804 'This should probably be in Response, not Request, as we're setting data, not getting it. Nothing's using it yet (fairly new), so nothing to update.' Session id is used by client request to specify its login data from cookie, so the session data, by extensions, is like a sort of request parameter. Also, WebResponse.php seems to be used for data actually sent to the client, like requested cookies. --- includes/WebRequest.php | 6 ++++++ includes/WebResponse.php | 4 ---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/includes/WebRequest.php b/includes/WebRequest.php index c3c1d89dad..467471250d 100644 --- a/includes/WebRequest.php +++ b/includes/WebRequest.php @@ -658,6 +658,9 @@ class WebRequest { return null; return $_SESSION[$key]; } + function setSessionData( $key, $data ) { + $_SESSION[$key] = $data; + } } /** @@ -722,5 +725,8 @@ class FauxRequest extends WebRequest { return null; return $this->session[$key]; } + function setSessionData( $key, $data ) { + $this->notImplemented( __METHOD__ ); + } } diff --git a/includes/WebResponse.php b/includes/WebResponse.php index 19feadd426..09d373850f 100644 --- a/includes/WebResponse.php +++ b/includes/WebResponse.php @@ -57,8 +57,4 @@ class WebResponse { $wgCookieSecure ); } } - - public function setSessionData( $key, $data ) { - $_SESSION[$key] = $data; - } } -- 2.20.1