aboutsummaryrefslogtreecommitdiffstats
path: root/phpBB/includes
diff options
context:
space:
mode:
authorNils Adermann <naderman@naderman.de>2010-03-11 16:08:19 +0100
committerNils Adermann <naderman@naderman.de>2010-09-17 14:00:01 +0200
commit6beeda79eb5a001b589e987d832acf4ea0ae5b4f (patch)
treebb1a00cdcf141eed665900bbe501e08b8600cd9b /phpBB/includes
parent76e530196bb99d02b3d6d7736fde027fa5e2bae8 (diff)
downloadforums-6beeda79eb5a001b589e987d832acf4ea0ae5b4f.tar
forums-6beeda79eb5a001b589e987d832acf4ea0ae5b4f.tar.gz
forums-6beeda79eb5a001b589e987d832acf4ea0ae5b4f.tar.bz2
forums-6beeda79eb5a001b589e987d832acf4ea0ae5b4f.tar.xz
forums-6beeda79eb5a001b589e987d832acf4ea0ae5b4f.zip
[feature/request-class] Replace direct use of GET/REQUEST with request_var.
Now with $_VARs causing fatal errors we should really be able to find and delete all of these occurances. PHPBB3-9716
Diffstat (limited to 'phpBB/includes')
-rw-r--r--phpBB/includes/session.php9
1 files changed, 5 insertions, 4 deletions
diff --git a/phpBB/includes/session.php b/phpBB/includes/session.php
index cc216392b8..0e6a23762e 100644
--- a/phpBB/includes/session.php
+++ b/phpBB/includes/session.php
@@ -130,7 +130,7 @@ class session
'root_script_path' => str_replace(' ', '%20', htmlspecialchars($root_script_path)),
'page' => $page,
- 'forum' => (isset($_REQUEST['f']) && $_REQUEST['f'] > 0) ? (int) $_REQUEST['f'] : 0,
+ 'forum' => request_var('f', 0),
);
return $page_array;
@@ -318,7 +318,7 @@ class session
}
// Is session_id is set or session_id is set and matches the url param if required
- if (!empty($this->session_id) && (!defined('NEED_SID') || (isset($_GET['sid']) && $this->session_id === $_GET['sid'])))
+ if (!empty($this->session_id) && (!defined('NEED_SID') || (isset($_GET['sid']) && $this->session_id === request_var('sid', ''))))
{
$sql = 'SELECT u.*, s.*
FROM ' . SESSIONS_TABLE . ' s, ' . USERS_TABLE . " u
@@ -1591,11 +1591,12 @@ class user extends session
$this->add_lang($lang_set);
unset($lang_set);
- if (!empty($_GET['style']) && $auth->acl_get('a_styles') && !defined('ADMIN_START'))
+ $style_request = request_var('style', 0);
+ if ($style_request && $auth->acl_get('a_styles') && !defined('ADMIN_START'))
{
global $SID, $_EXTRA_URL;
- $style = request_var('style', 0);
+ $style = $style_request;
$SID .= '&amp;style=' . $style;
$_EXTRA_URL = array('style=' . $style);
}