Revert to r14165 . Did too many changes, didnt even run parserTests (i am bad)
authorAntoine Musso <hashar@users.mediawiki.org>
Thu, 11 May 2006 22:40:38 +0000 (22:40 +0000)
committerAntoine Musso <hashar@users.mediawiki.org>
Thu, 11 May 2006 22:40:38 +0000 (22:40 +0000)
51 files changed:
includes/Article.php
includes/Block.php
includes/CacheManager.php
includes/ChangesList.php
includes/Database.php
includes/DifferenceEngine.php
includes/EditPage.php
includes/Exif.php
includes/Feed.php
includes/HistoryBlob.php
includes/Image.php
includes/JobQueue.php
includes/LinkBatch.php
includes/LinkCache.php
includes/LinksUpdate.php
includes/LoadBalancer.php
includes/LogPage.php
includes/MagicWord.php
includes/MessageCache.php
includes/OutputPage.php
includes/PageHistory.php
includes/Parser.php
includes/ParserCache.php
includes/QueryPage.php
includes/RecentChange.php
includes/Sanitizer.php
includes/SearchUpdate.php
includes/SiteConfiguration.php
includes/SiteStatsUpdate.php
includes/Skin.php
includes/SpecialAllpages.php
includes/SpecialBlockip.php
includes/SpecialBooksources.php
includes/SpecialContributions.php
includes/SpecialImport.php
includes/SpecialListusers.php
includes/SpecialLog.php
includes/SpecialNewpages.php
includes/SpecialPage.php
includes/SpecialPreferences.php
includes/SpecialUndelete.php
includes/SpecialUserlogin.php
includes/SpecialWantedpages.php
includes/SquidUpdate.php
includes/Title.php
includes/User.php
includes/WatchedItem.php
includes/WebRequest.php
includes/Wiki.php
languages/Language.php
languages/Messages.php

index 4d479f1..945f5c5 100644 (file)
@@ -26,32 +26,30 @@ class Article {
        /**@{{
         * @private
         */
-       private $mComment;                      //!<
-       private $mContent;                      //!<
-       private $mContentLoaded;        //!<
-       private $mCounter;                      //!<
-       private $mFileCache;            //!<
-       private $mForUpdate;            //!<
-       private $mGoodAdjustment;       //!<
-       private $mId;                           //!<
-       private $mLatest;                       //!<
-       private $mMinorEdit;            //!<
-       private $mOldId;                        //!<
-       private $mRedirectedFrom;       //!<
-       private $mRedirectUrl;          //!<
-       private $mRevIdFetched;         //!<
-       private $mRevision;                     //!<
-       private $mTable;                        //!<
-       private $mTimestamp;            //!<
-       private $mTotalAdjustment;      //!<
-       private $mUser;                         //!<
-       private $mUserText;                     //!<
+       var $mComment;                  //!<
+       var $mContent;                  //!<
+       var $mContentLoaded;    //!<
+       var $mCounter;                  //!<
+       var $mFileCache;                //!<
+       var $mForUpdate;                //!<
+       var $mGoodAdjustment;   //!<
+       var $mId;                               //!<
+       var $mLatest;                   //!<
+       var $mMinorEdit;                //!<
+       var $mOldId;                    //!<
+       var $mRedirectedFrom;   //!<
+       var $mRedirectUrl;              //!<
+       var $mRevIdFetched;             //!<
+       var $mRevision;                 //!<
+       var $mTable;                    //!<
+       var $mTimestamp;                //!<
+       var $mTitle;                    //!<
+       var $mTotalAdjustment;  //!<
+       var $mTouched;                  //!<
+       var $mUser;                             //!<
+       var $mUserText;                 //!<
        /**@}}*/
 
-       public  $mTitle;                        //!<
-       public  $mTouched;                      //!<
-
-
        /**
         * Constructor and clear the article
         * @param $title Reference to a Title object.
index 0a5f044..42969b6 100644 (file)
@@ -24,24 +24,9 @@ define ( 'EB_RANGE_ONLY', 4 );
  */
 class Block
 {
-       public
-               $mAddress,
-               $mAuto,
-               $mBy,
-               $mExpiry,
-               $mId,
-               $mRangeEnd,
-               $mRangeStart,
-               $mReason,
-               $mTimestamp,
-               $mUser ;
-
-       private
-               $mByName,
-               $mForUpdate,
-               $mFromMaster,
-               $mIntegerAddr,
-               $mNetworkBits ;
+       /* public*/ var $mAddress, $mUser, $mBy, $mReason, $mTimestamp, $mAuto, $mId, $mExpiry,
+                           $mRangeStart, $mRangeEnd;
+       /* private */ var $mNetworkBits, $mIntegerAddr, $mForUpdate, $mFromMaster, $mByName;
 
        function Block( $address = '', $user = '', $by = 0, $reason = '',
                $timestamp = '' , $auto = 0, $expiry = '' )
index 98a16ae..0d116f7 100644 (file)
@@ -24,9 +24,7 @@ require_once( 'Title.php' );
  * @package MediaWiki
  */
 class CacheManager {
-       private
-               $mTitle,
-               $mFileCache;
+       var $mTitle, $mFileCache;
 
        function CacheManager( &$title ) {
                $this->mTitle =& $title;
index f8c8585..59c410f 100644 (file)
@@ -14,19 +14,12 @@ require_once("RecentChange.php");
  */
 class RCCacheEntry extends RecentChange
 {
-       private
-               $curlink,
-               $difflink,
-               $lastlink,
-               $link,
-               $secureName,
-               $timestamp,
-               $userlink,
-               $usertalklink,
-               $versionlink,
-               $watched ;
-
-       function newFromParent( $rc ) {
+       var $secureName, $link;
+       var $curlink , $difflink, $lastlink , $usertalklink , $versionlink ;
+       var $userlink, $timestamp, $watched;
+
+       function newFromParent( $rc )
+       {
                $rc2 = new RCCacheEntry;
                $rc2->mAttribs = $rc->mAttribs;
                $rc2->mExtra = $rc->mExtra;
index 4ec9686..8376f06 100644 (file)
@@ -25,7 +25,7 @@ define( 'DEADLOCK_DELAY_MIN', 500000 );
 define( 'DEADLOCK_DELAY_MAX', 1500000 );
 
 class DBObject {
-       private $mData;
+       var $mData;
 
        function DBObject($data) {
                $this->mData = $data;
@@ -52,17 +52,17 @@ class Database {
        /**#@+
         * @private
         */
-       private $mLastQuery = '';
+       var $mLastQuery = '';
 
-       private $mServer, $mUser, $mPassword, $mConn = null, $mDBname;
-       private $mOut, $mOpened = false;
+       var $mServer, $mUser, $mPassword, $mConn = null, $mDBname;
+       var $mOut, $mOpened = false;
 
-       private $mFailFunction;
-       private $mTablePrefix;
-       private $mFlags;
-       private $mTrxLevel = 0;
-       private $mErrorCount = 0;
-       private $mLBInfo = array();
+       var $mFailFunction;
+       var $mTablePrefix;
+       var $mFlags;
+       var $mTrxLevel = 0;
+       var $mErrorCount = 0;
+       var $mLBInfo = array();
        /**#@-*/
 
 #------------------------------------------------------------------------------
@@ -1795,7 +1795,7 @@ class DatabaseMysql extends Database {
  * @package MediaWiki
  */
 class ResultWrapper {
-       private $db, $result;
+       var $db, $result;
 
        /**
         * @todo document
index 3a58392..15f5878 100644 (file)
@@ -21,24 +21,14 @@ class DifferenceEngine {
        /**#@+
         * @private
         */
-       private
-               $mNewid,
-               $mNewPage,
-               $mNewRev,
-               $mNewtext,
-               $mNewtitle,
-
-               $mOldid,
-               $mOldPage,
-               $mOldRev,
-               $mOldtext,
-               $mOldtitle,
-
-               $mPagetitle,
-               $mRcidMarkPatrolled,
-               $mRevisionsLoaded = false, // Have the revisions been loaded
-               $mTextLoaded = 0, // How many text blobs have been loaded, 0, 1 or 2?
-               $mTitle;
+       var $mOldid, $mNewid, $mTitle;
+       var $mOldtitle, $mNewtitle, $mPagetitle;
+       var $mOldtext, $mNewtext;
+       var $mOldPage, $mNewPage;
+       var $mRcidMarkPatrolled;
+       var $mOldRev, $mNewRev;
+       var $mRevisionsLoaded = false; // Have the revisions been loaded
+       var $mTextLoaded = 0; // How many text blobs have been loaded, 0, 1 or 2?
        /**#@-*/
 
        /**
@@ -603,10 +593,9 @@ define('USE_ASSERTS', function_exists('assert'));
  * @subpackage DifferenceEngine
  */
 class _DiffOp {
-       private
-               $closing,
-               $orig,
-               $type;
+       var $type;
+       var $orig;
+       var $closing;
 
        function reverse() {
                trigger_error('pure virtual', E_USER_ERROR);
@@ -628,8 +617,7 @@ class _DiffOp {
  * @subpackage DifferenceEngine
  */
 class _DiffOp_Copy extends _DiffOp {
-       public
-               $type = 'copy';
+       var $type = 'copy';
 
        function _DiffOp_Copy ($orig, $closing = false) {
                if (!is_array($closing))
@@ -650,7 +638,7 @@ class _DiffOp_Copy extends _DiffOp {
  * @subpackage DifferenceEngine
  */
 class _DiffOp_Delete extends _DiffOp {
-       public $type = 'delete';
+       var $type = 'delete';
 
        function _DiffOp_Delete ($lines) {
                $this->orig = $lines;
@@ -669,7 +657,7 @@ class _DiffOp_Delete extends _DiffOp {
  * @subpackage DifferenceEngine
  */
 class _DiffOp_Add extends _DiffOp {
-       public $type = 'add';
+       var $type = 'add';
 
        function _DiffOp_Add ($lines) {
                $this->closing = $lines;
@@ -688,8 +676,7 @@ class _DiffOp_Add extends _DiffOp {
  * @subpackage DifferenceEngine
  */
 class _DiffOp_Change extends _DiffOp {
-       private
-               $type = 'change';
+       var $type = 'change';
 
        function _DiffOp_Change ($orig, $closing) {
                $this->orig = $orig;
@@ -1148,7 +1135,7 @@ class _DiffEngine
  */
 class Diff
 {
-       public $edits;
+       var $edits;
 
        /**
         * Constructor.
@@ -1360,7 +1347,7 @@ class DiffFormatter
         * This should be left at zero for this class, but subclasses
         * may want to set this to other values.
         */
-       public $leading_context_lines = 0;
+       var $leading_context_lines = 0;
 
        /**
         * Number of trailing context "lines" to preserve.
@@ -1368,7 +1355,7 @@ class DiffFormatter
         * This should be left at zero for this class, but subclasses
         * may want to set this to other values.
         */
-       public $trailing_context_lines = 0;
+       var $trailing_context_lines = 0;
 
        /**
         * Format a diff.
index bc295ff..abb5e16 100644 (file)
  */
 
 class EditPage {
-       private
-               $allowBlankSummary = false,
-               $autoSumm = '',
-               $deletedSinceEdit = false,
-               $firsttime,
-               $formtype,
-               $hookError = '',
-               $isConflict = false,
-               $isCssJsSubpage = false,
-               $kblength = false,
-               $lastDelete,
-               $mArticle,
-               $missingComment = false,
-               $missingSummary = false,
-               $mMetaData = '',
-               $mTitle,
-               $mTokenOk = true,
-               $tooBig = false ;
+       var $mArticle;
+       var $mTitle;
+       var $mMetaData = '';
+       var $isConflict = false;
+       var $isCssJsSubpage = false;
+       var $deletedSinceEdit = false;
+       var $formtype;
+       var $firsttime;
+       var $lastDelete;
+       var $mTokenOk = true;
+       var $tooBig = false;
+       var $kblength = false;
+       var $missingComment = false;
+       var $missingSummary = false;
+       var $allowBlankSummary = false;
+       var $autoSumm = '';
+       var $hookError = '';
 
        # Form values
-       private
-               $diff = false,
-               $editintro = '',
-               $edittime = '',
-               $minoredit = false,
-               $oldid = 0,
-               $preview = false,
-               $recreate = false,
-               $save = false,
-               $scrolltop = null,
-               $section = '',
-               $starttime = '',
-               $summary = '',
-               $textbox1 = '',
-               $textbox2 = '',
-               $watchthis = false ;
+       var $save = false, $preview = false, $diff = false;
+       var $minoredit = false, $watchthis = false, $recreate = false;
+       var $textbox1 = '', $textbox2 = '', $summary = '';
+       var $edittime = '', $section = '', $starttime = '';
+       var $oldid = 0, $editintro = '', $scrolltop = null;
 
        /**
         * @todo document
@@ -400,7 +388,7 @@ class EditPage {
                        $this->watchthis = $request->getCheck( 'wpWatchthis' );
 
                        # Don't force edit summaries when a user is editing their own user or talk page
-                       if( ( $this->mTitle->getNamespace() == NS_USER || $this->mTitle->getNamespace() == NS_USER_TALK ) && $this->mTitle->getText() == $wgUser->getName() ) {
+                       if( ( $this->mTitle->mNamespace == NS_USER || $this->mTitle->mNamespace == NS_USER_TALK ) && $this->mTitle->getText() == $wgUser->getName() ) {
                                $this->allowBlankSummary = true;
                        } else {
                                $this->allowBlankSummary = $request->getBool( 'wpIgnoreBlankSummary' );
index d51a5f2..1009688 100644 (file)
@@ -55,29 +55,29 @@ class Exif {
         * is the value, in the case of more than one possible value type they are
         * seperated by commas.
         */
-       private $mExifTags;
+       var $mExifTags;
 
        /**
         * A one dimentional array of all Exif tags
         */
-       private $mFlatExifTags;
+       var $mFlatExifTags;
 
        /**
         * The raw Exif data returned by exif_read_data()
         */
-       private $mRawExifData;
+       var $mRawExifData;
 
        /**
         * A Filtered version of $mRawExifData that has been pruned of invalid
         * tags and tags that contain content they shouldn't contain according
         * to the Exif specification
         */
-       private $mFilteredExifData;
+       var $mFilteredExifData;
 
        /**
         * Filtered and formatted Exif data, see FormatExif::getFormattedData()
         */
-       private $mFormattedExifData;
+       var $mFormattedExifData;
 
        //@}
 
@@ -89,17 +89,17 @@ class Exif {
        /**
         * The file being processed
         */
-       private $file;
+       var $file;
 
        /**
         * The basename of the file being processed
         */
-       private $basename;
+       var $basename;
 
        /**
         * The private log to log to
         */
-       private $log = 'exif';
+       var $log = 'exif';
 
        //@}
 
@@ -605,7 +605,7 @@ class FormatExif {
         * @var array
         * @private
         */
-       private $mExif;
+       var $mExif;
 
        /**
         * Constructor
index fb32445..7663e82 100644 (file)
@@ -35,12 +35,11 @@ class FeedItem {
         * @var string
         * @private
         */
-       private
-               $Author = '',
-               $Date = '',
-               $Description = '',
-               $Title = 'Wiki',
-               $Url = '' ;
+       var $Title = 'Wiki';
+       var $Description = '';
+       var $Url = '';
+       var $Date = '';
+       var $Author = '';
        /**#@-*/
 
        /**#@+
index daa444c..4dee4da 100644 (file)
@@ -52,14 +52,10 @@ class HistoryBlob
  * The real object
  * @package MediaWiki
  */
-class ConcatenatedGzipHistoryBlob extends HistoryBlob {
-       private
-               $mCompressed = false,
-               $mDefaultHash = '',
-               $mFast = 0,
-               $mItems = array(),
-               $mSize = 0,
-               $mVersion = 0 ;
+class ConcatenatedGzipHistoryBlob extends HistoryBlob
+{
+       /* private */ var $mVersion = 0, $mCompressed = false, $mItems = array(), $mDefaultHash = '';
+       /* private */ var $mFast = 0, $mSize = 0;
 
        function ConcatenatedGzipHistoryBlob() {
                if ( !function_exists( 'gzdeflate' ) ) {
@@ -186,10 +182,7 @@ $wgBlobCache = array();
  * @package MediaWiki
  */
 class HistoryBlobStub {
-       private
-               $mHash,
-               $mOldId,
-               $mRef;
+       var $mOldId, $mHash, $mRef;
 
        /** @todo document */
        function HistoryBlobStub( $hash = '', $oldid = 0 ) {
@@ -285,8 +278,7 @@ class HistoryBlobStub {
  * @package MediaWiki
  */
 class HistoryBlobCurStub {
-       private
-               $mCurId;
+       var $mCurId;
 
        /** @todo document */
        function HistoryBlobCurStub( $curid = 0 ) {
index 2fcce3b..4547a8c 100644 (file)
@@ -32,8 +32,7 @@ class Image
        /**#@+
         * @private
         */
-       private
-               $name,          # name of the image (constructor)
+       var     $name,          # name of the image (constructor)
                $imagePath,     # Path of the image (loadFromXxx)
                $url,           # Image URL (accessor)
                $title,         # Title object for this image (constructor)
@@ -52,6 +51,7 @@ class Image
                $dataLoaded,    # Whether or not all this has been loaded from the database (loadFromXxx)
                $lastError;     # Error string associated with a thumbnail display error
 
+
        /**#@-*/
 
        /**
index ebb5d74..d589d8c 100644 (file)
@@ -5,8 +5,7 @@ if ( !defined( 'MEDIAWIKI' ) ) {
 }
 
 class Job {
-       private
-               $command,
+       var $command,
                $title,
                $params,
                $id,
index e385ede..e0f0f6f 100644 (file)
@@ -10,9 +10,8 @@
 class LinkBatch {
        /**
         * 2-d array, first index namespace, second index dbkey, value arbitrary
-        * @todo FIXME should it really be public ?
         */
-       public $data = array();
+       var $data = array();
 
        function LinkBatch( $arr = array() ) {
                foreach( $arr as $item ) {
index 355f157..451b3f0 100644 (file)
 class LinkCache {
        // Increment $mClassVer whenever old serialized versions of this class
        // becomes incompatible with the new version.
-       private
-               $mBadLinks,
-               $mClassVer = 3,
-               $mForUpdate,
-               $mGoodLinks,
-               $mPageLinks ;
+       /* private */ var $mClassVer = 3;
+
+       /* private */ var $mPageLinks;
+       /* private */ var $mGoodLinks, $mBadLinks;
+       /* private */ var $mForUpdate;
 
        /**
         * Get an instance of this class
index 9375de6..bd09502 100644 (file)
@@ -13,8 +13,7 @@ class LinksUpdate {
        /**@{{
         * @private
         */
-       private
-               $mId,            //!< Page ID of the article linked from
+       var $mId,            //!< Page ID of the article linked from
                $mTitle,         //!< Title object of the article linked from
                $mLinks,         //!< Map of title strings to IDs for the links in the document
                $mImages,        //!< DB keys of the images used, in the array key only
index 7206b7b..465f3b9 100644 (file)
@@ -32,23 +32,11 @@ define( 'AVG_STATUS_POLL', 2000 );
  * @package MediaWiki
  */
 class LoadBalancer {
-
-       private
-               $mAllowLagged,
-               $mConnections,
-               $mErrorConnection,
-               $mFailFunction,
-               $mForce,
-               $mGroupLoads,
-               $mLaggedSlaveMode,
-               $mLastError = 'Unknown error',
-               $mLastIndex,
-               $mLoads,
-               $mReadIndex,
-               $mServers,
-               $mWaitForFile,
-               $mWaitForPos,
-               $mWaitTimeout ;
+       /* private */ var $mServers, $mConnections, $mLoads, $mGroupLoads;
+       /* private */ var $mFailFunction, $mErrorConnection;
+       /* private */ var $mForce, $mReadIndex, $mLastIndex, $mAllowLagged;
+       /* private */ var $mWaitForFile, $mWaitForPos, $mWaitTimeout;
+       /* private */ var $mLaggedSlaveMode, $mLastError = 'Unknown error';
 
        function LoadBalancer()
        {
@@ -62,7 +50,7 @@ class LoadBalancer {
                $this->mAllowLag = false;
        }
 
-       static function newFromParams( $servers, $failFunction = false, $waitTimeout = 10 )
+       function newFromParams( $servers, $failFunction = false, $waitTimeout = 10 )
        {
                $lb = new LoadBalancer;
                $lb->initialise( $servers, $failFunction, $waitTimeout );
index f9181a7..5741e0c 100644 (file)
@@ -33,9 +33,9 @@
  */
 class LogPage {
        /* @access private */
-       private $type, $action, $comment, $params, $target;
+       var $type, $action, $comment, $params, $target;
        /* @acess public */
-       public $updateRecentChanges;
+       var $updateRecentChanges;
 
        /**
          * Constructor
@@ -93,25 +93,23 @@ class LogPage {
        /**
         * @static
         */
-       static function validTypes() {
+       function validTypes() {
                static $types = array( '', 'block', 'protect', 'rights', 'delete', 'upload', 'move' );
                wfRunHooks( 'LogPageValidTypes', array( &$types ) );
                return $types;
        }
 
        /**
-        * @param $type
         * @static
         */
-       static function isLogType( $type ) {
+       function isLogType( $type ) {
                return in_array( $type, LogPage::validTypes() );
        }
 
        /**
-        * @param $type
         * @static
         */
-       static function logName( $type ) {
+       function logName( $type ) {
                static $typeText = array(
                        ''        => 'log',
                        'block'   => 'blocklogpage',
@@ -134,7 +132,7 @@ class LogPage {
        /**
         * @static
         */
-       static function logHeader( $type ) {
+       function logHeader( $type ) {
                static $headerText = array(
                        ''        => 'alllogstext',
                        'block'   => 'blocklogtext',
@@ -150,16 +148,9 @@ class LogPage {
        }
 
        /**
-        * @param $type
-        * @param $action
-        * @param $title (Default: NULL)
-        * @param $skin (Default: NULL)
-        * @param $params (Default: array() )
-        * @param $filterWikilinks (Default: false)
-        * @param $translate (Default: false)
         * @static
         */
-       static function actionText( $type, $action, $title = NULL, $skin = NULL, $params = array(), $filterWikilinks=false, $translate=false ) {
+       function actionText( $type, $action, $title = NULL, $skin = NULL, $params = array(), $filterWikilinks=false, $translate=false ) {
                global $wgLang, $wgContLang;
                static $actions = array(
                        'block/block'       => 'blocklogentry',
@@ -265,10 +256,9 @@ class LogPage {
 
        /**
         * Create a blob from a parameter array
-        * @param $params Need documentation.
         * @static
         */
-       static function makeParamBlob( $params ) {
+       function makeParamBlob( $params ) {
                return implode( "\n", $params );
        }
 
@@ -276,7 +266,7 @@ class LogPage {
         * Extract a parameter array from a blob
         * @static
         */
-       static function extractParams( $blob ) {
+       function extractParams( $blob ) {
                if ( $blob === '' ) {
                        return array();
                } else {
index cd654ce..1dfadfe 100644 (file)
@@ -152,21 +152,11 @@ class MagicWord {
        /**#@+
         * @access private
         */
-       private
-               $mBaseRegex,
-               $mModified,
-               $mRegex,
-               $mRegexStart,
-               $mVariableRegex ;
+       var $mId, $mSynonyms, $mCaseSensitive, $mRegex;
+       var $mRegexStart, $mBaseRegex, $mVariableRegex;
+       var $mModified;
        /**#@-*/
 
-       /** @todo FIXME should they really be public ? */
-       public
-               $mCaseSensitive,
-               $mId,
-               $mSynonyms ;
-
-
        function MagicWord($id = 0, $syn = '', $cs = false) {
                $this->mId = $id;
                $this->mSynonyms = (array)$syn;
@@ -182,7 +172,7 @@ class MagicWord {
         * Factory: creates an object representing an ID
         * @static
         */
-       static function &get( $id ) {
+       function &get( $id ) {
                global $wgMagicWords;
 
                if ( !is_array( $wgMagicWords ) ) {
index df66cdd..ea7ae52 100644 (file)
@@ -22,18 +22,11 @@ define( 'MSG_WAIT_TIMEOUT', 10);
  * @package MediaWiki
  */
 class MessageCache {
-       private
-               $mCache,
-               $mDeferred = true,
-               $mDisable,
-               $mExpiry,
-               $mExtensionMessages = array(),
-               $mInitialised = false,
-               $mKeys,
-               $mMemcKey,
-               $mParser,
-               $mParserOptions,
-               $mUseCache ;
+       var $mCache, $mUseCache, $mDisable, $mExpiry;
+       var $mMemcKey, $mKeys, $mParserOptions, $mParser;
+       var $mExtensionMessages = array();
+       var $mInitialised = false;
+       var $mDeferred = true;
 
        function initialise( &$memCached, $useDB, $expiry, $memcPrefix) {
                $fname = 'MessageCache::initialise';
index 71aa0b8..4b3710d 100644 (file)
@@ -13,30 +13,24 @@ if ( $wgUseTeX )
  * @package MediaWiki
  */
 class OutputPage {
-       private $mHeaders, $mMetatags, $mKeywords;
-       private $mLinktags, $mPagetitle;
-       private $mHTMLtitle, $mRobotpolicy, $mIsarticle, $mPrintable;
-       private $mSubtitle, $mRedirect, $mStatusCode;
-       private $mLastModified, $mETag ;
-       private $mScripts, $mLinkColours;
-       private $mSuppressQuickbar;
-       private $mOnloadHandler;
-       private $mDoNothing;
-       private $mContainsOldMagic, $mContainsNewMagic;
-       private $mIsArticleRelated;
-       private $mShowFeedLinks = false;
-       private $mEnableClientCache = true;
-       private $mArticleBodyOnly = false;
-       private $mNewSectionLink = false;
-
-       /** @todo FIXME shouldn't those be private ? */
-       public
-               $mPageLinkTitle,
-               $mCategoryLinks,
-               $mDebugtext,
-               $mBodytext,
-               $mParserOptions // Called in SkinTemplate when using Special::UserLogin
-       ;
+       var $mHeaders, $mMetatags, $mKeywords;
+       var $mLinktags, $mPagetitle, $mBodytext, $mDebugtext;
+       var $mHTMLtitle, $mRobotpolicy, $mIsarticle, $mPrintable;
+       var $mSubtitle, $mRedirect, $mStatusCode;
+       var $mLastModified, $mETag, $mCategoryLinks;
+       var $mScripts, $mLinkColours, $mPageLinkTitle;
+
+       var $mSuppressQuickbar;
+       var $mOnloadHandler;
+       var $mDoNothing;
+       var $mContainsOldMagic, $mContainsNewMagic;
+       var $mIsArticleRelated;
+       var $mParserOptions;
+       var $mShowFeedLinks = false;
+       var $mEnableClientCache = true;
+       var $mArticleBodyOnly = false;
+       
+       var $mNewSectionLink = false;
 
        /**
         * Constructor
@@ -611,7 +605,7 @@ class OutputPage {
                print $outs;
        }
 
-       static function setEncodings() {
+       function setEncodings() {
                global $wgInputEncoding, $wgOutputEncoding;
                global $wgUser, $wgContLang;
 
index 312056d..4cfee47 100644 (file)
@@ -21,14 +21,11 @@ define('DIR_NEXT', 1);
  */
 
 class PageHistory {
-       private
-               $lastdate,
-               $linesonpage,
-               $mArticle,
-               $mLatestId = null,
-               $mNotificationTimestamp,
-               $mSkin,
-               $mTitle ;
+       var $mArticle, $mTitle, $mSkin;
+       var $lastdate;
+       var $linesonpage;
+       var $mNotificationTimestamp;
+       var $mLatestId = null;
 
        /**
         * Construct a new PageHistory.
index 62e1e96..8976971 100644 (file)
@@ -93,33 +93,20 @@ class Parser
         * @private
         */
        # Persistent:
-       private $mTagHooks, $mFunctionHooks;
+       var $mTagHooks, $mFunctionHooks;
 
        # Cleared with clearState():
-       private
-               $mArgStack,
-               $mAutonumber,
-               $mDTopen,
-               $mIncludeCount,
-               $mInPre,
-               $mInterwikiLinkHolders,
-               $mLastSection,
-               $mLinkHolders,
-               $mOutput,
-               $mStripState = array(),
-               $mUniqPrefix,
-               $mVariables ;
-
-       private
-               $mTemplates,    // cache of already loaded templates, avoids
+       var $mOutput, $mAutonumber, $mDTopen, $mStripState = array();
+       var $mVariables, $mIncludeCount, $mArgStack, $mLastSection, $mInPre;
+       var $mInterwikiLinkHolders, $mLinkHolders, $mUniqPrefix;
+       var $mTemplates,        // cache of already loaded templates, avoids
                                // multiple SQL queries for the same string
            $mTemplatePath;     // stores an unsorted hash of all the templates already loaded
                                // in this path. Used for loop detection.
 
        # Temporary
        # These are variables reset at least once per parse regardless of $clearState
-       private
-               $mOptions,      // ParserOptions object
+       var $mOptions,      // ParserOptions object
                $mTitle,        // Title context, used for self-link rendering and similar things
                $mOutputType,   // Output type, one of the OT_xxx constants
                $mRevisionId;   // ID to display in {{REVISIONID}} tags
@@ -4090,25 +4077,20 @@ class Parser
  */
 class ParserOutput
 {
-       private
-               $mCacheTime,        # Time when this object was generated, or -1 for uncacheable. Used in ParserCache.
-               $mCategories,       # Map of category names to sort keys
-               $mExternalLinks,    # External link URLs, in the key only
-               $mImages,           # DB keys of the images used, in the array key only
+       var $mText,             # The output text
                $mLanguageLinks,    # List of the full text of language links, in the order they appear
+               $mCategories,       # Map of category names to sort keys
+               $mContainsOldMagic, # Boolean variable indicating if the input contained variables like {{CURRENTDAY}}
+               $mCacheTime,        # Time when this object was generated, or -1 for uncacheable. Used in ParserCache.
+               $mVersion,          # Compatibility check
+               $mTitleText,        # title text of the chosen language variant
                $mLinks,            # 2-D map of NS/DBK to ID for the links in the document. ID=zero for broken.
-               $mNewSection,           # Show a new section link?
-               $mSubtitle,                     # Additional subtitle
                $mTemplates,        # 2-D map of NS/DBK to ID for the template references. ID=zero for broken.
-               $mTitleText,        # title text of the chosen language variant
-               $mVersion ;         # Compatibility check
-
-
-       /** @todo FIXME should those be public ? */
-       public
-               $mContainsOldMagic, # Boolean variable indicating if the input contained variables like {{CURRENTDAY}}
+               $mImages,           # DB keys of the images used, in the array key only
+               $mExternalLinks,    # External link URLs, in the key only
                $mHTMLtitle,            # Display HTML title
-               $mText ;            # The output text // public cause its used in ParserCache::save
+               $mSubtitle,                     # Additional subtitle
+               $mNewSection;           # Show a new section link?
 
        function ParserOutput( $text = '', $languageLinks = array(), $categoryLinks = array(),
                $containsOldMagic = false, $titletext = '' )
@@ -4205,19 +4187,18 @@ class ParserOutput
 class ParserOptions
 {
        # All variables are private
-       private
-               $mAllowExternalImages,       # Allow external images inline
-               $mAllowExternalImagesFrom,   # If not, any exception?
-               $mAllowSpecialInclusion,     # Allow inclusion of special pages
-               $mDateFormat,                # Date format index
-               $mEditSection,               # Create "edit section" links
-               $mInterfaceMessage,          # Which lang to call for PLURAL and GRAMMAR
-               $mInterwikiMagic,            # Interlanguage links are removed and returned in an array
-               $mNumberHeadings,            # Automatically number headings
-               $mSkin,                      # Reference to the preferred skin
-               $mTidy,                      # Ask for tidy cleanup
-               $mUseDynamicDates,           # Use DateFormatter to format dates
-               $mUseTeX ;                   # Use texvc to expand <math> tags
+       var $mUseTeX;                    # Use texvc to expand <math> tags
+       var $mUseDynamicDates;           # Use DateFormatter to format dates
+       var $mInterwikiMagic;            # Interlanguage links are removed and returned in an array
+       var $mAllowExternalImages;       # Allow external images inline
+       var $mAllowExternalImagesFrom;   # If not, any exception?
+       var $mSkin;                      # Reference to the preferred skin
+       var $mDateFormat;                # Date format index
+       var $mEditSection;               # Create "edit section" links
+       var $mNumberHeadings;            # Automatically number headings
+       var $mAllowSpecialInclusion;     # Allow inclusion of special pages
+       var $mTidy;                      # Ask for tidy cleanup
+       var $mInterfaceMessage;          # Which lang to call for PLURAL and GRAMMAR
 
        function getUseTeX()                        { return $this->mUseTeX; }
        function getUseDynamicDates()               { return $this->mUseDynamicDates; }
@@ -4254,7 +4235,7 @@ class ParserOptions
         * Get parser options
         * @static
         */
-       static function newFromUser( &$user ) {
+       function newFromUser( &$user ) {
                $popts = new ParserOptions;
                $popts->initialiseFromUser( $user );
                return $popts;
index 32514ad..712c664 100644 (file)
@@ -102,8 +102,7 @@ class ParserCache {
 
                // Save the timestamp so that we don't have to load the revision row on view
                $parserOutput->mTimestamp = $article->getTimestamp();
-
-               # FIXME : attempt to access private parserOutput::mText ?
+               
                $parserOutput->mText .= "\n<!-- Saved in parser cache with key $key and timestamp $now -->\n";
                wfDebug( "Saved in parser cache with key $key and timestamp $now\n" );
 
index 8377d99..7848c53 100644 (file)
@@ -56,21 +56,21 @@ if ( !$wgDisableCounters )
  *
  * @package MediaWiki
  */
-abstract class QueryPage {
+class QueryPage {
        /**
         * Whether or not we want plain listoutput rather than an ordered list
         *
         * @var bool
         */
-       private $listoutput = false;
+       var $listoutput = false;
        
        /**
         * The offset and limit in use, as passed to the query() function
         *
         * @var integer
         */
-       protected $offset = 0;
-       protected $limit = 0;
+       var $offset = 0;
+       var $limit = 0;
 
        /**
         * A mutator for $this->listoutput;
index 26d72ab..659e688 100644 (file)
  * @todo document functions and variables
  * @package MediaWiki
  */
-class RecentChange {
-
-       private
-               $mExtra = array(),
-               $mMovedToTitle = false,
-               $mTitle = false;
-
-       /** @todo FIXME should those really be public ? */
-       public
-               $mAttribs = array();
-
-       public
-               $numberofWatchingusers = 0 ; # Dummy to prevent error message in SpecialRecentchangeslinked
+class RecentChange
+{
+       var $mAttribs = array(), $mExtra = array();
+       var $mTitle = false, $mMovedToTitle = false;
+       var $numberofWatchingusers = 0 ; # Dummy to prevent error message in SpecialRecentchangeslinked
 
        # Factory methods
 
-       static function newFromRow( $row ) {
+       /* static */ function newFromRow( $row )
+       {
                $rc = new RecentChange;
                $rc->loadFromRow( $row );
                return $rc;
        }
 
-       static function newFromCurRow( $row, $rc_this_oldid = 0 ) {
+       /* static */ function newFromCurRow( $row, $rc_this_oldid = 0 )
+       {
                $rc = new RecentChange;
                $rc->loadFromCurRow( $row, $rc_this_oldid );
                $rc->notificationtimestamp = false;
@@ -202,7 +196,7 @@ class RecentChange {
        }
 
        # Makes an entry in the database corresponding to an edit
-       static function notifyEdit( $timestamp, &$title, $minor, &$user, $comment,
+       /*static*/ function notifyEdit( $timestamp, &$title, $minor, &$user, $comment,
                $oldId, $lastTimestamp, $bot = "default", $ip = '', $oldSize = 0, $newSize = 0,
                $newId = 0)
        {
@@ -251,7 +245,7 @@ class RecentChange {
 
        # Makes an entry in the database corresponding to page creation
        # Note: the title object must be loaded with the new id using resetArticleID()
-       static function notifyNew( $timestamp, &$title, $minor, &$user, $comment, $bot = "default",
+       /*static*/ function notifyNew( $timestamp, &$title, $minor, &$user, $comment, $bot = "default",
          $ip='', $size = 0, $newId = 0 )
        {
                if ( !$ip ) {
@@ -297,7 +291,7 @@ class RecentChange {
        }
 
        # Makes an entry in the database corresponding to a rename
-       static function notifyMove( $timestamp, &$oldTitle, &$newTitle, &$user, $comment, $ip='', $overRedir = false )
+       /*static*/ function notifyMove( $timestamp, &$oldTitle, &$newTitle, &$user, $comment, $ip='', $overRedir = false )
        {
                if ( !$ip ) {
                        $ip = wfGetIP();
@@ -336,17 +330,17 @@ class RecentChange {
                $rc->save();
        }
 
-       static function notifyMoveToNew( $timestamp, &$oldTitle, &$newTitle, &$user, $comment, $ip='' ) {
+       /* static */ function notifyMoveToNew( $timestamp, &$oldTitle, &$newTitle, &$user, $comment, $ip='' ) {
                RecentChange::notifyMove( $timestamp, $oldTitle, $newTitle, $user, $comment, $ip, false );
        }
 
-       static function notifyMoveOverRedirect( $timestamp, &$oldTitle, &$newTitle, &$user, $comment, $ip='' ) {
+       /* static */ function notifyMoveOverRedirect( $timestamp, &$oldTitle, &$newTitle, &$user, $comment, $ip='' ) {
                RecentChange::notifyMove( $timestamp, $oldTitle, $newTitle, $user, $comment, $ip, true );
        }
 
        # A log entry is different to an edit in that previous revisions are
        # not kept
-       static function notifyLog( $timestamp, &$title, &$user, $comment, $ip='',
+       /*static*/ function notifyLog( $timestamp, &$title, &$user, $comment, $ip='',
           $type, $action, $target, $logComment, $params )
        {
                if ( !$ip ) {
index c91c741..7845419 100644 (file)
@@ -634,7 +634,7 @@ class Sanitizer {
         * @param string $id
         * @return string
         */
-       static function escapeId( $id ) {
+       function escapeId( $id ) {
                static $replace = array(
                        '%3A' => ':',
                        '%' => '.'
@@ -651,7 +651,7 @@ class Sanitizer {
         * @return string
         * @private
         */
-       private static function armorLinksCallback( $matches ) {
+       function armorLinksCallback( $matches ) {
                return str_replace( ':', '&#58;', $matches[1] );
        }
 
@@ -750,7 +750,6 @@ class Sanitizer {
         * @param string $text
         * @return string
         * @private
-        * @todo FIXME called from parser.php so not that much private
         */
        function normalizeCharReferences( $text ) {
                return preg_replace_callback(
@@ -762,7 +761,7 @@ class Sanitizer {
         * @param string $matches
         * @return string
         */
-       static function normalizeCharReferencesCallback( $matches ) {
+       function normalizeCharReferencesCallback( $matches ) {
                $ret = null;
                if( $matches[1] != '' ) {
                        $ret = Sanitizer::normalizeEntity( $matches[1] );
@@ -788,7 +787,7 @@ class Sanitizer {
         * @param string $name
         * @return string
         */
-       static function normalizeEntity( $name ) {
+       function normalizeEntity( $name ) {
                global $wgHtmlEntities;
                if( isset( $wgHtmlEntities[$name] ) ) {
                        return "&$name;";
@@ -836,9 +835,8 @@ class Sanitizer {
         * @param string $text
         * @return string
         * @public
-        * @static
         */
-       public static function decodeCharReferences( $text ) {
+       function decodeCharReferences( $text ) {
                return preg_replace_callback(
                        MW_CHAR_REFS_REGEX,
                        array( 'Sanitizer', 'decodeCharReferencesCallback' ),
@@ -849,7 +847,7 @@ class Sanitizer {
         * @param string $matches
         * @return string
         */
-       static function decodeCharReferencesCallback( $matches ) {
+       function decodeCharReferencesCallback( $matches ) {
                if( $matches[1] != '' ) {
                        return Sanitizer::decodeEntity( $matches[1] );
                } elseif( $matches[2] != '' ) {
@@ -1086,7 +1084,7 @@ class Sanitizer {
         * @return string
         * @static
         */
-       static function hackDocType() {
+       function hackDocType() {
                global $wgHtmlEntities;
                $out = "<!DOCTYPE html [\n";
                foreach( $wgHtmlEntities as $entity => $codepoint ) {
index 135d9f0..1f222b3 100644 (file)
  */
 class SearchUpdate {
 
-       private
-               $mId = 0,
-               $mNamespace,
-               $mTitle,
-               $mText,
-               $mTitleWords;
+       /* private */ var $mId = 0, $mNamespace, $mTitle, $mText;
+       /* private */ var $mTitleWords;
 
        function SearchUpdate( $id, $title, $text = false ) {
                $nt = Title::newFromText( $title );
index 293de95..8fd5d6b 100644 (file)
@@ -15,10 +15,10 @@ define('SITE_CONFIGURATION', 1);
 
 /** @package MediaWiki */
 class SiteConfiguration {
-       private $suffixes = array();
-       private $wikis = array();
-       private $settings = array();
-       private $localVHosts = array();
+       var $suffixes = array();
+       var $wikis = array();
+       var $settings = array();
+       var $localVHosts = array();
 
        /** */
        function get( $setting, $wiki, $suffix, $params = array() ) {
index 112ab2c..1b6d380 100644 (file)
@@ -11,7 +11,7 @@
  */
 class SiteStatsUpdate {
 
-       private $mViews, $mEdits, $mGood, $mPages, $mUsers;
+       var $mViews, $mEdits, $mGood, $mPages, $mUsers;
 
        function SiteStatsUpdate( $views, $edits, $good, $pages = 0, $users = 0 ) {
                $this->mViews = $views;
index eeffd3e..d1be56e 100644 (file)
@@ -39,12 +39,10 @@ class Skin extends Linker {
        /**#@+
         * @private
         */
-       private
-               $lastdate,
-               $lastline,
-               $rc_cache,     # Cache for Enhanced Recent Changes
-               $rcCacheIndex, # Recent Changes Cache Counter for visibility toggle
-               $rcMoveIndex;
+       var $lastdate, $lastline;
+       var $rc_cache ; # Cache for Enhanced Recent Changes
+       var $rcCacheIndex ; # Recent Changes Cache Counter for visibility toggle
+       var $rcMoveIndex;
        /**#@-*/
 
        /** Constructor, call parent constructor */
index 1142e68..8ea51ba 100644 (file)
@@ -38,12 +38,11 @@ function wfSpecialAllpages( $par=NULL, $specialPage ) {
 }
 
 class SpecialAllpages {
-       private
-               $maxPerPage = 960,
-               $name = 'Allpages',
-               # Determines, which message describes the input field 'nsfrom' (->SpecialPrefixindex.php)
-               $nsfromMsg = 'allpagesfrom',
-               $topLevelMax = 50;
+       var $maxPerPage=960;
+       var $topLevelMax=50;
+       var $name='Allpages';
+       # Determines, which message describes the input field 'nsfrom' (->SpecialPrefixindex.php)
+       var $nsfromMsg='allpagesfrom';
 
 /**
  * HTML for the top form
index 3e7f7f7..b3f67ab 100644 (file)
@@ -37,10 +37,7 @@ function wfSpecialBlockip( $par ) {
  * @subpackage SpecialPage
  */
 class IPBlockForm {
-       private
-               $BlockAddress,
-               $BlockExpiry,
-               $BlockReason;
+       var $BlockAddress, $BlockExpiry, $BlockReason;
 
        function IPBlockForm( $par ) {
                global $wgRequest;
index a4a899f..960f622 100644 (file)
@@ -29,7 +29,7 @@ function wfSpecialBooksources( $par ) {
  * @subpackage SpecialPage
  */
 class BookSourceList {
-       private $mIsbn;
+       var $mIsbn;
 
        function BookSourceList( $isbn ) {
                $this->mIsbn = $isbn;
index 10633c1..5354cca 100644 (file)
@@ -6,12 +6,8 @@
 
 /** @package MediaWiki */
 class contribs_finder {
-       private
-               $dbr,
-               $limit,
-               $namespace,
-               $offset,
-               $username;
+       var $username, $offset, $limit, $namespace;
+       var $dbr;
 
        function contribs_finder($username) {
                $this->username = $username;
@@ -116,7 +112,7 @@ class contribs_finder {
                return $rows[count($rows) - 1]->rev_timestamp;
        }
 
-       private function make_sql() {
+       /* private */ function make_sql() {
                $userCond = $condition = $index = $offsetQuery = '';
 
                extract($this->dbr->tableNames('page', 'revision'));
index d61b351..91abcea 100644 (file)
@@ -121,15 +121,14 @@ function wfSpecialImport( $page = '' ) {
  * @subpackage SpecialPage
  */
 class WikiRevision {
-       private
-               $comment = '',
-               $id = 0,
-               $minor = false,
-               $text = '',
-               $timestamp = '20010115000000',
-               $title = NULL,
-               $user = 0,
-               $user_text = '' ;
+       var $title = NULL;
+       var $id = 0;
+       var $timestamp = "20010115000000";
+       var $user = 0;
+       var $user_text = "";
+       var $text = "";
+       var $comment = "";
+       var $minor = false;
 
        function setTitle( $text ) {
                $this->title = Title::newFromText( $text );
@@ -249,11 +248,10 @@ class WikiRevision {
  * @subpackage SpecialPage
  */
 class WikiImporter {
-       private
-               $lastfield,
-               $mPageCallback = null,
-               $mRevisionCallback = null,
-               $mSource = null ;
+       var $mSource = null;
+       var $mPageCallback = null;
+       var $mRevisionCallback = null;
+       var $lastfield;
 
        function WikiImporter( $source ) {
                $this->setRevisionCallback( array( &$this, "importRevision" ) );
index a3db1a7..028665c 100644 (file)
@@ -41,10 +41,8 @@ require_once('QueryPage.php');
  * @subpackage SpecialPage
  */
 class ListUsersPage extends QueryPage {
-       // Variables used by constructor
-       public
-               $requestedGroup = '',
-               $requestedUser = '' ;
+       var $requestedGroup = '';
+       var $requestedUser = '';
 
        function getName() {
                return 'Listusers';
index 4d71a6c..f673306 100644 (file)
@@ -42,13 +42,8 @@ function wfSpecialLog( $par = '' ) {
  * @subpackage SpecialPage
  */
 class LogReader {
-       private
-               $db,
-               $joinClauses,
-               $title = null,
-               $type = '',
-               $user = '',
-               $whereClauses ;
+       var $db, $joinClauses, $whereClauses;
+       var $type = '', $user = '', $title = null;
 
        /**
         * @param WebRequest $request For internal use use a FauxRequest object to pass arbitrary parameters.
@@ -216,9 +211,8 @@ class LogViewer {
        /**
         * @var LogReader $reader
         */
-       private
-               $numResults = 0,
-               $reader ;
+       var $reader;
+       var $numResults = 0;
 
        /**
         * @param LogReader &$reader where to get our data from
index 359cdf6..3f72352 100644 (file)
@@ -16,8 +16,7 @@ require_once( 'QueryPage.php' );
  * @subpackage SpecialPage
  */
 class NewPagesPage extends QueryPage {
-       private
-               $namespace;
+       var $namespace;
 
        function NewPagesPage( $namespace = NS_MAIN ) {
                $this->namespace = $namespace;
index b88b6dc..bf6c745 100644 (file)
@@ -109,32 +109,33 @@ class SpecialPage
         * The name of the class, used in the URL.
         * Also used for the default <h1> heading, @see getDescription()
         */
-       private $mName;
+       var $mName;
        /**
         * Minimum user level required to access this page, or "" for anyone.
         * Also used to categorise the pages in Special:Specialpages
         */
-       private $mRestriction;
+       var $mRestriction;
        /**
         * Listed in Special:Specialpages?
         */
-       private $mListed;
+       var $mListed;
        /**
         * Function name called by the default execute()
         */
-       private $mFunction;
+       var $mFunction;
        /**
         * File which needs to be included before the function above can be called
         */
-       private $mFile;
+       var $mFile;
        /**
         * Whether or not this special page is being included from an article
         */
-       private $mIncluding;
+       var $mIncluding;
        /**
         * Whether the special page can be included in an article
         */
-       private $mIncludable;
+       var $mIncludable;
+
 
        /**#@-*/
 
@@ -145,7 +146,7 @@ class SpecialPage
         * Use this for a special page extension
         * @static
         */
-       static function addPage( &$obj ) {
+       function addPage( &$obj ) {
                global $wgSpecialPages;
                $wgSpecialPages[$obj->mName] = $obj;
        }
@@ -155,7 +156,7 @@ class SpecialPage
         * Occasionally used to disable expensive or dangerous special pages
         * @static
         */
-       static function removePage( $name ) {
+       function removePage( $name ) {
                global $wgSpecialPages;
                unset( $wgSpecialPages[$name] );
        }
@@ -165,7 +166,7 @@ class SpecialPage
         * @static
         * @param string $name
         */
-       static function getPage( $name ) {
+       function getPage( $name ) {
                global $wgSpecialPages;
                if ( array_key_exists( $name, $wgSpecialPages ) ) {
                        return $wgSpecialPages[$name];
@@ -179,7 +180,7 @@ class SpecialPage
         * @param string $name
         * @return mixed Title object if the redirect exists, otherwise NULL
         */
-       static function getRedirect( $name ) {
+       function getRedirect( $name ) {
                global $wgUser;
 
                $redirects = array(
@@ -227,7 +228,7 @@ class SpecialPage
         * Returns a 2d array where the first index is the restriction name
         * @static
         */
-       static function getPages() {
+       function getPages() {
                global $wgSpecialPages;
                $pages = array(
                  '' => array(),
@@ -324,7 +325,7 @@ class SpecialPage
         * a redirect.
         * @static
         */
-       static function capturePath( &$title ) {
+       function capturePath( &$title ) {
                global $wgOut, $wgTitle;
 
                $oldTitle = $wgTitle;
index 627f04b..8b972b0 100644 (file)
@@ -22,37 +22,12 @@ function wfSpecialPreferences() {
  * @subpackage SpecialPage
  */
 class PreferencesForm {
-
-       private
-               $mAction,
-               $mCols,
-               $mDate,
-               $mEmailFlag,
-               $mHourDiff,
-               $mImageSize,
-               $mMath,
-               $mNewpass,
-               $mNick,
-               $mOldpass,
-               $mPosted,
-               $mQuickbar,
-               $mRealName,
-               $mRecent,
-               $mReset,
-               $mRetypePass,
-               $mRows,
-               $mSearch,
-               $mSearchChars,
-               $mSearchLines,
-               $mSearchNs,
-               $mSkin,
-               $mStubs,
-               $mToggles,
-               $mUnderline,
-               $mUserEmail,
-               $mUserLanguage,
-               $mUserVariant,
-               $mWatchlistEdits ;
+       var $mQuickbar, $mOldpass, $mNewpass, $mRetypePass, $mStubs;
+       var $mRows, $mCols, $mSkin, $mMath, $mDate, $mUserEmail, $mEmailFlag, $mNick;
+       var $mUserLanguage, $mUserVariant;
+       var $mSearch, $mRecent, $mHourDiff, $mSearchLines, $mSearchChars, $mAction;
+       var $mReset, $mPosted, $mToggles, $mSearchNs, $mRealName, $mImageSize;
+       var $mUnderline, $mWatchlistEdits;
 
        /**
         * Constructor
index feaaef4..541d7aa 100644 (file)
@@ -27,8 +27,7 @@ function wfSpecialUndelete( $par ) {
  * @subpackage SpecialPage
  */
 class PageArchive {
-       public
-               $title;
+       var $title;
 
        function PageArchive( &$title ) {
                if( is_null( $title ) ) {
@@ -309,15 +308,8 @@ class PageArchive {
  * @subpackage SpecialPage
  */
 class UndeleteForm {
-       private
-               $mAction,
-               $mAllowed,
-               $mComment,
-               $mRestore,
-               $mTarget,
-               $mTargetObj,
-               $mTargetTimestamp,
-               $mTimestamp ;
+       var $mAction, $mTarget, $mTimestamp, $mRestore, $mTargetObj;
+       var $mTargetTimestamp, $mAllowed, $mComment;
 
        function UndeleteForm( &$request, $par = "" ) {
                global $wgUser;
index 5e37f6f..65ff6e2 100644 (file)
@@ -25,21 +25,9 @@ function wfSpecialUserlogin() {
  * @subpackage SpecialPage
  */
 class LoginForm {
-       private
-               $mAction,
-               $mCookieCheck,
-               $mCreateaccount,
-               $mCreateaccountMail,
-               $mDomain,
-               $mEmail,                        // Entered user email ??
-               $mLoginattempt,
-               $mMailmypassword,
-               $mName,                         // Entered username
-               $mPassword,                     // Entered user password
-               $mPosted,
-               $mRemember,                     // bool, if user want to be remembered
-               $mReturnTo,                     // article to return to once logged in
-               $mRetype ;
+       var $mName, $mPassword, $mRetype, $mReturnTo, $mCookieCheck, $mPosted;
+       var $mAction, $mCreateaccount, $mCreateaccountMail, $mMailmypassword;
+       var $mLoginattempt, $mRemember, $mEmail, $mDomain;
 
        /**
         * Constructor
index 55118dc..92958e8 100644 (file)
@@ -16,7 +16,7 @@ require_once 'QueryPage.php';
  * @subpackage SpecialPage
  */
 class WantedPagesPage extends QueryPage {
-       private $nlinks;
+       var $nlinks;
 
        function WantedPagesPage( $inc = false, $nlinks = true ) {
                $this->setListoutput( $inc );
index 680b0f4..dba47c5 100644 (file)
@@ -9,8 +9,7 @@
  * @package MediaWiki
  */
 class SquidUpdate {
-       public $urlArr;  // FIXME : is it really public ?
-       private $mMaxTitles;
+       var $urlArr, $mMaxTitles;
 
        function SquidUpdate( $urlArr = Array(), $maxTitles = false ) {
                global $wgMaxSquidPurgeTitles;
index 4c76682..3197b46 100644 (file)
@@ -37,24 +37,23 @@ class Title {
         * @access private
         */
 
-       private $mTextform;           # Text form (spaces not underscores) of the main part
-       private $mUrlform;            # URL-encoded form of the main part
-       private $mDbkeyform;          # Main part with underscores
-       private $mNamespace;          # Namespace index, i.e. one of the NS_xxxx constants
-       private $mInterwiki;          # Interwiki prefix (or null string)
-       private $mFragment;           # Title fragment (i.e. the bit after the #)
-       private $mLatestID;         # ID of most recent revision
-       private $mRestrictions;       # Array of groups allowed to edit this article
+       var $mTextform;           # Text form (spaces not underscores) of the main part
+       var $mUrlform;            # URL-encoded form of the main part
+       var $mDbkeyform;          # Main part with underscores
+       var $mNamespace;          # Namespace index, i.e. one of the NS_xxxx constants
+       var $mInterwiki;          # Interwiki prefix (or null string)
+       var $mFragment;           # Title fragment (i.e. the bit after the #)
+       var $mArticleID;          # Article ID, fetched from the link cache on demand
+       var $mLatestID;         # ID of most recent revision
+       var $mRestrictions;       # Array of groups allowed to edit this article
                                # Only null or "sysop" are supported
-       private $mPrefixedText;       # Text form including namespace/interwiki, initialised on demand
-       private $mDefaultNamespace;   # Namespace index when there is no namespace
+       var $mRestrictionsLoaded; # Boolean for initialisation on demand
+       var $mPrefixedText;       # Text form including namespace/interwiki, initialised on demand
+       var $mDefaultNamespace;   # Namespace index when there is no namespace
                            # Zero except in {{transclusion}} tags
-       private $mWatched;      # Is $wgUser watching this page? NULL if unfilled, accessed through userIsWatching()
+       var $mWatched;      # Is $wgUser watching this page? NULL if unfilled, accessed through userIsWatching()
        /**#@-*/
 
-       /** @todo FIXME should those be public ? */
-       public  $mArticleID;          # Article ID, fetched from the link cache on demand
-       public  $mRestrictionsLoaded; # Boolean for initialisation on demand
 
        /**
         * Constructor
@@ -105,7 +104,7 @@ class Title {
         * @static
         * @access public
         */
-       public static function newFromText( $text, $defaultNamespace = NS_MAIN ) {
+       function newFromText( $text, $defaultNamespace = NS_MAIN ) {
                global $wgTitleCache;
                $fname = 'Title::newFromText';
 
@@ -160,7 +159,7 @@ class Title {
         * @static
         * @access public
         */
-       public static function newFromURL( $url ) {
+       function newFromURL( $url ) {
                global $wgLegalTitleChars;
                $t = new Title();
 
@@ -216,7 +215,7 @@ class Title {
         * @static
         * @access public
         */
-       public static function &makeTitle( $ns, $title ) {
+       function &makeTitle( $ns, $title ) {
                $t =& new Title();
                $t->mInterwiki = '';
                $t->mFragment = '';
@@ -239,7 +238,7 @@ class Title {
         * @static
         * @access public
         */
-       public static function makeTitleSafe( $ns, $title ) {
+       function makeTitleSafe( $ns, $title ) {
                $t = new Title();
                $t->mDbkeyform = Title::makeName( $ns, $title );
                if( $t->secureAndSplit() ) {
@@ -360,7 +359,7 @@ class Title {
         * @param string $title the DB key form the title
         * @return string the prefixed form of the title
         */
-       public static function makeName( $ns, $title ) {
+       /* static */ function makeName( $ns, $title ) {
                global $wgContLang;
 
                $n = $wgContLang->getNsText( $ns );
@@ -1243,23 +1242,6 @@ class Title {
                return (int)$n;
        }
 
-       /**
-        * Used to grab from the logging facility the date at wich the
-        * article got deleted. For special pages and article that never
-        * got deleted, return 0.
-        * @return date when the title got deleted
-        */
-       function getDeletedDate() {
-               $fname = 'Title::getLastDelete';
-               if( $this->getNamespace() < 0 or !$this->isDeleted() ) {
-                       $n = 0;
-               } else {
-                       $dbr =& wfGetDB( DB_SLAVE );
-                       $n = $dbr->selectField( 'logging', 'MAX(log_timestamp)', array( 'log_namespace' => $this->getNamespace(),
-                               'log_title' => $this->getDBkey() ), $fname );
-               }
-               return $n;
-       }
        /**
         * Get the article ID for this Title from the link cache,
         * adding it if necessary
index c6e2252..cb6a6de 100644 (file)
@@ -27,26 +27,26 @@ class User {
        /**@{{
         * @private
         */
-       private $mBlockedby;    //!<
-       private $mBlockreason;  //!<
-       private $mDataLoaded;   //!<
-       private $mEmail;                //!<
-       private $mEmailAuthenticated; //!<
-       private $mGroups;               //!<
-       private $mHash;                 //!<
-       private $mId;                   //!<
-       private $mName;                 //!<
-       private $mNewpassword;  //!<
-       private $mNewtalk;              //!<
-       private $mOptions;              //!<
-       private $mPassword;             //!<
-       private $mRealName;             //!<
-       private $mRegistration; //!<
-       private $mRights;               //!<
-       private $mSkin;                 //!<
-       private $mToken;                //!<
-       private $mTouched;              //!<
-       private $mVersion;              //!< serialized version
+       var $mBlockedby;        //!<
+       var $mBlockreason;      //!<
+       var $mDataLoaded;       //!<
+       var $mEmail;            //!<
+       var $mEmailAuthenticated; //!<
+       var $mGroups;           //!<
+       var $mHash;                     //!<
+       var $mId;                       //!<
+       var $mName;                     //!<
+       var $mNewpassword;      //!<
+       var $mNewtalk;          //!<
+       var $mOptions;          //!<
+       var $mPassword;         //!<
+       var $mRealName;         //!<
+       var $mRegistration;     //!<
+       var $mRights;           //!<
+       var $mSkin;                     //!<
+       var $mToken;            //!<
+       var $mTouched;          //!<
+       var $mVersion;          //!< serialized version
        /**@}} */
 
        /** Constructor using User:loadDefaults() */
@@ -61,7 +61,7 @@ class User {
         * @return User
         * @static
         */
-       static function newFromName( $name ) {
+       function newFromName( $name ) {
                # Force usernames to capital
                global $wgContLang;
                $name = $wgContLang->ucfirst( $name );
@@ -168,7 +168,7 @@ class User {
         * @return integer|null Database user id (null: if non existent
         * @static
         */
-       public static function idFromName( $name ) {
+       function idFromName( $name ) {
                $fname = "User::idFromName";
 
                $nt = Title::newFromText( $name );
@@ -641,9 +641,8 @@ class User {
 
        /**
         * Initialise php session
-        * @static
         */
-       static function SetupSession() {
+       function SetupSession() {
                global $wgSessionsInMemcached, $wgCookiePath, $wgCookieDomain;
                if( $wgSessionsInMemcached ) {
                        require_once( 'MemcachedSessions.php' );
@@ -661,7 +660,7 @@ class User {
         * Create a new user object using data from session
         * @static
         */
-       static function loadFromSession() {
+       function loadFromSession() {
                global $wgMemc, $wgDBname, $wgCookiePrefix;
 
                if ( isset( $_SESSION['wsUserID'] ) ) {
index 2697a19..3885bb9 100644 (file)
@@ -9,9 +9,7 @@
  * @package MediaWiki
  */
 class WatchedItem {
-       private
-               $mTitle,
-               $mUser;
+       var $mTitle, $mUser;
 
        /**
         * Create a WatchedItem object with the given user and title
index f4b0294..736e215 100644 (file)
@@ -445,9 +445,8 @@ class WebRequest {
  * @package MediaWiki
  */
 class FauxRequest extends WebRequest {
-       private
-               $data = null,
-               $wasPosted = false ;
+       var $data = null;
+       var $wasPosted = false;
 
        function FauxRequest( $data, $wasPosted = false ) {
                if( is_array( $data ) ) {
index 0938bd7..be3a8b4 100644 (file)
@@ -5,8 +5,8 @@
 
 class MediaWiki {
 
-       public $GET; /* Stores the $_GET variables at time of creation, can be changed */
-       public $params = array();
+       var $GET; /* Stores the $_GET variables at time of creation, can be changed */
+       var $params = array();
        
        /**
         * Constructor
index 9f70456..17b5d12 100644 (file)
@@ -290,7 +290,7 @@ if (!$wgCachedMessageArrays) {
 
 /* a fake language converter */
 class fakeConverter {
-       private $mLang;
+       var $mLang;
        function fakeConverter($langobj) {$this->mLang = $langobj;}
        function convert($t, $i) {return $t;}
        function parserConvert($t, $p) {return $t;}
@@ -309,7 +309,7 @@ class fakeConverter {
 #--------------------------------------------------------------------------
 
 class Language {
-       private $mConverter;
+       var $mConverter;
        function Language() {
 
                # Copies any missing values in the specified arrays from En to the current language
index 1f25938..fa4908b 100644 (file)
@@ -503,8 +503,6 @@ If you are here by mistake, just click your browser's '''back''' button.",
 'anontalkpagetext' => "----''This is the discussion page for an anonymous user who has not created an account yet or who does not use it. We therefore have to use the numerical [[IP address]] to identify him/her. Such an IP address can be shared by several users. If you are an anonymous user and feel that irrelevant comments have been directed at you, please [[Special:Userlogin|create an account or log in]] to avoid future confusion with other anonymous users.''",
 'noarticletext' => 'There is currently no text in this page, you can [[{{ns:special}}:Search/{{PAGENAME}}|search for this page title]] in other pages or [{{fullurl:{{NAMESPACE}}:{{PAGENAME}}|action=edit}} edit this page].',
 'noarticletextanon' => '{{int:noarticletext}}',
-'noarticletextdeleted' => 'There is currently no text in this page, you can [[{{ns:special}}:Search/{{PAGENAME}}|search for this page title]] in other pages or [{{fullurl:{{NAMESPACE}}:{{PAGENAME}}|action=edit}} edit this page].',
-'noarticletextdeletedanon' => '{{int:noarticletext}}',
 'clearyourcache' => "'''Note:''' After saving, you may have to bypass your browser's cache to see the changes. '''Mozilla / Firefox / Safari:''' hold down ''Shift'' while clicking ''Reload'', or press ''Ctrl-Shift-R'' (''Cmd-Shift-R'' on Apple Mac); '''IE:''' hold ''Ctrl'' while clicking ''Refresh'', or press ''Ctrl-F5''; '''Konqueror:''': simply click the ''Reload'' button, or press ''F5''; '''Opera''' users may need to completely clear their cache in ''Tools&rarr;Preferences''.",
 'usercssjsyoucanpreview' => '<strong>Tip:</strong> Use the \'Show preview\' button to test your new CSS/JS before saving.',
 'usercsspreview' => '\'\'\'Remember that you are only previewing your user CSS, it has not yet been saved!\'\'\'',