Revert globals patch pending further work

diff --git a/includes/Article.php b/includes/Article.php
index c988c80..7592c7b 100644 (file)
--- a/includes/Article.php
+++ b/includes/Article.php
@@ -42,8 +42,6 @@ class Article {
        function getContent( $noredir = false )
        {
                global $action,$section,$count; # From query string
-               $section = $_REQUEST["section"];
-               $count   = $_REQUEST['count'];
                $fname =  "Article::getContent"; 
                wfProfileIn( $fname );
 
@@ -702,8 +700,6 @@ class Article {
        {
                global $wgUser, $wgOut;
                global $wpConfirm, $wpReason, $image, $oldimage;
-               $wpReason  = $_REQUEST["wpReason"];
-               $wpConfirm = $_REQUEST["wpConfirm"];
 
                # This code desperately needs to be totally rewritten
                
@@ -956,7 +952,6 @@ class Article {
        function rollback()
        {
                global $wgUser, $wgLang, $wgOut, $from;
-               $from = $_REQUEST["from"];
 
                if ( ! $wgUser->isSysop() ) {
                        $wgOut->sysopRequired();

diff --git a/includes/EditPage.php b/includes/EditPage.php
index 8879d61..43aafdf 100644 (file)
--- a/includes/EditPage.php
+++ b/includes/EditPage.php
@@ -22,16 +22,6 @@ class EditPage {
                global $wgOut, $wgUser, $wgWhitelistEdit;
                global $wpTextbox1, $wpSummary, $wpSave, $wpPreview;
                global $wpMinoredit, $wpEdittime, $wpTextbox2;
-               global $wpSection , $wpWatchthis;
-               $wpTextbox1 = $_REQUEST["wpTextbox1"];
-               $wpSummary  = $_REQUEST["wpSummary"];
-               $wpSave     = $_REQUEST["wpSave"];
-               $wpPreview  = $_REQUEST["wpPreview"];
-               $wpMinoredit= $_REQUEST["wpMinoredit"];
-               $wpEdittime = $_REQUEST["wpEdittime"];
-               $wpTextbox2 = $_REQUEST["wpTextbox2"];
-               $wpWatchthis = $_REQUEST["wpWatchthis"];
-               $wpSection = $_REQUEST["wpSection"];
 
                $fields = array( "wpTextbox1", "wpSummary", "wpTextbox2" );
                wfCleanFormFields( $fields );

diff --git a/includes/Setup.php b/includes/Setup.php
index 5306ca5..3079dc5 100644 (file)
--- a/includes/Setup.php
+++ b/includes/Setup.php
@@ -91,15 +91,10 @@ if( !$wgCommandLineMode ) {
        session_set_cookie_params( 0, $wgCookiePath, $wgCookieDomain );
        session_cache_limiter( "private, must-revalidate" );
        session_start();
-       global $wsUserID, $wsUserName, $wsUserPassword, $wsUploadFiles;
-       # Reaad back session variables
-       $wsUserID       = $_SESSION["wsUserID"];
-       $wsUserName     = $_SESSION["wsUserName"];
-       $wsUserPassword = $_SESSION["wsUserPassword"];
-       $wsUploadFiles  = $_SESSION["wsUploadFiles"];
-
-
-
+       session_register( "wsUserID" );
+       session_register( "wsUserName" );
+       session_register( "wsUserPassword" );
+       session_register( "wsUploadFiles" );
 }
 
 $wgUser = User::loadFromSession();

diff --git a/includes/SpecialAsksql.php b/includes/SpecialAsksql.php
index a52aa1f..a4ecf02 100644 (file)
--- a/includes/SpecialAsksql.php
+++ b/includes/SpecialAsksql.php
@@ -23,7 +23,6 @@ class SqlQueryForm {
                global $wgOut, $wgUser, $wgLang;
                global $wpSqlQuery;
                global $wgLogQueries;
-               $wpSqlQuery = $_REQUEST["wpSqlQuery"];
 
                $wgOut->setPagetitle( wfMsg( "asksql" ) );
                $note = wfMsg( "asksqltext" );
@@ -62,7 +61,6 @@ class SqlQueryForm {
                global $wgOut, $wgUser, $wgServer, $wgScript, $wgArticlePath, $wgLang;
                global $wpSqlQuery;
                global $wgDBsqluser, $wgDBsqlpassword;
-               $wpSqlQuery = $_REQUEST["wpSqlQuery"];
 
                # Use a limit, folks!
                $wpSqlQuery = trim( $wpSqlQuery );

diff --git a/includes/SpecialBlockip.php b/includes/SpecialBlockip.php
index 6851d99..f03e535 100644 (file)
--- a/includes/SpecialBlockip.php
+++ b/includes/SpecialBlockip.php
@@ -23,9 +23,6 @@ class IPBlockForm {
        {
                global $wgOut, $wgUser, $wgLang;
                global $ip, $wpBlockAddress, $wpBlockReason;
-               $wpBlockAddress  = $_REQUEST["wpBlockAddress"];
-               $wpBlockReason   = $_REQUEST["wpBlockReason"];
-               $ip              = $_REQUEST["ip"];
 
                $wgOut->setPagetitle( wfMsg( "blockip" ) );
                $wgOut->addWikiText( wfMsg( "blockiptext" ) );
@@ -63,9 +60,6 @@ class IPBlockForm {
        {
                global $wgOut, $wgUser, $wgLang;
                global $ip, $wpBlockAddress, $wpBlockReason, $wgSysopUserBans;
-               $wpBlockAddress  = $_REQUEST["wpBlockAddress"];
-               $wpBlockReason   = $_REQUEST["wpBlockReason"];
-               $ip              = $_REQUEST["ip"];
                
                $userId = 0;
                $wpBlockAddress = trim( $wpBlockAddress );
@@ -109,10 +103,11 @@ class IPBlockForm {
        function showSuccess()
        {
                global $wgOut, $wgUser;
+               global $ip;
 
                $wgOut->setPagetitle( wfMsg( "blockip" ) );
                $wgOut->setSubtitle( wfMsg( "blockipsuccesssub" ) );
-               $text = wfMsg( "blockipsuccesstext", $_REQUEST["ip"] );
+               $text = wfMsg( "blockipsuccesstext", $ip );
                $wgOut->addWikiText( $text );
        }
 }

diff --git a/includes/SpecialEmailuser.php b/includes/SpecialEmailuser.php
index 4140f14..ababac2 100644 (file)
--- a/includes/SpecialEmailuser.php
+++ b/includes/SpecialEmailuser.php
@@ -53,8 +53,6 @@ class EmailUserForm {
        {
                global $wgOut, $wgUser, $wgLang;
                global $wpSubject, $wpText, $target;
-               $wpSubject = $_REQUEST["wpSubject"];
-               $wpText    = $_REQUEST["wpText"];
 
                $wgOut->setPagetitle( wfMsg( "emailpage" ) );
                $wgOut->addWikiText( wfMsg( "emailpagetext" ) );
@@ -107,11 +105,9 @@ class EmailUserForm {
        {
                global $wgOut, $wgUser, $wgLang, $wgOutputEncoding;
                global $wpSubject, $wpText, $target;
-               $wpSubject = $_REQUEST["wpSubject"];
-               $wpText    = $_REQUEST["wpText"];
            
                $from = wfQuotedPrintable( $wgUser->getName() ) . " <" . $wgUser->getEmail() . ">";
-               $to = wfQuotedPrintable( $this->mAddress );
+           $to = wfQuotedPrintable( $this->mAddress );
 
                $headers =
                        "MIME-Version: 1.0\r\n" .

diff --git a/includes/SpecialImagelist.php b/includes/SpecialImagelist.php
index d7fde56..9224467 100644 (file)
--- a/includes/SpecialImagelist.php
+++ b/includes/SpecialImagelist.php
@@ -4,9 +4,6 @@ function wfSpecialImagelist()
 {
        global $wgUser, $wgOut, $wgLang, $sort;
        global $wpIlMatch, $wpIlSubmit;
-       $sort        = $_REQUEST['sort'];
-       $wpIlMatch   = $_REQUEST["wpIlMatch"];
-       $wpIlSubmit  = $_REQUEST["wpIlSubmit"];
 
        $fields = array( 'wpIlMatch' );
        wfCleanFormFields( $fields );

diff --git a/includes/SpecialPreferences.php b/includes/SpecialPreferences.php
index 9069347..6c0c396 100644 (file)
--- a/includes/SpecialPreferences.php
+++ b/includes/SpecialPreferences.php
@@ -65,25 +65,6 @@ function wfSpecialPreferences()
        global $wpSkin, $wpMath, $wpDate, $wpUserEmail, $wpEmailFlag, $wpNick, $wpSearch, $wpRecent;
        global $wpSearchLines, $wpSearchChars, $wpStubs;
        global $wpRows, $wpCols, $wpHourDiff, $HTTP_POST_VARS;
-       $wpQuickbar    = $_REQUEST["wpQuickbar"];
-       $wpOldpass     = $_REQUEST["wpOldpass"];
-       $wpNewpass     = $_REQUEST["wpNewpass"];
-       $wpRetypePass  = $_REQUEST["wpRetypePass"];
-       $wpSkin        = $_REQUEST["wpSkin"];
-       $wpMath        = $_REQUEST["wpMath"];
-       $wpDate        = $_REQUEST["wpDate"];
-       $wpUserEmail   = $_REQUEST["wpUserEmail"];
-       $wpEmailFlag   = $_REQUEST["wpEmailFlag"];
-       $wpNick        = $_REQUEST["wpNick"];
-       $wpSearch      = $_REQUEST["wpSearch"];
-       $wpRecent      = $_REQUEST["wpRecent"];
-       $wpSearchLines = $_REQUEST["wpSearchLines"];
-       $wpSearchChars = $_REQUEST["wpSearchChars"];
-       $wpStubs       = $_REQUEST["wpStubs"];
-       $wpRows        = $_REQUEST["wpRows"];
-       $wpCols        = $_REQUEST["wpCols"];
-       $wpHourDiff    = $_REQUEST["wpHourDiff"];
-
 
        if ( "" != $wpNewpass ) {
                if ( $wpNewpass != $wpRetypePass ) {
@@ -120,7 +101,8 @@ function wfSpecialPreferences()
        foreach ( $namespaces as $i => $namespaces ) {
                if ( $i >= 0 ) {
                        $nsvar = "wpNs$i";
-                       $wgUser->setOption( "searchNs{$i}", validateCheckbox( $_REQUEST[$nsvar] ) );
+                       global $$nsvar;
+                       $wgUser->setOption( "searchNs{$i}", validateCheckbox( $$nsvar ) );
                }
        }
 

diff --git a/includes/SpecialRecentchanges.php b/includes/SpecialRecentchanges.php
index bb3855a..c0fa2ca 100644 (file)
--- a/includes/SpecialRecentchanges.php
+++ b/includes/SpecialRecentchanges.php
@@ -4,11 +4,6 @@ function wfSpecialRecentchanges( $par )
 {
        global $wgUser, $wgOut, $wgLang, $wgTitle;
        global $days, $hideminor, $from, $hidebots; # From query string
-       $days      = $_REQUEST["days"];
-       $hideminor = $_REQUEST["hideminor"];
-       $from      = $_REQUEST["from"];
-       $hidebots  = $_REQUEST["hidebots"];
-
        $fname = "wfSpecialRecentchanges";
 
        if( $par ) {

diff --git a/includes/SpecialRecentchangeslinked.php b/includes/SpecialRecentchangeslinked.php
index fa86e3a..458cfe6 100644 (file)
--- a/includes/SpecialRecentchangeslinked.php
+++ b/includes/SpecialRecentchangeslinked.php
@@ -5,10 +5,6 @@ function wfSpecialRecentchangeslinked( $par = NULL )
 {
        global $wgUser, $wgOut, $wgLang, $wgTitle;
        global $days, $target, $hideminor; # From query string
-       $days      = $_REQUEST["days"];
-       $hideminor = $_REQUEST["hideminor"];
-       $from      = $_REQUEST["from"];
-
        $fname = "wfSpecialRecentchangeslinked";
 
        $wgOut->setPagetitle( wfMsg( "recentchanges" ) );

diff --git a/includes/SpecialUndelete.php b/includes/SpecialUndelete.php
index 8d308d3..6c8a24b 100644 (file)
--- a/includes/SpecialUndelete.php
+++ b/includes/SpecialUndelete.php
@@ -3,8 +3,6 @@
 function wfSpecialUndelete( $par )
 {
     global $wgLang, $wgUser, $wgOut, $action, $target, $timestamp, $restore;
-    $restore   = $_REQUEST["restore"];
-    $timestamp = $_REQUEST["timestamp"];
     
        if( $par != "" ) $target = $par;
     if( isset($target ) ) {

diff --git a/includes/SpecialUnusedimages.php b/includes/SpecialUnusedimages.php
index 9b32f55..a594be0 100644 (file)
--- a/includes/SpecialUnusedimages.php
+++ b/includes/SpecialUnusedimages.php
@@ -4,9 +4,6 @@ function wfSpecialUnusedimages()
 {
        global $wgUser, $wgOut, $wgLang, $wgTitle;
        global $limit, $offset; # From query string
-       $limit  = $_REQUEST["limit"];
-       $offset = $_REQUEST["offset"];
-
        $fname = "wfSpecialUnusedimages";
 
        list( $limit, $offset ) = wfCheckLimits();

diff --git a/includes/SpecialUpload.php b/includes/SpecialUpload.php
index 5f69dc2..16ab982 100644 (file)
--- a/includes/SpecialUpload.php
+++ b/includes/SpecialUpload.php
@@ -4,8 +4,6 @@ function wfSpecialUpload()
 {
        global $wgUser, $wgOut, $wpUpload, $wpReUpload, $action;
        global $wgDisableUploads;
-       $wpUpload   = $_REQUEST["wpUpload"];
-       $wpReUpload = $_REQUEST["wpReUpload"];
        
        $fields = array( "wpUploadFile", "wpUploadDescription" );
        wfCleanFormFields( $fields );
@@ -41,16 +39,6 @@ function processUpload()
        global $wpUploadSaveName, $wpUploadTempName, $wpUploadSize;
        global $wgSavedFile, $wgUploadOldVersion, $wpUploadOldVersion;
        global $wgUseCopyrightUpload , $wpUploadCopyStatus , $wpUploadSource ;
-       $wpUploadAffirm       = $_REQUEST["wpUploadAffirm"];
-       $wpUploadFile         = $_REQUEST["wpUploadFile"];
-       $wpUploadDescription  = $_REQUEST["wpUploadDescription"];
-       $wpIgnoreWarning      = $_REQUEST["wpIgnoreWarning"];
-       $wpUploadSaveName     = $_REQUEST["wpUploadSaveName"];
-       $wpUploadTempName     = $_REQUEST["wpUploadTempName"];
-       $wpUploadSize         = $_REQUEST["wpUploadSize"];
-       $wpUploadOldVersion   = $_REQUEST["wpUploadOldVersion"];
-       $wpUploadCopyStatus   = $_REQUEST["wpUploadCopyStatus"];
-       $wpUploadSource       = $_REQUEST["wpUploadSource"];
 
        if ( $wgUseCopyrightUpload )
          {
@@ -156,7 +144,6 @@ function unsaveUploadedFile()
 {
        global $wpSessionKey, $wpUploadOldVersion;
        global $wgUploadDirectory, $wgOut, $wsUploadFiles;
-       $wpSessionKey       = $_REQUEST["wpSessionKey"];
        
        $wgSavedFile = $wsUploadFiles[$wpSessionKey];
        $wgUploadOldVersion = $wpUploadOldVersion;
@@ -186,7 +173,6 @@ function uploadWarning( $warning )
        global $wgSavedFile, $wgUploadOldVersion;
        global $wpSessionKey, $wpUploadOldVersion, $wsUploadFiles;
        global $wgUseCopyrightUpload , $wpUploadCopyStatus , $wpUploadSource ;
-       $wpSessionKey       = $_REQUEST["wpSessionKey"];
 
        # wgSavedFile is stored in the session not the form, for security
        $wpSessionKey = mt_rand( 0, 0x7fffffff );
@@ -239,14 +225,6 @@ function mainUploadForm( $msg )
        global $wpUploadDescription, $wpIgnoreWarning;
        global $wgUseCopyrightUpload , $wpUploadSource , $wpUploadCopyStatus ;
 
-       $wpUpload            = $_REQUEST["wpUpload"];
-       $wpUploadAffirm      = $_REQUEST["wpUploadAffirm"];
-       $wpUploadFile        = $_REQUEST["wpUploadFile"];
-       $wpUploadDescription = $_REQUEST["wpUploadDescription"];
-       $wpIgnoreWarning     = $_REQUEST["wpIgnoreWarning"];
-       $wpUploadSource      = $_REQUEST["wpUploadSource"];
-       $wpUploadCopyStatus  = $_REQUEST["wpUploadCopyStatus"];
-
        if ( "" != $msg ) {
                $sub = wfMsg( "uploaderror" );
                $wgOut->addHTML( "<h2>{$sub}</h2>\n" .

diff --git a/includes/SpecialUserlogin.php b/includes/SpecialUserlogin.php
index 96e553c..f84f640 100644 (file)
--- a/includes/SpecialUserlogin.php
+++ b/includes/SpecialUserlogin.php
@@ -5,10 +5,6 @@ function wfSpecialUserlogin()
        global $wpCreateaccount, $wpCreateaccountMail;
        global $wpLoginattempt, $wpMailmypassword;
        global $action;
-       $wpCreateaccount     = $_REQUEST["wpCreateaccount"];
-       $wpCreateaccountMail = $_REQUEST["wpCreateaccountMail"];
-       $wpLoginattempt      = $_REQUEST["wpLoginattempt"];
-       $wpMailmypassword    = $_REQUEST["wpMailmypassword"];
 
        $fields = array( "wpName", "wpPassword", "wpName",
          "wpPassword", "wpRetype", "wpEmail" );
@@ -31,8 +27,6 @@ function wfSpecialUserlogin()
 /* private */ function addNewAccountMailPassword()
 {
        global $wgOut, $wpEmail, $wpName;
-       $wpEmail = $_REQUEST["wpEmail"];
-       $wpName  = $_REQUEST["wpName"];
        
        if ("" == $wpEmail) {
                mainLoginForm( wfMsg( "noemail", $wpName ) );
@@ -66,11 +60,6 @@ function wfSpecialUserlogin()
 {
        global $wgUser, $wgOut, $wpPassword, $wpRetype, $wpName, $wpRemember;
        global $wpEmail, $wgDeferredUpdateList;
-       $wpPassword = $_REQUEST["wpPassword"];
-       $wpRetype   = $_REQUEST["wpRetype"];
-       $wpName     = $_REQUEST["wpName"];
-       $wpRemember = $_REQUEST["wpRemember"];
-       $wpEmail    = $_REQUEST["wpEmail"];
 
        $u = addNewAccountInternal();
 
@@ -92,7 +81,6 @@ function wfSpecialUserlogin()
                 return;
         }
 
-
        if (!$wgUser->isAllowedToCreateAccount()) {
                userNotPrivilegedMessage();
                return;
@@ -137,9 +125,6 @@ function wfSpecialUserlogin()
 {
        global $wgUser, $wpName, $wpPassword, $wpRemember;
        global $returnto;
-       $wpPassword = $_REQUEST["wpPassword"];
-       $wpName     = $_REQUEST["wpName"];
-       $wpRemember = $_REQUEST["wpRemember"];
 
         if (!cookieCheck()) {
                 return;

diff --git a/includes/SpecialWatchlist.php b/includes/SpecialWatchlist.php
index 61c2aa1..7ec0fdd 100644 (file)
--- a/includes/SpecialWatchlist.php
+++ b/includes/SpecialWatchlist.php
@@ -6,10 +6,6 @@ function wfSpecialWatchlist()
 {
        global $wgUser, $wgOut, $wgLang, $wgTitle;
        global $days, $limit, $target; # From query string
-       $days      = $_REQUEST["days"];
-       $limit     = $_REQUEST["limit"];
-       $target    = $_REQUEST["target"];
-
        $fname = "wfSpecialWatchlist";
 
        $wgOut->setPagetitle( wfMsg( "watchlist" ) );

diff --git a/includes/User.php b/includes/User.php
index 0adb815..06be141 100644 (file)
--- a/includes/User.php
+++ b/includes/User.php
@@ -493,11 +493,6 @@ class User {
                } else {
                        setcookie( "{$wgDBname}Password", "", time() - 3600 );
                }
-               # Store Session variables
-               $_SESSION["wsUserID"]       = $wsUserID;
-               $_SESSION["wsUserName"]     = $wsUserName;
-               $_SESSION["wsUserPassword"] = $wsUserPassword;
-               $_SESSION["wsUploadFiles"]  = $wsUploadFiles;
        }
 
        function logout()