diff --git a/.htaccess b/.htaccess index 3fbeec0186c..16f05555852 100644 --- a/.htaccess +++ b/.htaccess @@ -20,17 +20,16 @@ ErrorDocument 500 /error.php # Overload PHP variables: - php_value register_globals 1 php_value track_vars 1 php_value short_open_tag 1 php_value magic_quotes_gpc 0 php_value magic_quotes_runtime 0 php_value magic_quotes_sybase 0 - php_value arg_separator.output "&" + php_value arg_separator.output "&" php_value session.cache_expire 200000 php_value session.cookie_lifetime 2000000 php_value session.auto_start 0 - php_value session.save_handler user + php_value session.save_handler user php_value session.cache_limiter none php_value allow_call_time_pass_reference Off @@ -44,7 +43,7 @@ ErrorDocument 500 /error.php #RewriteCond %{REQUEST_FILENAME} !-d #RewriteCond %{QUERY_STRING} ^id=([^&]+)$ #RewriteRule node.php index.php?q=node/view/%1 [L] - + # Rewrite old-style URLs of the form 'module.php?mod=x': #RewriteCond %{REQUEST_FILENAME} !-f #RewriteCond %{REQUEST_FILENAME} !-d @@ -54,7 +53,7 @@ ErrorDocument 500 /error.php # Rewrite URLs of the form 'index.php?q=x': RewriteCond %{REQUEST_FILENAME} !-f RewriteCond %{REQUEST_FILENAME} !-d - RewriteRule ^(.*)$ index.php?q=$1 [L,QSA] + RewriteRule ^(.*)$ index.php?q=$1 [L,QSA] # $Id$ diff --git a/CHANGELOG b/CHANGELOG index 72a38922f7f..88f314961eb 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,4 +1,4 @@ -Drupal x.x.x, xxxx-xx-xx (to be released) +Drupal 4.2.0, xxxx-xx-xx (to be released) ------------------------ - added support for clean URLs. @@ -23,6 +23,8 @@ Drupal x.x.x, xxxx-xx-xx (to be released) * added default node settings to control the behaviour for promotion, moderation and other options. - themes: * replaced theme "Goofy" by "Xtemplate", a template driven theme. +- removed the 'register_globals = on' requirement. +- added better installation instructions. Drupal 4.1.0, 2003-02-01 ------------------------ diff --git a/INSTALL b/INSTALL index 2ce43d20902..92ffb3059b8 100644 --- a/INSTALL +++ b/INSTALL @@ -15,7 +15,6 @@ SERVER CONFIGURATION Your PHP must have the following settings: - register_globals 1 magic_quotes_gpc 0 session.save_handler user diff --git a/database/database.mssql b/database/database.mssql index d4749d16555..37e08316257 100644 --- a/database/database.mssql +++ b/database/database.mssql @@ -456,7 +456,7 @@ INSERT INTO system VALUES ('modules/story.module','story','module','',1); INSERT INTO system VALUES ('modules/taxonomy.module','taxonomy','module','',1); INSERT INTO system VALUES ('themes/marvin/marvin.theme','marvin','theme','Internet explorer, Netscape, Opera',1); -INSERT INTO variable(name,value) VALUES('update_start', '2002-05-15'); +INSERT INTO variable(name,value) VALUES('update_start', 's:10:"2002-05-15";'); INSERT INTO variable(name,value) VALUES('theme_default','s:6:"marvin";'); INSERT INTO blocks(module,delta,status) VALUES('user', '0', '1'); diff --git a/database/database.pgsql b/database/database.pgsql index dda55d3b700..08d2772898f 100644 --- a/database/database.pgsql +++ b/database/database.pgsql @@ -563,7 +563,7 @@ INSERT INTO system VALUES ('modules/story.module','story','module','',1); INSERT INTO system VALUES ('modules/taxonomy.module','taxonomy','module','',1); INSERT INTO system VALUES ('themes/marvin/marvin.theme','marvin','theme','Internet explorer, Netscape, Opera',1); -INSERT INTO variable(name,value) VALUES('update_start', '2003-04-19'); +INSERT INTO variable(name,value) VALUES('update_start', 'S:10:"2003-04-19";'); INSERT INTO variable(name,value) VALUES('theme_default','s:6:"marvin";'); INSERT INTO blocks(module,delta,status) VALUES('user', '0', '1'); diff --git a/error.php b/error.php index acafb499deb..0a939ea6a18 100644 --- a/error.php +++ b/error.php @@ -3,9 +3,9 @@ include_once "includes/common.inc"; -$errors = array(500 => "500 error: internal server error", 404 => "404 error: `$REDIRECT_URL' not found", 403 => "403 error: access denied - forbidden", 401 => "401 error: authorization required", 400 => "400 error: bad request"); +$errors = array(500 => "500 error: internal server error", 404 => "404 error: '".`$_SERVER["REDIRECT_URL"] ."' not found", 403 => "403 error: access denied - forbidden", 401 => "401 error: authorization required", 400 => "400 error: bad request"); -watchdog("httpd", $errors[$REDIRECT_STATUS]); +watchdog("httpd", $errors[$_SERVER["REDIRECT_STATUS"]); drupal_goto($base_url); diff --git a/includes/common.inc b/includes/common.inc index 85ebb076bb6..611135f3765 100644 --- a/includes/common.inc +++ b/includes/common.inc @@ -2,7 +2,6 @@ // $Id$ function conf_init() { - global $HTTP_HOST, $PHP_SELF; /* ** Try finding a matching configuration file by stripping the website's @@ -10,9 +9,9 @@ function conf_init() { ** default value 'conf'. */ - $uri = $PHP_SELF; + $uri = $_SERVER["PHP_SELF"]; - $file = strtolower(strtr($HTTP_HOST . substr($uri, 0, strrpos($uri, "/")), "/:", "..")); + $file = strtolower(strtr($_SERVER["HTTP_HOST"] . substr($uri, 0, strrpos($uri, "/")), "/:", "..")); while (strlen($file) > 4) { if (file_exists("includes/$file.php")) { @@ -61,11 +60,10 @@ function check_php_setting($name, $value) { function arg($index) { - global $q; static $arguments; if (empty($arguments)) { - $arguments = explode("/", $q); + $arguments = explode("/", $_GET["q"]); } return $arguments[$index]; @@ -102,13 +100,11 @@ function object2array($node) { function request_uri() { // since request_uri() is only available on apache, we generate equivalent using other environment vars. - global $REQUEST_URI, $PATH_INFO, $QUERY_STRING; - - if (isset($REQUEST_URI)) { - return $REQUEST_URI; + if (isset($_SERVER["REQUEST_URI"])) { + return $_SERVER["REQUEST_URI"]; } else { - return $PATH_INFO ."?". $QUERY_STRING; + return $_SERVER["PATH_INFO"] ."?". $_SERVER["QUERY_STRING"]; } } @@ -351,7 +347,9 @@ function search_form($action = 0, $query = 0, $options = 0) { * Collect the search results: */ function search_data() { - global $keys, $edit; + global $keys; + + $edit = $_POST["edit"]; if (isset($keys)) { foreach (module_list() as $name) { @@ -386,7 +384,8 @@ function search_data() { * ("Restrict search to", help text, etc). */ function search_type($type = 0, $action = 0, $query = 0, $options = 0) { - global $edit; + + $edit = $_POST["edit"]; if (isset($type)) { $edit["type"][$type] = "on"; @@ -437,10 +436,10 @@ function drupal_goto($url) { */ function referer_save() { - global $referer, $HTTP_REFERER; + global $referer; - if (!strstr($HTTP_REFERER, request_uri())) { - $referer = $HTTP_REFERER; + if (!strstr($_SERVER["HTTP_REFERER"], request_uri())) { + $referer = $_SERVER["HTTP_REFERER"]; session_register("referer"); } } @@ -664,9 +663,9 @@ function cache_clear_all($cid = NULL) { } function page_set_cache() { - global $user, $REQUEST_METHOD; + global $user; - if (!$user->uid && $REQUEST_METHOD == "GET") { + if (!$user->uid && $_SERVER["REQUEST_METHOD"] == "GET") { if ($data = ob_get_contents()) { cache_set(request_uri(), $data, (time() + variable_get("cache_clear", 120))); } @@ -674,11 +673,11 @@ function page_set_cache() { } function page_get_cache() { - global $user, $REQUEST_METHOD; + global $user; $cache = NULL; - if (!$user->uid && $REQUEST_METHOD == "GET") { + if (!$user->uid && $_SERVER["REQUEST_METHOD"] == "GET") { $cache = cache_get(request_uri()); if (empty($cache)) { diff --git a/includes/menu.inc b/includes/menu.inc index 1793bfdc4d5..e4698ce3ad5 100644 --- a/includes/menu.inc +++ b/includes/menu.inc @@ -47,12 +47,12 @@ function menu_item($in_path) { } function menu_trail() { - global $_gmenu, $q; + global $_gmenu; static $trail; // cache if (empty($trail)) { $trail = array(); - $path = $q; + $path = $_GET["q"]; while ($path) { if ($_gmenu[$path]) { @@ -142,12 +142,12 @@ function menu_map($parent = "") { } function menu_execute_action() { - global $_gmenu, $q; + global $_gmenu; $trail = menu_trail(); $selected_menu = array_pop($trail); if ($_gmenu[$selected_menu]["callback"]) { - $arg = substr($q, strlen($selected_menu) + 1); + $arg = substr($_GET["q"], strlen($selected_menu) + 1); if (empty($arg)) { return call_user_func($_gmenu[$selected_menu]["callback"]); } diff --git a/includes/pager.inc b/includes/pager.inc index c31693ec4b8..109ce61a730 100644 --- a/includes/pager.inc +++ b/includes/pager.inc @@ -301,7 +301,7 @@ function pager_query($query, $limit = 10, $element = 0, $count_query = "") { } function pager_link($from_new, $attributes = array()) { - global $q; + $q = $_GET["q"]; foreach($attributes as $key => $value) { $query[] = "$key=$value"; diff --git a/includes/theme.inc b/includes/theme.inc index 1822da23a0c..31f7aabbc68 100644 --- a/includes/theme.inc +++ b/includes/theme.inc @@ -179,7 +179,7 @@ function theme_init() { * @param string $region main|left|right */ function theme_blocks($region) { - global $user, $REQUEST_URI; + global $user; $result = db_query("SELECT * FROM blocks WHERE (status = '1' OR custom = '1') ". ($region != "all" ? "AND region = %d " : "") ."ORDER BY weight, module", $region == "left" ? 0 : 1); diff --git a/includes/xmlrpcs.inc b/includes/xmlrpcs.inc index 61ce1913ceb..7e61d877126 100644 --- a/includes/xmlrpcs.inc +++ b/includes/xmlrpcs.inc @@ -148,7 +148,6 @@ class xmlrpc_server { var $dmap=array(); function xmlrpc_server($dispMap, $serviceNow=1) { - global $HTTP_RAW_POST_DATA; // dispMap is a despatch array of methods // mapped to function names and signatures // if a method @@ -208,14 +207,14 @@ class xmlrpc_server { } function parseRequest($data="") { - global $_xh,$HTTP_RAW_POST_DATA; + global $_xh; global $xmlrpcerr, $xmlrpcstr, $xmlrpcerrxml, $xmlrpc_defencoding, $_xmlrpcs_dmap; if ($data=="") { - $data=$HTTP_RAW_POST_DATA; + $data=$_SERVER["HTTP_RAW_POST_DATA"]; } $parser = xml_parser_create($xmlrpc_defencoding); @@ -291,13 +290,12 @@ class xmlrpc_server { } function echoInput() { - global $HTTP_RAW_POST_DATA; // a debugging routine: just echos back the input // packet as a string value $r=new xmlrpcresp; - $r->xv=new xmlrpcval( "'Aha said I: '" . $HTTP_RAW_POST_DATA, "string"); + $r->xv=new xmlrpcval( "'Aha said I: '" . $_SERVER["HTTP_RAW_POST_DATA"], "string"); print $r->serialize(); } } diff --git a/index.php b/index.php index f7f539b2176..b66948282e4 100644 --- a/index.php +++ b/index.php @@ -3,7 +3,7 @@ include_once "includes/common.inc"; -if (isset($q)) { +if (isset($_GET["q"])) { $mod = arg(0); } @@ -20,7 +20,6 @@ else { page_header(); check_php_setting("magic_quotes_gpc", 0); - check_php_setting("register_globals", 1); if (module_hook(variable_get("site_frontpage", "node"), "page")) { module_invoke(variable_get("site_frontpage", "node"), "page"); diff --git a/modules/admin.module b/modules/admin.module index 670c851229a..abe7dfbfcfd 100644 --- a/modules/admin.module +++ b/modules/admin.module @@ -25,7 +25,7 @@ function admin_admin() { } function admin_page() { - global $user, $base_url; + global $base_url; if (user_access("access administration pages")) { page_header(); diff --git a/modules/aggregator.module b/modules/aggregator.module index 69abf97bf36..b5ed8f92b0c 100644 --- a/modules/aggregator.module +++ b/modules/aggregator.module @@ -488,7 +488,8 @@ function import_tag() { } function import_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer news feeds")) { diff --git a/modules/aggregator/aggregator.module b/modules/aggregator/aggregator.module index 69abf97bf36..b5ed8f92b0c 100644 --- a/modules/aggregator/aggregator.module +++ b/modules/aggregator/aggregator.module @@ -488,7 +488,8 @@ function import_tag() { } function import_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer news feeds")) { diff --git a/modules/archive.module b/modules/archive.module index 71aec57f617..1dd7b1a2687 100644 --- a/modules/archive.module +++ b/modules/archive.module @@ -140,7 +140,10 @@ function archive_link($type) { } function archive_page() { - global $date, $edit, $op, $month, $year, $meta; + global $date, $month, $year, $meta; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; theme("header"); diff --git a/modules/archive/archive.module b/modules/archive/archive.module index 71aec57f617..1dd7b1a2687 100644 --- a/modules/archive/archive.module +++ b/modules/archive/archive.module @@ -140,7 +140,10 @@ function archive_link($type) { } function archive_page() { - global $date, $edit, $op, $month, $year, $meta; + global $date, $month, $year, $meta; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; theme("header"); diff --git a/modules/block.module b/modules/block.module index a6bac4fbbda..c70f0f6c564 100644 --- a/modules/block.module +++ b/modules/block.module @@ -260,7 +260,8 @@ function block_box_delete($bid) { } function block_admin() { - global $edit, $op; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer blocks")) { diff --git a/modules/block/block.module b/modules/block/block.module index a6bac4fbbda..c70f0f6c564 100644 --- a/modules/block/block.module +++ b/modules/block/block.module @@ -260,7 +260,8 @@ function block_box_delete($bid) { } function block_admin() { - global $edit, $op; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer blocks")) { diff --git a/modules/book.module b/modules/book.module index dd44b6b824d..bc30b2546fb 100644 --- a/modules/book.module +++ b/modules/book.module @@ -160,8 +160,9 @@ function book_validate(&$node) { } function book_form(&$node, &$help, &$error) { - global $user, $op; + global $user; + $op = $_POST["op"]; $output = form_select(t("Parent"), "parent", $node->parent, book_toc(), t("The parent subject or category the page belongs in.")); if (function_exists("taxonomy_node_form")) { @@ -197,7 +198,10 @@ function book_form(&$node, &$help, &$error) { } function book_node_link($node = 0) { - global $user, $op, $edit; + global $user; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if ($node->type != "book") { @@ -337,7 +341,7 @@ function book_next($node) { } function book_body($node) { - global $op; + $op = $_POST["op"]; if ($node->format == 1) { // Make sure only authorized users can preview PHP pages. @@ -705,7 +709,8 @@ function book_admin_links() { } function book_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer nodes")) { if (empty($op)) { diff --git a/modules/book/book.module b/modules/book/book.module index dd44b6b824d..bc30b2546fb 100644 --- a/modules/book/book.module +++ b/modules/book/book.module @@ -160,8 +160,9 @@ function book_validate(&$node) { } function book_form(&$node, &$help, &$error) { - global $user, $op; + global $user; + $op = $_POST["op"]; $output = form_select(t("Parent"), "parent", $node->parent, book_toc(), t("The parent subject or category the page belongs in.")); if (function_exists("taxonomy_node_form")) { @@ -197,7 +198,10 @@ function book_form(&$node, &$help, &$error) { } function book_node_link($node = 0) { - global $user, $op, $edit; + global $user; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if ($node->type != "book") { @@ -337,7 +341,7 @@ function book_next($node) { } function book_body($node) { - global $op; + $op = $_POST["op"]; if ($node->format == 1) { // Make sure only authorized users can preview PHP pages. @@ -705,7 +709,8 @@ function book_admin_links() { } function book_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer nodes")) { if (empty($op)) { diff --git a/modules/cloud.module b/modules/cloud.module index 71e49b7bba8..61377194395 100644 --- a/modules/cloud.module +++ b/modules/cloud.module @@ -185,7 +185,8 @@ function cloud_block($op = "list", $delta = 0) { } function cloud_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (empty($op)) { $op = arg(3); diff --git a/modules/comment.module b/modules/comment.module index d7b042b96b1..c52e37c94fd 100644 --- a/modules/comment.module +++ b/modules/comment.module @@ -676,7 +676,8 @@ function comment_link($type, $node = 0, $main = 0) { } function comment_page() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (empty($op)) { $op = arg(1); @@ -880,7 +881,7 @@ function comment_mod_roles($edit) { } function comment_mod_votes($edit) { - global $op; + $op = $_POST["op"]; $mid = arg(3); @@ -928,7 +929,7 @@ function comment_mod_votes($edit) { } function comment_mod_filters($edit) { - global $op; + $op = $_POST["op"]; $fid = arg(3); @@ -978,7 +979,10 @@ function comment_mod_filters($edit) { function comment_admin() { - global $op, $id, $edit, $mod, $keys, $order, $status, $comment_page, $comment_settings; + global $id, $mod, $keys, $order, $status, $comment_page, $comment_settings; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (empty($op)) { $op = arg(2); @@ -1122,9 +1126,11 @@ function comment_controls($threshold = 1, $mode = 3, $order = 1, $nid, $page = 0 } function comment_moderation_form($comment) { - global $comment_votes, $op, $user, $node; + global $comment_votes, $user, $node; static $votes; + $op = $_POST["op"]; + if ($op == "reply") { // preview comment: $output .= " "; @@ -1393,7 +1399,6 @@ function comment_already_moderated($uid, $users) { } function comment_search($keys) { - global $PHP_SELF; /* ** Return the results of performing a search using the indexed search diff --git a/modules/comment/comment.module b/modules/comment/comment.module index d7b042b96b1..c52e37c94fd 100644 --- a/modules/comment/comment.module +++ b/modules/comment/comment.module @@ -676,7 +676,8 @@ function comment_link($type, $node = 0, $main = 0) { } function comment_page() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (empty($op)) { $op = arg(1); @@ -880,7 +881,7 @@ function comment_mod_roles($edit) { } function comment_mod_votes($edit) { - global $op; + $op = $_POST["op"]; $mid = arg(3); @@ -928,7 +929,7 @@ function comment_mod_votes($edit) { } function comment_mod_filters($edit) { - global $op; + $op = $_POST["op"]; $fid = arg(3); @@ -978,7 +979,10 @@ function comment_mod_filters($edit) { function comment_admin() { - global $op, $id, $edit, $mod, $keys, $order, $status, $comment_page, $comment_settings; + global $id, $mod, $keys, $order, $status, $comment_page, $comment_settings; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (empty($op)) { $op = arg(2); @@ -1122,9 +1126,11 @@ function comment_controls($threshold = 1, $mode = 3, $order = 1, $nid, $page = 0 } function comment_moderation_form($comment) { - global $comment_votes, $op, $user, $node; + global $comment_votes, $user, $node; static $votes; + $op = $_POST["op"]; + if ($op == "reply") { // preview comment: $output .= " "; @@ -1393,7 +1399,6 @@ function comment_already_moderated($uid, $users) { } function comment_search($keys) { - global $PHP_SELF; /* ** Return the results of performing a search using the indexed search diff --git a/modules/drupal.module b/modules/drupal.module index f9306dbffb0..8e3aaaff958 100644 --- a/modules/drupal.module +++ b/modules/drupal.module @@ -175,7 +175,6 @@ function drupal_auth_help() { } function drupal_user($type, $edit, $user) { - global $HTTP_HOST; $module = "drupal"; $name = module_invoke($module, "info", "name"); @@ -187,7 +186,7 @@ function drupal_user($type, $edit, $user) { } else { // TODO: use a variation of $base_url instead of $HTTP_HOST below - $output .= form_item(t("$name ID"), "$user->name@$HTTP_HOST"); + $output .= form_item(t("$name ID"), "$user->name@". $_SERVER["HTTP_HOST"]); } return $output; } diff --git a/modules/drupal/drupal.module b/modules/drupal/drupal.module index f9306dbffb0..8e3aaaff958 100644 --- a/modules/drupal/drupal.module +++ b/modules/drupal/drupal.module @@ -175,7 +175,6 @@ function drupal_auth_help() { } function drupal_user($type, $edit, $user) { - global $HTTP_HOST; $module = "drupal"; $name = module_invoke($module, "info", "name"); @@ -187,7 +186,7 @@ function drupal_user($type, $edit, $user) { } else { // TODO: use a variation of $base_url instead of $HTTP_HOST below - $output .= form_item(t("$name ID"), "$user->name@$HTTP_HOST"); + $output .= form_item(t("$name ID"), "$user->name@". $_SERVER["HTTP_HOST"]); } return $output; } diff --git a/modules/forum.module b/modules/forum.module index 814b07c69dd..3c9edcc4934 100644 --- a/modules/forum.module +++ b/modules/forum.module @@ -449,7 +449,8 @@ function _forum_message_taxonomy() { } function forum_page() { - global $sortby, $forum_per_page, $from, $op, $user; + global $sortby, $forum_per_page, $from, $user; + $op = $_POST["op"]; if (user_access("access content")) { if (module_exist("taxonomy")) { diff --git a/modules/forum/forum.module b/modules/forum/forum.module index 814b07c69dd..3c9edcc4934 100644 --- a/modules/forum/forum.module +++ b/modules/forum/forum.module @@ -449,7 +449,8 @@ function _forum_message_taxonomy() { } function forum_page() { - global $sortby, $forum_per_page, $from, $op, $user; + global $sortby, $forum_per_page, $from, $user; + $op = $_POST["op"]; if (user_access("access content")) { if (module_exist("taxonomy")) { diff --git a/modules/import.module b/modules/import.module index 69abf97bf36..b5ed8f92b0c 100644 --- a/modules/import.module +++ b/modules/import.module @@ -488,7 +488,8 @@ function import_tag() { } function import_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer news feeds")) { diff --git a/modules/locale.module b/modules/locale.module index 09453e86c8a..41197e51e73 100644 --- a/modules/locale.module +++ b/modules/locale.module @@ -72,7 +72,7 @@ function locale_delete($lid) { } function locale_save($lid) { - global $edit; + $edit = $_POST["edit"]; foreach ($edit as $key=>$value) { db_query("UPDATE locales SET $key = '%s' WHERE lid = %d", $value, $lid); } @@ -124,7 +124,9 @@ function locale_languages($translation) { } function locale_seek() { - global $id, $edit, $languages, $op, $locale_settings; + global $id, $languages, $locale_settings; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if ($op != 'overview' && !$edit && session_is_registered("locale_settings")) { $edit = $locale_settings; @@ -197,7 +199,8 @@ function locale_seek() { } function locale_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer locales")) { locale_admin_initialize(); diff --git a/modules/locale/locale.module b/modules/locale/locale.module index 09453e86c8a..41197e51e73 100644 --- a/modules/locale/locale.module +++ b/modules/locale/locale.module @@ -72,7 +72,7 @@ function locale_delete($lid) { } function locale_save($lid) { - global $edit; + $edit = $_POST["edit"]; foreach ($edit as $key=>$value) { db_query("UPDATE locales SET $key = '%s' WHERE lid = %d", $value, $lid); } @@ -124,7 +124,9 @@ function locale_languages($translation) { } function locale_seek() { - global $id, $edit, $languages, $op, $locale_settings; + global $id, $languages, $locale_settings; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if ($op != 'overview' && !$edit && session_is_registered("locale_settings")) { $edit = $locale_settings; @@ -197,7 +199,8 @@ function locale_seek() { } function locale_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer locales")) { locale_admin_initialize(); diff --git a/modules/node.module b/modules/node.module index aa0c8b2b1e4..7ec69b2079d 100644 --- a/modules/node.module +++ b/modules/node.module @@ -422,7 +422,6 @@ function node_perm() { } function node_search($keys) { - global $PHP_SELF; // Return the results of performing a search using the indexed search // for this particular type of node. @@ -597,7 +596,7 @@ function node_admin_nodes() { */ function node_admin_settings($edit) { - global $op; + $op = $_POST["op"]; if ($op == t("Save configuration")) { /* @@ -762,7 +761,8 @@ function node_revision_list($node) { } function node_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer nodes")) { @@ -1094,7 +1094,9 @@ function node_form($edit, $error = NULL) { } function node_add($type) { - global $user, $edit; + global $user; + + $edit = $_POST["edit"]; /* ** If a node type has been specified, validate it existence. If no @@ -1353,7 +1355,10 @@ function node_delete($edit) { } function node_page() { - global $op, $id, $user, $edit, $or, $and; + global $id, $user, $or, $and; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("access content")) { if (empty($op)) { diff --git a/modules/node/node.module b/modules/node/node.module index aa0c8b2b1e4..7ec69b2079d 100644 --- a/modules/node/node.module +++ b/modules/node/node.module @@ -422,7 +422,6 @@ function node_perm() { } function node_search($keys) { - global $PHP_SELF; // Return the results of performing a search using the indexed search // for this particular type of node. @@ -597,7 +596,7 @@ function node_admin_nodes() { */ function node_admin_settings($edit) { - global $op; + $op = $_POST["op"]; if ($op == t("Save configuration")) { /* @@ -762,7 +761,8 @@ function node_revision_list($node) { } function node_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer nodes")) { @@ -1094,7 +1094,9 @@ function node_form($edit, $error = NULL) { } function node_add($type) { - global $user, $edit; + global $user; + + $edit = $_POST["edit"]; /* ** If a node type has been specified, validate it existence. If no @@ -1353,7 +1355,10 @@ function node_delete($edit) { } function node_page() { - global $op, $id, $user, $edit, $or, $and; + global $id, $user, $or, $and; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("access content")) { if (empty($op)) { diff --git a/modules/page.module b/modules/page.module index 178c7c4e859..87f70d74446 100644 --- a/modules/page.module +++ b/modules/page.module @@ -89,7 +89,8 @@ function page_link($type) { } function page_body($node) { - global $op; + $op = $_POST["op"]; + if ($node->format == 0) { // HTML type $output = check_output($node->body); diff --git a/modules/page/page.module b/modules/page/page.module index 178c7c4e859..87f70d74446 100644 --- a/modules/page/page.module +++ b/modules/page/page.module @@ -89,7 +89,8 @@ function page_link($type) { } function page_body($node) { - global $op; + $op = $_POST["op"]; + if ($node->format == 0) { // HTML type $output = check_output($node->body); diff --git a/modules/poll.module b/modules/poll.module index 0fc1db8a913..3c8477df50c 100644 --- a/modules/poll.module +++ b/modules/poll.module @@ -7,7 +7,7 @@ function poll_allowvotes(&$node) { ** We only need to determine this once for a poll, but we don't do this in ** poll_load() (i.e. for every poll that is loaded) for speed reasons. */ - global $REMOTE_ADDR, $user; + global $user; if ($node->allowvotes != -1) { return $node; @@ -20,7 +20,7 @@ function poll_allowvotes(&$node) { $id = "_". $user->uid ."_"; } else { - $id = $REMOTE_ADDR; + $id = $_SERVER["REMOTE_ADDR"]; } if (!strstr($node->voters, $id)) { $node->allowvotes = $node->active; diff --git a/modules/poll/poll.module b/modules/poll/poll.module index 0fc1db8a913..3c8477df50c 100644 --- a/modules/poll/poll.module +++ b/modules/poll/poll.module @@ -7,7 +7,7 @@ function poll_allowvotes(&$node) { ** We only need to determine this once for a poll, but we don't do this in ** poll_load() (i.e. for every poll that is loaded) for speed reasons. */ - global $REMOTE_ADDR, $user; + global $user; if ($node->allowvotes != -1) { return $node; @@ -20,7 +20,7 @@ function poll_allowvotes(&$node) { $id = "_". $user->uid ."_"; } else { - $id = $REMOTE_ADDR; + $id = $_SERVER["REMOTE_ADDR"]; } if (!strstr($node->voters, $id)) { $node->allowvotes = $node->active; diff --git a/modules/profile.module b/modules/profile.module index d09639c85a7..8fb6da5882d 100644 --- a/modules/profile.module +++ b/modules/profile.module @@ -136,7 +136,7 @@ function _profile_form($edit, $mode) { function _profile_validate($edit, $mode) { - global $profile_fields, $user, $HTTP_POST_VARS; + global $profile_fields, $user; $enabled_fields = _profile_active_fields($mode); @@ -206,19 +206,19 @@ function _profile_user_view(&$user, $mode) { } function _profile_validate_avatar(&$edit) { - global $HTTP_POST_FILES, $user; + global $user; // check that uploaded file is an image, with a max file size and max height/width unset($edit["profile_avatar"]); - if ($HTTP_POST_FILES["edit"]["name"]["profile_avatar"] == "") { + if ($_FILES["edit"]["name"]["profile_avatar"] == "") { $edit["profile_avatar"] = $user->profile_avatar; return ""; } - $image_file = $HTTP_POST_FILES["edit"]["tmp_name"]["profile_avatar"]; + $image_file = $_FILES["edit"]["tmp_name"]["profile_avatar"]; if (is_uploaded_file($image_file)) { - $extension = strtolower(strrchr($HTTP_POST_FILES["edit"]["name"]["profile_avatar"], ".")); + $extension = strtolower(strrchr($_FILES["edit"]["name"]["profile_avatar"], ".")); $size = getimagesize($image_file); list($maxwidth, $maxheight) = explode("x", variable_get("profile_avatar_dimensions", "85x85")); if ((!in_array($size[2], array(1,2,3))) || (!in_array($extension, array(".gif", ".jpg", ".png", ".jpeg")))) { diff --git a/modules/profile/profile.module b/modules/profile/profile.module index d09639c85a7..8fb6da5882d 100644 --- a/modules/profile/profile.module +++ b/modules/profile/profile.module @@ -136,7 +136,7 @@ function _profile_form($edit, $mode) { function _profile_validate($edit, $mode) { - global $profile_fields, $user, $HTTP_POST_VARS; + global $profile_fields, $user; $enabled_fields = _profile_active_fields($mode); @@ -206,19 +206,19 @@ function _profile_user_view(&$user, $mode) { } function _profile_validate_avatar(&$edit) { - global $HTTP_POST_FILES, $user; + global $user; // check that uploaded file is an image, with a max file size and max height/width unset($edit["profile_avatar"]); - if ($HTTP_POST_FILES["edit"]["name"]["profile_avatar"] == "") { + if ($_FILES["edit"]["name"]["profile_avatar"] == "") { $edit["profile_avatar"] = $user->profile_avatar; return ""; } - $image_file = $HTTP_POST_FILES["edit"]["tmp_name"]["profile_avatar"]; + $image_file = $_FILES["edit"]["tmp_name"]["profile_avatar"]; if (is_uploaded_file($image_file)) { - $extension = strtolower(strrchr($HTTP_POST_FILES["edit"]["name"]["profile_avatar"], ".")); + $extension = strtolower(strrchr($_FILES["edit"]["name"]["profile_avatar"], ".")); $size = getimagesize($image_file); list($maxwidth, $maxheight) = explode("x", variable_get("profile_avatar_dimensions", "85x85")); if ((!in_array($size[2], array(1,2,3))) || (!in_array($extension, array(".gif", ".jpg", ".png", ".jpeg")))) { diff --git a/modules/queue.module b/modules/queue.module index 7d4f1d274d5..69c7f45a8e3 100644 --- a/modules/queue.module +++ b/modules/queue.module @@ -111,7 +111,10 @@ function queue_overview() { } function queue_view($nid) { - global $op, $edit, $user; + global $user; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; /* ** An associative array with the possible voting options: diff --git a/modules/search.module b/modules/search.module index c87372241da..e680e2f1c2c 100644 --- a/modules/search.module +++ b/modules/search.module @@ -51,7 +51,7 @@ function search_settings() { * */ function search_admin() { - global $op; + $op = $_POST["op"]; // Only allow people with sufficient access. if (user_access("administer search")) { @@ -92,7 +92,6 @@ function search_cron() { * explanation of array items */ function do_search($search_array) { - global $PHP_SELF; $keys = strtolower($search_array["keys"]); $type = $search_array["type"]; @@ -167,10 +166,10 @@ function do_search($search_array) { $count = $value["count"]; switch ($type) { case "node": - $find[$i++] = array("count" => $count, "title" => $title, "link" => (strstr($PHP_SELF, "admin") ? url("admin/node/edit/$lno") : url("node/view/$lno")), "user" => $name, "date" => $created, "keywords" => implode("|", $words)); + $find[$i++] = array("count" => $count, "title" => $title, "link" => (strstr($_SERVER["PHP_SELF"], "admin") ? url("admin/node/edit/$lno") : url("node/view/$lno")), "user" => $name, "date" => $created, "keywords" => implode("|", $words)); break; case "comment": - $find[$i++] = array("count" => $count, "title" => $title, "link" => (strstr($PHP_SELF, "admin") ? url("admin/comment/edit/$lno") : url("node/view/$nid#$lno")), "user" => $name, "date" => $created, "keywords" => implode("|", $words)); + $find[$i++] = array("count" => $count, "title" => $title, "link" => (strstr($_SERVER["PHP_SELF"], "admin") ? url("admin/comment/edit/$lno") : url("node/view/$nid#$lno")), "user" => $name, "date" => $created, "keywords" => implode("|", $words)); break; } } @@ -306,7 +305,9 @@ function search_save($edit) { } function search_view($keys = NULL) { - global $edit, $type; + global $type; + + $edit = $_POST["edit"]; if (user_access("search content")) { // Construct the search form: diff --git a/modules/search/search.module b/modules/search/search.module index c87372241da..e680e2f1c2c 100644 --- a/modules/search/search.module +++ b/modules/search/search.module @@ -51,7 +51,7 @@ function search_settings() { * */ function search_admin() { - global $op; + $op = $_POST["op"]; // Only allow people with sufficient access. if (user_access("administer search")) { @@ -92,7 +92,6 @@ function search_cron() { * explanation of array items */ function do_search($search_array) { - global $PHP_SELF; $keys = strtolower($search_array["keys"]); $type = $search_array["type"]; @@ -167,10 +166,10 @@ function do_search($search_array) { $count = $value["count"]; switch ($type) { case "node": - $find[$i++] = array("count" => $count, "title" => $title, "link" => (strstr($PHP_SELF, "admin") ? url("admin/node/edit/$lno") : url("node/view/$lno")), "user" => $name, "date" => $created, "keywords" => implode("|", $words)); + $find[$i++] = array("count" => $count, "title" => $title, "link" => (strstr($_SERVER["PHP_SELF"], "admin") ? url("admin/node/edit/$lno") : url("node/view/$lno")), "user" => $name, "date" => $created, "keywords" => implode("|", $words)); break; case "comment": - $find[$i++] = array("count" => $count, "title" => $title, "link" => (strstr($PHP_SELF, "admin") ? url("admin/comment/edit/$lno") : url("node/view/$nid#$lno")), "user" => $name, "date" => $created, "keywords" => implode("|", $words)); + $find[$i++] = array("count" => $count, "title" => $title, "link" => (strstr($_SERVER["PHP_SELF"], "admin") ? url("admin/comment/edit/$lno") : url("node/view/$nid#$lno")), "user" => $name, "date" => $created, "keywords" => implode("|", $words)); break; } } @@ -306,7 +305,9 @@ function search_save($edit) { } function search_view($keys = NULL) { - global $edit, $type; + global $type; + + $edit = $_POST["edit"]; if (user_access("search content")) { // Construct the search form: diff --git a/modules/statistics.module b/modules/statistics.module index 4c6a6fb7799..f227baabfff 100644 --- a/modules/statistics.module +++ b/modules/statistics.module @@ -259,7 +259,8 @@ function statistics_help() { /* Administration hook, defines module's administrative page */ function statistics_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; /* Only allow people with sufficient access. */ if ((user_access("administer statistics module")) || (user_access("administer statistics"))) { @@ -393,7 +394,7 @@ function statistics_admin_accesslog_table($type, $id) { function statistics_recent_refer($nid = 0) { - global $HTTP_HOST, $view; + global $view; if (empty($view)) { $view = arg(3); @@ -406,12 +407,12 @@ function statistics_recent_refer($nid = 0) { $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url <> '' ORDER BY timestamp DESC"; } elseif ($view == "internal") { - $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url LIKE '%". check_input($HTTP_HOST) ."%' ORDER BY timestamp DESC"; + $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' ORDER BY timestamp DESC"; $describe = "internal "; } else { /* default to external referrers */ - $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url NOT LIKE '%". check_input($HTTP_HOST) ."%' AND url <> '' ORDER BY timestamp DESC"; + $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url NOT LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' AND url <> '' ORDER BY timestamp DESC"; $describe = "external "; } @@ -423,11 +424,11 @@ function statistics_recent_refer($nid = 0) { $query = "SELECT url,timestamp FROM accesslog WHERE url <> '' ORDER BY timestamp DESC"; } elseif ($view == "internal") { - $query = "SELECT url,timestamp FROM accesslog WHERE url LIKE '%". check_input($HTTP_HOST) ."%' ORDER BY timestamp DESC"; + $query = "SELECT url,timestamp FROM accesslog WHERE url LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' ORDER BY timestamp DESC"; $describe = "internal "; } else { - $query = "SELECT url,timestamp FROM accesslog WHERE url NOT LIKE '%". check_input($HTTP_HOST) ."%' AND url <> '' ORDER BY timestamp DESC"; + $query = "SELECT url,timestamp FROM accesslog WHERE url NOT LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' AND url <> '' ORDER BY timestamp DESC"; $describe = "external "; } @@ -447,7 +448,7 @@ function statistics_recent_refer($nid = 0) { function statistics_top_refer($nid = 0) { - global $HTTP_HOST, $view; + global $view; $node = node_load(array("nid" => $nid)); @@ -456,12 +457,12 @@ function statistics_top_refer($nid = 0) { $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url <> '' GROUP BY url ORDER BY count DESC"; } elseif ($view == "internal") { - $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url LIKE '%". check_input($HTTP_HOST) ."%' GROUP BY url ORDER BY count DESC"; + $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' GROUP BY url ORDER BY count DESC"; $describe = "internal "; } else { /* default to external */ - $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url NOT LIKE '%". check_input($HTTP_HOST) ."%' AND url <> '' GROUP BY url ORDER BY count DESC"; + $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url NOT LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' AND url <> '' GROUP BY url ORDER BY count DESC"; $describe = "external "; } @@ -472,12 +473,12 @@ function statistics_top_refer($nid = 0) { $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url <> '' GROUP BY url ORDER BY count DESC"; } elseif ($view == "internal") { - $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url LIKE '%". check_input($HTTP_HOST) ."%' GROUP BY url ORDER BY count DESC"; + $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' GROUP BY url ORDER BY count DESC"; $describe = "internal "; } else { /* default to external */ - $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url NOT LIKE '%". check_input($HTTP_HOST) ."%' AND url <> '' GROUP BY url ORDER BY count DESC"; + $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url NOT LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' AND url <> '' GROUP BY url ORDER BY count DESC"; $describe = "external "; } diff --git a/modules/statistics/statistics.module b/modules/statistics/statistics.module index 4c6a6fb7799..f227baabfff 100644 --- a/modules/statistics/statistics.module +++ b/modules/statistics/statistics.module @@ -259,7 +259,8 @@ function statistics_help() { /* Administration hook, defines module's administrative page */ function statistics_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; /* Only allow people with sufficient access. */ if ((user_access("administer statistics module")) || (user_access("administer statistics"))) { @@ -393,7 +394,7 @@ function statistics_admin_accesslog_table($type, $id) { function statistics_recent_refer($nid = 0) { - global $HTTP_HOST, $view; + global $view; if (empty($view)) { $view = arg(3); @@ -406,12 +407,12 @@ function statistics_recent_refer($nid = 0) { $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url <> '' ORDER BY timestamp DESC"; } elseif ($view == "internal") { - $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url LIKE '%". check_input($HTTP_HOST) ."%' ORDER BY timestamp DESC"; + $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' ORDER BY timestamp DESC"; $describe = "internal "; } else { /* default to external referrers */ - $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url NOT LIKE '%". check_input($HTTP_HOST) ."%' AND url <> '' ORDER BY timestamp DESC"; + $query = "SELECT url,timestamp FROM accesslog WHERE nid='$nid' AND url NOT LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' AND url <> '' ORDER BY timestamp DESC"; $describe = "external "; } @@ -423,11 +424,11 @@ function statistics_recent_refer($nid = 0) { $query = "SELECT url,timestamp FROM accesslog WHERE url <> '' ORDER BY timestamp DESC"; } elseif ($view == "internal") { - $query = "SELECT url,timestamp FROM accesslog WHERE url LIKE '%". check_input($HTTP_HOST) ."%' ORDER BY timestamp DESC"; + $query = "SELECT url,timestamp FROM accesslog WHERE url LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' ORDER BY timestamp DESC"; $describe = "internal "; } else { - $query = "SELECT url,timestamp FROM accesslog WHERE url NOT LIKE '%". check_input($HTTP_HOST) ."%' AND url <> '' ORDER BY timestamp DESC"; + $query = "SELECT url,timestamp FROM accesslog WHERE url NOT LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' AND url <> '' ORDER BY timestamp DESC"; $describe = "external "; } @@ -447,7 +448,7 @@ function statistics_recent_refer($nid = 0) { function statistics_top_refer($nid = 0) { - global $HTTP_HOST, $view; + global $view; $node = node_load(array("nid" => $nid)); @@ -456,12 +457,12 @@ function statistics_top_refer($nid = 0) { $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url <> '' GROUP BY url ORDER BY count DESC"; } elseif ($view == "internal") { - $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url LIKE '%". check_input($HTTP_HOST) ."%' GROUP BY url ORDER BY count DESC"; + $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' GROUP BY url ORDER BY count DESC"; $describe = "internal "; } else { /* default to external */ - $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url NOT LIKE '%". check_input($HTTP_HOST) ."%' AND url <> '' GROUP BY url ORDER BY count DESC"; + $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE nid = '$nid' AND url NOT LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' AND url <> '' GROUP BY url ORDER BY count DESC"; $describe = "external "; } @@ -472,12 +473,12 @@ function statistics_top_refer($nid = 0) { $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url <> '' GROUP BY url ORDER BY count DESC"; } elseif ($view == "internal") { - $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url LIKE '%". check_input($HTTP_HOST) ."%' GROUP BY url ORDER BY count DESC"; + $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' GROUP BY url ORDER BY count DESC"; $describe = "internal "; } else { /* default to external */ - $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url NOT LIKE '%". check_input($HTTP_HOST) ."%' AND url <> '' GROUP BY url ORDER BY count DESC"; + $query = "SELECT url, COUNT(url) AS count FROM accesslog WHERE url NOT LIKE '%". check_input($_SERVER["HTTP_HOST"]) ."%' AND url <> '' GROUP BY url ORDER BY count DESC"; $describe = "external "; } diff --git a/modules/system.module b/modules/system.module index a0e69bbeaa9..3158c24d7c0 100644 --- a/modules/system.module +++ b/modules/system.module @@ -310,7 +310,9 @@ function system_listing($type, $directory, $required = array()) { } function system_admin() { - global $op, $edit; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer site configuration")) { if ($op == t("Reset to defaults")) { diff --git a/modules/system/system.module b/modules/system/system.module index a0e69bbeaa9..3158c24d7c0 100644 --- a/modules/system/system.module +++ b/modules/system/system.module @@ -310,7 +310,9 @@ function system_listing($type, $directory, $required = array()) { } function system_admin() { - global $op, $edit; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer site configuration")) { if ($op == t("Reset to defaults")) { diff --git a/modules/taxonomy.module b/modules/taxonomy.module index 02ad8161030..c6d9f810381 100644 --- a/modules/taxonomy.module +++ b/modules/taxonomy.module @@ -730,7 +730,8 @@ function taxonomy_page() { */ function taxonomy_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer taxonomy")) { if (empty($op)) { diff --git a/modules/taxonomy/taxonomy.module b/modules/taxonomy/taxonomy.module index 02ad8161030..c6d9f810381 100644 --- a/modules/taxonomy/taxonomy.module +++ b/modules/taxonomy/taxonomy.module @@ -730,7 +730,8 @@ function taxonomy_page() { */ function taxonomy_admin() { - global $op, $edit; + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer taxonomy")) { if (empty($op)) { diff --git a/modules/user.module b/modules/user.module index 8db17ffb5bc..5782a278266 100644 --- a/modules/user.module +++ b/modules/user.module @@ -27,17 +27,15 @@ function sess_read($key) { } function sess_write($key, $value) { - global $HTTP_SERVER_VARS; - db_query("UPDATE users SET hostname = '%s', session = '%s', timestamp = %d WHERE sid = '$key'", $HTTP_SERVER_VARS["REMOTE_ADDR"], $value, time()); + db_query("UPDATE users SET hostname = '%s', session = '%s', timestamp = %d WHERE sid = '$key'", $_SERVER["REMOTE_ADDR"], $value, time()); return ''; } function sess_destroy($key) { - global $HTTP_SERVER_VARS; - db_query("UPDATE users SET hostname = '%s', timestamp = %d, sid = '' WHERE sid = '$key'", $HTTP_SERVER_VARS["REMOTE_ADDR"], time()); + db_query("UPDATE users SET hostname = '%s', timestamp = %d, sid = '' WHERE sid = '$key'", $_SERVER["REMOTE_ADDR"], time()); } function sess_gc($lifetime) { @@ -288,16 +286,18 @@ function user_perm() { } function user_search($keys) { - global $PHP_SELF; + $result = db_query_range("SELECT * FROM users WHERE name LIKE '%$keys%'", 0, 20); while ($account = db_fetch_object($result)) { - $find[$i++] = array("title" => $account->name, "link" => (strstr($PHP_SELF, "admin") ? url("admin/user/edit/$account->uid") : url("user/view/$account->uid")), "user" => $account->name); + $find[$i++] = array("title" => $account->name, "link" => (strstr($_SERVER["PHP_SELF"], "admin") ? url("admin/user/edit/$account->uid") : url("user/view/$account->uid")), "user" => $account->name); } return $find; } function user_block($op = "list", $delta = 0) { - global $user, $edit; + global $user; + + $edit = $_POST["edit"]; if ($op == "list") { $blocks[0]["info"] = t("Log in"); @@ -505,7 +505,7 @@ function user_auth_help_links() { /*** User features *********************************************************/ function user_login($edit = array(), $msg = "") { - global $user, $referer; + global $user; /* ** If we are already logged on, go to the user page instead. @@ -746,6 +746,7 @@ function user_pass($edit = array()) { function user_register($edit = array()) { global $user, $base_url; + $edit = $_POST["edit"]; /* ** If we are already logged on, go to the user page instead. */ @@ -871,7 +872,9 @@ function user_register($edit = array()) { function user_delete() { - global $edit, $user; + global $user; + + $edit = $_SERVER["edit"]; if ($edit["confirm"]) { watchdog("user","$user->name deactivated her own account."); @@ -1060,7 +1063,9 @@ function user_view($uid = 0) { } function user_page() { - global $edit, $op; + + $edit = $_POST["edit"]; + $op = $_POST["op"]; if (empty($op)) { $op = arg(1); @@ -1185,8 +1190,8 @@ function user_admin_create($edit = array()) { } function user_admin_access($edit = array()) { - global $op, $id, $type; + $op = $_POST["op"]; $type = arg(3); $id = arg(4); @@ -1329,8 +1334,8 @@ function user_admin_perm($edit = array()) { } function user_admin_role($edit = array()) { - global $op; + $op = $_POST["op"]; $id = arg(3); if ($op == t("Save role")) { @@ -1378,8 +1383,8 @@ function user_admin_role($edit = array()) { } function user_admin_edit($edit = array()) { - global $op; + $op = $_SERVER["op"]; $id = arg(3); if ($account = user_load(array("uid" => $id))) { @@ -1520,7 +1525,9 @@ function user_role_init() { } function user_admin() { - global $op, $edit; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer users")) { /* diff --git a/modules/user/user.module b/modules/user/user.module index 8db17ffb5bc..5782a278266 100644 --- a/modules/user/user.module +++ b/modules/user/user.module @@ -27,17 +27,15 @@ function sess_read($key) { } function sess_write($key, $value) { - global $HTTP_SERVER_VARS; - db_query("UPDATE users SET hostname = '%s', session = '%s', timestamp = %d WHERE sid = '$key'", $HTTP_SERVER_VARS["REMOTE_ADDR"], $value, time()); + db_query("UPDATE users SET hostname = '%s', session = '%s', timestamp = %d WHERE sid = '$key'", $_SERVER["REMOTE_ADDR"], $value, time()); return ''; } function sess_destroy($key) { - global $HTTP_SERVER_VARS; - db_query("UPDATE users SET hostname = '%s', timestamp = %d, sid = '' WHERE sid = '$key'", $HTTP_SERVER_VARS["REMOTE_ADDR"], time()); + db_query("UPDATE users SET hostname = '%s', timestamp = %d, sid = '' WHERE sid = '$key'", $_SERVER["REMOTE_ADDR"], time()); } function sess_gc($lifetime) { @@ -288,16 +286,18 @@ function user_perm() { } function user_search($keys) { - global $PHP_SELF; + $result = db_query_range("SELECT * FROM users WHERE name LIKE '%$keys%'", 0, 20); while ($account = db_fetch_object($result)) { - $find[$i++] = array("title" => $account->name, "link" => (strstr($PHP_SELF, "admin") ? url("admin/user/edit/$account->uid") : url("user/view/$account->uid")), "user" => $account->name); + $find[$i++] = array("title" => $account->name, "link" => (strstr($_SERVER["PHP_SELF"], "admin") ? url("admin/user/edit/$account->uid") : url("user/view/$account->uid")), "user" => $account->name); } return $find; } function user_block($op = "list", $delta = 0) { - global $user, $edit; + global $user; + + $edit = $_POST["edit"]; if ($op == "list") { $blocks[0]["info"] = t("Log in"); @@ -505,7 +505,7 @@ function user_auth_help_links() { /*** User features *********************************************************/ function user_login($edit = array(), $msg = "") { - global $user, $referer; + global $user; /* ** If we are already logged on, go to the user page instead. @@ -746,6 +746,7 @@ function user_pass($edit = array()) { function user_register($edit = array()) { global $user, $base_url; + $edit = $_POST["edit"]; /* ** If we are already logged on, go to the user page instead. */ @@ -871,7 +872,9 @@ function user_register($edit = array()) { function user_delete() { - global $edit, $user; + global $user; + + $edit = $_SERVER["edit"]; if ($edit["confirm"]) { watchdog("user","$user->name deactivated her own account."); @@ -1060,7 +1063,9 @@ function user_view($uid = 0) { } function user_page() { - global $edit, $op; + + $edit = $_POST["edit"]; + $op = $_POST["op"]; if (empty($op)) { $op = arg(1); @@ -1185,8 +1190,8 @@ function user_admin_create($edit = array()) { } function user_admin_access($edit = array()) { - global $op, $id, $type; + $op = $_POST["op"]; $type = arg(3); $id = arg(4); @@ -1329,8 +1334,8 @@ function user_admin_perm($edit = array()) { } function user_admin_role($edit = array()) { - global $op; + $op = $_POST["op"]; $id = arg(3); if ($op == t("Save role")) { @@ -1378,8 +1383,8 @@ function user_admin_role($edit = array()) { } function user_admin_edit($edit = array()) { - global $op; + $op = $_SERVER["op"]; $id = arg(3); if ($account = user_load(array("uid" => $id))) { @@ -1520,7 +1525,9 @@ function user_role_init() { } function user_admin() { - global $op, $edit; + + $op = $_POST["op"]; + $edit = $_POST["edit"]; if (user_access("administer users")) { /* diff --git a/scripts/code-clean.sh b/scripts/code-clean.sh index 462fc3d1727..3053fff1b99 100644 --- a/scripts/code-clean.sh +++ b/scripts/code-clean.sh @@ -4,5 +4,5 @@ find . -name ".#*" -type f | xargs rm -f find . -name "*.rej" -type f | xargs rm -f find . -name "*.orig" -type f | xargs rm -f find . -name "DEADJOE" -type f | xargs rm -f -find . -name "*" -type f | grep -v ".psp" | grep -v ".gif" | grep -v ".jpg" | grep -v ".png" | grep -v ".tgz" | xargs perl -wi -pe 's/\s+$/\n/' -find . -name "*" -type f | grep -v ".psp" | grep -v ".gif" | grep -v ".jpg" | grep -v ".png" | grep -v ".tgz" | xargs perl -wi -pe 's/\t/ /g' +find . -type f | grep -v ".psp" | grep -v ".gif" | grep -v ".jpg" | grep -v ".png" | grep -v ".tgz" | xargs perl -wi -pe 's/\s+$/\n/' +find . -type f | grep -v ".psp" | grep -v ".gif" | grep -v ".jpg" | grep -v ".png" | grep -v ".tgz" | xargs perl -wi -pe 's/\t/ /g' diff --git a/themes/marvin/marvin.theme b/themes/marvin/marvin.theme index abc58227884..6614ade7f0b 100644 --- a/themes/marvin/marvin.theme +++ b/themes/marvin/marvin.theme @@ -27,7 +27,7 @@ } function header($title = "") { - global $HTTP_USER_AGENT, $base_url; + global $base_url; ?> @@ -48,7 +48,7 @@