Merge changes from 1.4:
[lhc/web/wiklou.git] / includes / Article.php
index cd3abe6..a216fc9 100644 (file)
@@ -1,32 +1,63 @@
 <?php
-# $Id$
-#
-# Class representing a Wikipedia article and history.
-# See design.doc for an overview.
+/**
+ * File for articles
+ * @package MediaWiki
+ */
 
-# Note: edit user interface and cache support functions have been
-# moved to separate EditPage and CacheManager classes.
-
-require_once ( 'CacheManager.php' );
+/**
+ * Need the CacheManager to be loaded
+ */
+require_once( 'CacheManager.php' );
+require_once( 'Revision.php' );
 
 $wgArticleCurContentFields = false;
 $wgArticleOldContentFields = false;
 
+/**
+ * Class representing a Wikipedia article and history.
+ *
+ * See design.doc for an overview.
+ * Note: edit user interface and cache support functions have been
+ * moved to separate EditPage and CacheManager classes.
+ *
+ * @package MediaWiki
+ */
 class Article {
-       /* private */ var $mContent, $mContentLoaded;
-       /* private */ var $mUser, $mTimestamp, $mUserText;
-       /* private */ var $mCounter, $mComment, $mCountAdjustment;
-       /* private */ var $mMinorEdit, $mRedirectedFrom;
-       /* private */ var $mTouched, $mFileCache, $mTitle;
-       /* private */ var $mId, $mTable;
-       /* private */ var $mForUpdate;
-
+       /**#@+
+        * @access private
+        */
+       var $mContent, $mContentLoaded;
+       var $mUser, $mTimestamp, $mUserText;
+       var $mCounter, $mComment, $mCountAdjustment;
+       var $mMinorEdit, $mRedirectedFrom;
+       var $mTouched, $mFileCache, $mTitle;
+       var $mId, $mTable;
+       var $mForUpdate;
+       var $mOldId;
+       /**#@-*/
+
+       /**
+        * Constructor and clear the article
+        * @param mixed &$title
+        */
        function Article( &$title ) {
                $this->mTitle =& $title;
                $this->clear();
        }
 
-       /* private */ function clear() {
+       /**
+        * get the title object of the article
+        * @public
+        */
+       function getTitle() {
+               return $this->mTitle;   
+       }
+
+       /**
+         * Clear the object
+         * @private
+         */
+       function clear() {
                $this->mContentLoaded = false;
                $this->mCurID = $this->mUser = $this->mCounter = -1; # Not loaded
                $this->mRedirectedFrom = $this->mUserText =
@@ -36,169 +67,13 @@ class Article {
                $this->mForUpdate = false;
        }
 
-       # Get revision text associated with an old or archive row
-       # $row is usually an object from wfFetchRow(), both the flags and the text field must be included
-       /* static */ function getRevisionText( $row, $prefix = 'old_' ) {
-               # Get data
-               $textField = $prefix . 'text';
-               $flagsField = $prefix . 'flags';
-
-               if ( isset( $row->$flagsField ) ) {
-                       $flags = explode( ',', $row->$flagsField );
-               } else {
-                       $flags = array();
-               }
-
-               if ( isset( $row->$textField ) ) {
-                       $text = $row->$textField;
-               } else {
-                       return false;
-               }
-
-               if ( in_array( 'link', $flags ) ) {
-                       # Handle link type
-                       $text = Article::followLink( $text );
-               } elseif ( in_array( 'gzip', $flags ) ) {
-                       # Deal with optional compression of archived pages.
-                       # This can be done periodically via maintenance/compressOld.php, and
-                       # as pages are saved if $wgCompressRevisions is set.
-                       return gzinflate( $text );
-               }
-               return $text;
-       }
-
-       /* static */ function compressRevisionText( &$text ) {
-               global $wgCompressRevisions;
-               if( !$wgCompressRevisions ) {
-                       return '';
-               }
-               if( !function_exists( 'gzdeflate' ) ) {
-                       wfDebug( "Article::compressRevisionText() -- no zlib support, not compressing\n" );
-                       return '';
-               }
-               $text = gzdeflate( $text );
-               return 'gzip';
-       }
-
-       # Returns the text associated with a "link" type old table row
-       /* static */ function followLink( $link ) {
-               # Split the link into fields and values
-               $lines = explode( '\n', $link );
-               $hash = '';
-               $locations = array();
-               foreach ( $lines as $line ) {
-                       # Comments
-                       if ( $line{0} == '#' ) {
-                               continue;
-                       }
-                       # Field/value pairs
-                       if ( preg_match( '/^(.*?)\s*:\s*(.*)$/', $line, $matches ) ) {
-                               $field = strtolower($matches[1]);
-                               $value = $matches[2];
-                               if ( $field == 'hash' ) {
-                                       $hash = $value;
-                               } elseif ( $field == 'location' ) {
-                                       $locations[] = $value;
-                               }
-                       }
-               }
-
-               if ( $hash === '' ) {
-                       return false;
-               }
-
-               # Look in each specified location for the text
-               $text = false;
-               foreach ( $locations as $location ) {
-                       $text = Article::fetchFromLocation( $location, $hash );
-                       if ( $text !== false ) {
-                               break;
-                       }
-               }
-
-               return $text;
-       }
-
-       /* static */ function fetchFromLocation( $location, $hash ) {
-               global $wgLoadBalancer;
-               $fname = 'fetchFromLocation';
-               wfProfileIn( $fname );
-
-               $p = strpos( $location, ':' );
-               if ( $p === false ) {
-                       wfProfileOut( $fname );
-                       return false;
-               }
-
-               $type = substr( $location, 0, $p );
-               $text = false;
-               switch ( $type ) {
-                       case 'mysql':
-                               # MySQL locations are specified by mysql://<machineID>/<dbname>/<tblname>/<index>
-                               # Machine ID 0 is the current connection
-                               if ( preg_match( '/^mysql:\/\/(\d+)\/([A-Za-z_]+)\/([A-Za-z_]+)\/([A-Za-z_]+)$/',
-                                 $location, $matches ) ) {
-                                       $machineID = $matches[1];
-                                       $dbName = $matches[2];
-                                       $tblName = $matches[3];
-                                       $index = $matches[4];
-                                       if ( $machineID == 0 ) {
-                                               # Current connection
-                                               $db =& $this->getDB();
-                                       } else {
-                                               # Alternate connection
-                                               $db =& $wgLoadBalancer->getConnection( $machineID );
-
-                                               if ( array_key_exists( $machineId, $wgKnownMysqlServers ) ) {
-                                                       # Try to open, return false on failure
-                                                       $params = $wgKnownDBServers[$machineId];
-                                                       $db = Database::newFromParams( $params['server'], $params['user'], $params['password'],
-                                                               $dbName, 1, DBO_IGNORE );
-                                               }
-                                       }
-                                       if ( $db->isOpen() ) {
-                                               $index = $db->strencode( $index );
-                                               $res = $db->query( "SELECT blob_data FROM $dbName.$tblName " .
-                                                       "WHERE blob_index='$index' " . $this->getSelectOptions(), $fname );
-                                               $row = $db->fetchObject( $res );
-                                               $text = $row->text_data;
-                                       }
-                               }
-                               break;
-                       case 'file':
-                               # File locations are of the form file://<filename>, relative to the current directory
-                               if ( preg_match( '/^file:\/\/(.*)$', $location, $matches ) )
-                               $filename = strstr( $location, 'file://' );
-                               $text = @file_get_contents( $matches[1] );
-               }
-               if ( $text !== false ) {
-                       # Got text, now we need to interpret it
-                       # The first line contains information about how to do this
-                       $p = strpos( $text, '\n' );
-                       $type = substr( $text, 0, $p );
-                       $text = substr( $text, $p + 1 );
-                       switch ( $type ) {
-                               case 'plain':
-                                       break;
-                               case 'gzip':
-                                       $text = gzinflate( $text );
-                                       break;
-                               case 'object':
-                                       $object = unserialize( $text );
-                                       $text = $object->getItem( $hash );
-                                       break;
-                               default:
-                                       $text = false;
-                       }
-               }
-               wfProfileOut( $fname );
-               return $text;
-       }
-
-       # Note that getContent/loadContent may follow redirects if
-       # not told otherwise, and so may cause a change to mTitle.
-
-       # Return the text of this revision
+       /**
+        * Note that getContent/loadContent may follow redirects if
+        * not told otherwise, and so may cause a change to mTitle.
+        *
+        * @param $noredir
+        * @return Return the text of this revision
+       */
        function getContent( $noredir ) {
                global $wgRequest;
 
@@ -246,12 +121,17 @@ class Article {
                }
        }
 
-       # This function returns the text of a section, specified by a number ($section).
-       # A section is text under a heading like == Heading == or <h1>Heading</h1>, or
-       # the first section before any such heading (section 0).
-       #
-       # If a section contains subsections, these are also returned.
-       #
+       /**
+        * This function returns the text of a section, specified by a number ($section).
+        * A section is text under a heading like == Heading == or <h1>Heading</h1>, or
+        * the first section before any such heading (section 0).
+        *
+        * If a section contains subsections, these are also returned.
+        *
+        * @param string $text text to look in
+        * @param integer $section section number
+        * @return string text of the requested section
+        */
        function getSection($text,$section) {
 
                # strip NOWIKI etc. to avoid confusion (true-parameter causes HTML
@@ -310,224 +190,234 @@ class Article {
 
        }
 
-       # Return an array of the columns of the "cur"-table
-       function &getCurContentFields() {
-               global $wgArticleCurContentFields;
-               if ( !$wgArticleCurContentFields ) {
-                       $wgArticleCurContentFields = array( 'cur_text','cur_timestamp','cur_user', 'cur_user_text',
-                         'cur_comment','cur_counter','cur_restrictions','cur_touched' );
-               }
-               return $wgArticleCurContentFields;
+       /**
+        * Return an array of the columns of the "cur"-table
+        */
+       function getContentFields() {
+               return $wgArticleContentFields = array(
+                 'old_text','old_flags',
+                 'rev_timestamp','rev_user', 'rev_user_text', 'rev_comment','page_counter',
+                 'page_namespace', 'page_title', 'page_restrictions','page_touched','page_is_redirect' );
        }
 
-       # Return an array of the columns of the "old"-table
-       function &getOldContentFields() {
-               global $wgArticleOldContentFields;
-               if ( !$wgArticleOldContentFields ) {
-                       $wgArticleOldContentFields = array( 'old_namespace','old_title','old_text','old_timestamp',
-                         'old_user','old_user_text','old_comment','old_flags' );
+       /**
+        * Return the oldid of the article that is to be shown.
+        * For requests with a "direction", this is not the oldid of the
+        * query
+        */
+       function getOldID() {
+               global $wgRequest, $wgOut;
+               static $lastid;
+               
+               if ( isset( $lastid ) ) {
+                       return $lastid;
                }
-               return $wgArticleOldContentFields;
+               # Query variables :P
+               $oldid = $wgRequest->getVal( 'oldid' );
+               if ( isset( $oldid ) ) {
+                       $oldid = IntVal( $oldid );
+                       if ( $wgRequest->getVal( 'direction' ) == 'next' ) {
+                               $nextid = $this->mTitle->getNextRevisionID( $oldid );
+                               if ( $nextid  ) {
+                                       $oldid = $nextid;
+                               } else {
+                                       $wgOut->redirect( $this->mTitle->getFullURL( 'redirect=no' ) );
+                               }
+                       } elseif ( $wgRequest->getVal( 'direction' ) == 'prev' ) {
+                               $previd = $this->mTitle->getPreviousRevisionID( $oldid );
+                               if ( $previd ) {
+                                       $oldid = $previd;
+                               } else {
+                                       # TODO
+                               }
+                       }
+                       $lastid = $oldid;
+               }
+               return @$oldid; # "@" to be able to return "unset" without PHP complaining
        }
 
-       # Load the revision (including cur_text) into this object
+
+       /**
+        * Load the revision (including cur_text) into this object
+        */
        function loadContent( $noredir = false ) {
-               global $wgOut, $wgMwRedir, $wgRequest;
+               global $wgOut, $wgRequest;
 
-               $dbr =& $this->getDB();
+               if ( $this->mContentLoaded ) return;
+               
                # Query variables :P
-               $oldid = $wgRequest->getVal( 'oldid' );
+               $oldid = $this->getOldID();
                $redirect = $wgRequest->getVal( 'redirect' );
 
-               if ( $this->mContentLoaded ) return;
                $fname = 'Article::loadContent';
 
                # Pre-fill content with error message so that if something
                # fails we'll have something telling us what we intended.
 
                $t = $this->mTitle->getPrefixedText();
-               if ( isset( $oldid ) ) {
-                       $oldid = IntVal( $oldid );
+
+               $noredir = $noredir || ($wgRequest->getVal( 'redirect' ) == 'no');
+               $this->mOldId = $oldid;
+               $this->fetchContent( $oldid, $noredir, true );
+       }
+       
+       /**
+        * Get text of an article from database
+        * @param int $oldid 0 for whatever the latest revision is
+        * @param bool $noredir Set to true to avoid following redirects
+        * @param bool $globalTitle Set to true to change the global $wgTitle object when following redirects or other unexpected title changes
+        * @return string
+        */
+       function fetchContent( $oldid = 0, $noredir = false, $globalTitle = false ) {
+               if ( $this->mContentLoaded ) {
+                       return $this->mContent;
+               }
+               $dbr =& $this->getDB();
+               $fname = 'Article::fetchContent';
+
+               # Pre-fill content with error message so that if something
+               # fails we'll have something telling us what we intended.
+               $t = $this->mTitle->getPrefixedText();
+               if( $oldid ) {
                        $t .= ',oldid='.$oldid;
                }
-               if ( isset( $redirect ) ) {
+               if( isset( $redirect ) ) {
                        $redirect = ($redirect == 'no') ? 'no' : 'yes';
                        $t .= ',redirect='.$redirect;
                }
                $this->mContent = wfMsg( 'missingarticle', $t );
 
-               if ( ! $oldid ) {       # Retrieve current version
+               if( !$oldid ) {
+                       # Retrieve current version
                        $id = $this->getID();
-                       if ( 0 == $id ) return;
-
-                       $s = $dbr->selectRow( 'cur', $this->getCurContentFields(), array( 'cur_id' => $id ), $fname, 
-                               $this->getSelectOptions() );
-                       if ( $s === false ) {
-                               return;
+                       if ( 0 == $id ) {
+                               return false;
                        }
 
-                       # If we got a redirect, follow it (unless we've been told
-                       # not to by either the function parameter or the query
-                       if ( ( 'no' != $redirect ) && ( false == $noredir ) ) {
-                               $rt = Title::newFromRedirect( $s->cur_text );
-                               # process if title object is valid and not special:userlogout
-                               if ( $rt && ! ( $rt->getNamespace() == NS_SPECIAL && $rt->getText() == 'Userlogout' ) ) {
-                                       # Gotta hand redirects to special pages differently:
-                                       # Fill the HTTP response "Location" header and ignore
-                                       # the rest of the page we're on.
-
-                                       if ( $rt->getInterwiki() != '' ) {
+                       $s = $dbr->selectRow( array( 'text', 'revision', 'page' ),
+                               $this->getContentFields(),
+                               "page_id='$id' AND rev_page=page_id AND rev_id=page_latest AND old_id=rev_id",
+                               $fname, $this->getSelectOptions() );
+               } else {
+                       # Historical revision
+                       $s = $dbr->selectRow( array( 'text', 'revision', 'page' ),
+                               $this->getContentFields(),
+                               "rev_page=page_id AND rev_id='$oldid' AND old_id=rev_id",
+                               $fname, $this->getSelectOptions() );
+               }
+               if ( $s === false ) {
+                       return false;
+               }
+
+               # If we got a redirect, follow it (unless we've been told
+               # not to by either the function parameter or the query
+               if ( !$oldid && !$noredir ) {
+                       $rt = Title::newFromRedirect( Revision::getRevisionText( $s ) );
+                       # process if title object is valid and not special:userlogout
+                       if ( $rt && ! ( $rt->getNamespace() == NS_SPECIAL && $rt->getText() == 'Userlogout' ) ) {
+                               # Gotta hand redirects to special pages differently:
+                               # Fill the HTTP response "Location" header and ignore
+                               # the rest of the page we're on.
+                               if( $globalTitle ) {
+                                       global $wgOut;
+                                       if ( $rt->getInterwiki() != '' && $rt->isLocal() ) {
                                                $wgOut->redirect( $rt->getFullURL() ) ;
-                                               return;
+                                               return false;
                                        }
                                        if ( $rt->getNamespace() == NS_SPECIAL ) {
                                                $wgOut->redirect( $rt->getFullURL() );
-                                               return;
+                                               return false;
                                        }
-                                       $rid = $rt->getArticleID();
-                                       if ( 0 != $rid ) {
-                                               $redirRow = $dbr->selectRow( 'cur', $this->getCurContentFields(), 
-                                                       array( 'cur_id' => $rid ), $fname, $this->getSelectOptions() );
-
-                                               if ( $redirRow !== false ) {
-                                                       $this->mRedirectedFrom = $this->mTitle->getPrefixedText();
-                                                       $this->mTitle = $rt;
-                                                       $s = $redirRow;
-                                               }
+                               }
+                               $rid = $rt->getArticleID();
+                               if ( 0 != $rid ) {
+                                       $redirRow = $dbr->selectRow( array( 'text', 'revision', 'page' ),
+                                               $this->getContentFields(),
+                                               "page_id='$rid' AND rev_page=page_id AND rev_id=page_latest AND old_id=rev_id",
+                                               $fname, $this->getSelectOptions() );
+
+                                       if ( $redirRow !== false ) {
+                                               $this->mRedirectedFrom = $this->mTitle->getPrefixedText();
+                                               $this->mTitle = $rt;
+                                               $s = $redirRow;
                                        }
                                }
                        }
+               }
 
-                       $this->mContent = $s->cur_text;
-                       $this->mUser = $s->cur_user;
-                       $this->mUserText = $s->cur_user_text;
-                       $this->mComment = $s->cur_comment;
-                       $this->mCounter = $s->cur_counter;
-                       $this->mTimestamp = wfTimestamp(TS_MW,$s->cur_timestamp);
-                       $this->mTouched = wfTimestamp(TS_MW,$s->cur_touched);
-                       $this->mTitle->mRestrictions = explode( ',', trim( $s->cur_restrictions ) );
-                       $this->mTitle->mRestrictionsLoaded = true;
-               } else { # oldid set, retrieve historical version
-                       $s = $dbr->getArray( 'old', $this->getOldContentFields(), array( 'old_id' => $oldid ), 
-                               $fname, $this->getSelectOptions() );
-                       if ( $s === false ) {
-                               return;
-                       }
-
-                       if( $this->mTitle->getNamespace() != $s->old_namespace ||
-                               $this->mTitle->getDBkey() != $s->old_title ) {
-                               $oldTitle = Title::makeTitle( $s->old_namesapce, $s->old_title );
-                               $this->mTitle = $oldTitle;
-                               $wgTitle = $oldTitle;
-                       }
-                       $this->mContent = Article::getRevisionText( $s );
-                       $this->mUser = $s->old_user;
-                       $this->mUserText = $s->old_user_text;
-                       $this->mComment = $s->old_comment;
-                       $this->mCounter = 0;
-                       $this->mTimestamp = wfTimestamp(TS_MW,$s->old_timestamp);
+               # if the title's different from expected, update...
+               if( $globalTitle &&
+                       ( $this->mTitle->getNamespace() != $s->page_namespace ||
+                       $this->mTitle->getDBkey() != $s->page_title ) ) {
+                       $oldTitle = Title::makeTitle( $s->page_namesapce, $s->page_title );
+                       $this->mTitle = $oldTitle;
+                       global $wgTitle;
+                       $wgTitle = $oldTitle;
                }
+               
+               # Back to the business at hand...
+               $this->mCounter = $s->page_counter;
+               $this->mTitle->mRestrictions = explode( ',', trim( $s->page_restrictions ) );
+               $this->mTitle->mRestrictionsLoaded = true;
+               $this->mTouched = wfTimestamp( TS_MW, $s->page_touched );
+
+               $this->mContent = Revision::getRevisionText( $s );
+               
+               $this->mUser = $s->rev_user;
+               $this->mUserText = $s->rev_user_text;
+               $this->mComment = $s->rev_comment;
+               $this->mTimestamp = wfTimestamp( TS_MW, $s->rev_timestamp );
+               
                $this->mContentLoaded = true;
                return $this->mContent;
        }
 
-       # Gets the article text without using so many damn globals
-       # Returns false on error
+       /**
+        * Gets the article text without using so many damn globals
+        * Returns false on error
+        *
+        * @param integer $oldid
+        */
        function getContentWithoutUsingSoManyDamnGlobals( $oldid = 0, $noredir = false ) {
-               global $wgMwRedir;
-
-               if ( $this->mContentLoaded ) {
-                       return $this->mContent;
-               }
-               $this->mContent = false;
-
-               $fname = 'Article::getContentWithout';
-               $dbr =& $this->getDB();
-
-               if ( ! $oldid ) {       # Retrieve current version
-                       $id = $this->getID();
-                       if ( 0 == $id ) {
-                               return false;
-                       }
-
-                       $s = $dbr->selectRow( 'cur', $this->getCurContentFields(), array( 'cur_id' => $id ), 
-                               $fname, $this->getSelectOptions() );
-                       if ( $s === false ) {
-                               return false;
-                       }
-
-                       # If we got a redirect, follow it (unless we've been told
-                       # not to by either the function parameter or the query
-                       if ( !$noredir ) {
-                               $rt = Title::newFromRedirect( $s->cur_text );
-                               if( $rt &&  $rt->getInterwiki() == '' && $rt->getNamespace() != NS_SPECIAL ) {
-                                       $rid = $rt->getArticleID();
-                                       if ( 0 != $rid ) {
-                                               $redirRow = $dbr->selectRow( 'cur', $this->getCurContentFields(), 
-                                                       array( 'cur_id' => $rid ), $fname, $this->getSelectOptions() );
-
-                                               if ( $redirRow !== false ) {
-                                                       $this->mRedirectedFrom = $this->mTitle->getPrefixedText();
-                                                       $this->mTitle = $rt;
-                                                       $s = $redirRow;
-                                               }
-                                       }
-                               }
-                       }
-
-                       $this->mContent = $s->cur_text;
-                       $this->mUser = $s->cur_user;
-                       $this->mUserText = $s->cur_user_text;
-                       $this->mComment = $s->cur_comment;
-                       $this->mCounter = $s->cur_counter;
-                       $this->mTimestamp = wfTimestamp(TS_MW,$s->cur_timestamp);
-                       $this->mTouched = wfTimestamp(TS_MW,$s->cur_touched);
-                       $this->mTitle->mRestrictions = explode( ',', trim( $s->cur_restrictions ) );
-                       $this->mTitle->mRestrictionsLoaded = true;
-               } else { # oldid set, retrieve historical version
-                       $s = $dbr->selectRow( 'old', $this->getOldContentFields(), array( 'old_id' => $oldid ), 
-                               $fname, $this->getSelectOptions() );
-                       if ( $s === false ) {
-                               return false;
-                       }
-                       $this->mContent = Article::getRevisionText( $s );
-                       $this->mUser = $s->old_user;
-                       $this->mUserText = $s->old_user_text;
-                       $this->mComment = $s->old_comment;
-                       $this->mCounter = 0;
-                       $this->mTimestamp = wfTimestamp(TS_MW,$s->old_timestamp);
-               }
-               $this->mContentLoaded = true;
-               return $this->mContent;
+               return $this->fetchContent( $oldid, $noredir, false );
        }
 
-       # Read/write accessor to select FOR UPDATE
+       /**
+        * Read/write accessor to select FOR UPDATE
+        */
        function forUpdate( $x = NULL ) {
                return wfSetVar( $this->mForUpdate, $x );
        }
-       
-       # Get the database which should be used for reads
+
+       /**
+        * Get the database which should be used for reads
+        */
        function &getDB() {
-               if ( $this->mForUpdate ) {
+               #if ( $this->mForUpdate ) {
                        return wfGetDB( DB_MASTER );
-               } else {
-                       return wfGetDB( DB_SLAVE );
-               }
+               #} else {
+               #       return wfGetDB( DB_SLAVE );
+               #}
        }
 
-       # Get options for all SELECT statements
-       # Can pass an option array, to which the class-wide options will be appended
+       /**
+        * Get options for all SELECT statements
+        * Can pass an option array, to which the class-wide options will be appended
+        */
        function getSelectOptions( $options = '' ) {
                if ( $this->mForUpdate ) {
-                       if ( $options ) {
+                       if ( is_array( $options ) ) {
                                $options[] = 'FOR UPDATE';
                        } else {
                                $options = 'FOR UPDATE';
                        }
-               } 
+               }
                return $options;
        }
-       
+
+       /**
+        * Return the Article ID
+        */
        function getID() {
                if( $this->mTitle ) {
                        return $this->mTitle->getArticleID();
@@ -536,47 +426,76 @@ class Article {
                }
        }
 
+       /**
+        * Get the view count for this article
+        */
        function getCount() {
                if ( -1 == $this->mCounter ) {
                        $id = $this->getID();
                        $dbr =& $this->getDB();
-                       $this->mCounter = $dbr->selectField( 'cur', 'cur_counter', 'cur_id='.$id, 
+                       $this->mCounter = $dbr->selectField( 'page', 'page_counter', array( 'page_id' => $id ),
                                'Article::getCount', $this->getSelectOptions() );
                }
                return $this->mCounter;
        }
 
-       # Would the given text make this article a "good" article (i.e.,
-       # suitable for including in the article count)?
+       /**
+        * Would the given text make this article a "good" article (i.e.,
+        * suitable for including in the article count)?
+        * @param string $text Text to analyze
+        * @return integer 1 if it can be counted else 0
+        */
        function isCountable( $text ) {
-               global $wgUseCommaCount, $wgMwRedir;
+               global $wgUseCommaCount;
 
-               if ( 0 != $this->mTitle->getNamespace() ) { return 0; }
-               if ( $wgMwRedir->matchStart( $text ) ) { return 0; }
+               if ( NS_MAIN != $this->mTitle->getNamespace() ) { return 0; }
+               if ( $this->isRedirect( $text ) ) { return 0; }
                $token = ($wgUseCommaCount ? ',' : '[[' );
                if ( false === strstr( $text, $token ) ) { return 0; }
                return 1;
        }
 
-       # Loads everything from cur except cur_text
-       # This isn't necessary for all uses, so it's only done if needed.
-       /* private */ function loadLastEdit() {
+       /** 
+        * Tests if the article text represents a redirect
+        */
+       function isRedirect( $text = false ) {
+               if ( $text === false ) {
+                       $this->loadContent();
+                       $titleObj = Title::newFromRedirect( $this->mText );
+               } else {
+                       $titleObj = Title::newFromRedirect( $text );
+               }
+               return $titleObj !== NULL;
+       }
+
+       /**
+        * Loads everything except the text
+        * This isn't necessary for all uses, so it's only done if needed.
+        * @private
+        */
+       function loadLastEdit() {
                global $wgOut;
-               if ( -1 != $this->mUser ) return;
+
+               if ( -1 != $this->mUser )
+                       return;
+
+               # New or non-existent articles have no user information
+               $id = $this->getID();
+               if ( 0 == $id ) return;
 
                $fname = 'Article::loadLastEdit';
 
                $dbr =& $this->getDB();
-               $s = $dbr->getArray( 'cur',
-                 array( 'cur_user','cur_user_text','cur_timestamp', 'cur_comment','cur_minor_edit' ),
-                 array( 'cur_id' => $this->getID() ), $fname, $this->getSelectOptions() );
+               $s = $dbr->selectRow( array( 'revision', 'page') ,
+                 array( 'rev_user','rev_user_text','rev_timestamp', 'rev_comment','rev_minor_edit' ),
+                 array( 'page_id' => $id, 'page_latest=rev_id' ), $fname, $this->getSelectOptions() );
 
                if ( $s !== false ) {
-                       $this->mUser = $s->cur_user;
-                       $this->mUserText = $s->cur_user_text;
-                       $this->mTimestamp = wfTimestamp(TS_MW,$s->cur_timestamp);
-                       $this->mComment = $s->cur_comment;
-                       $this->mMinorEdit = $s->cur_minor_edit;
+                       $this->mUser = $s->rev_user;
+                       $this->mUserText = $s->rev_user_text;
+                       $this->mTimestamp = wfTimestamp(TS_MW,$s->rev_timestamp);
+                       $this->mComment = $s->rev_comment;
+                       $this->mMinorEdit = $s->rev_minor_edit;
                }
        }
 
@@ -613,47 +532,50 @@ class Article {
                $title = $this->mTitle;
                $contribs = array();
                $dbr =& $this->getDB();
-               $oldTable = $dbr->tableName( 'old' );
+               $revTable = $dbr->tableName( 'revision' );
                $userTable = $dbr->tableName( 'user' );
-               $encDBkey = $dbr->strencode( $title->getDBkey() );
+               $encDBkey = $dbr->addQuotes( $title->getDBkey() );
                $ns = $title->getNamespace();
                $user = $this->getUser();
+               $pageId = $this->getId();
 
-               $sql = "SELECT old_user, old_user_text, user_real_name, MAX(old_timestamp) as timestamp
-                       FROM $oldTable LEFT JOIN $userTable ON old_user = user_id
-                       WHERE old_namespace = $user
-                       AND old_title = $encDBkey
-                       AND old_user != $user
-                       GROUP BY old_user
+               $sql = "SELECT rev_user, rev_user_text, user_real_name, MAX(rev_timestamp) as timestamp
+                       FROM $revTable LEFT JOIN $userTable ON rev_user = user_id
+                       WHERE rev_page = $pageId
+                       AND rev_user != $user
+                       GROUP BY rev_user, rev_user_text, user_real_name
                        ORDER BY timestamp DESC";
 
                if ($limit > 0) { $sql .= ' LIMIT '.$limit; }
                $sql .= ' '. $this->getSelectOptions();
 
                $res = $dbr->query($sql, $fname);
-               
+
                while ( $line = $dbr->fetchObject( $res ) ) {
-                       $contribs[] = array($line->old_user, $line->old_user_text, $line->user_real_name);
+                       $contribs[] = array($line->rev_user, $line->rev_user_text, $line->user_real_name);
                }
 
                $dbr->freeResult($res);
                return $contribs;
        }
 
-       # This is the default action of the script: just view the page of
-       # the given title.
-
+       /**
+        * This is the default action of the script: just view the page of
+        * the given title.
+       */
        function view() {
-               global $wgUser, $wgOut, $wgLang, $wgRequest, $wgMwRedir, $wgOnlySysopsCanPatrol;
+               global $wgUser, $wgOut, $wgRequest, $wgOnlySysopsCanPatrol, $wgLang;
                global $wgLinkCache, $IP, $wgEnableParserCache, $wgStylePath, $wgUseRCPatrol;
+               global $wgEnotif;
                $sk = $wgUser->getSkin();
 
                $fname = 'Article::view';
                wfProfileIn( $fname );
                # Get variables from query string
-               $oldid = $wgRequest->getVal( 'oldid' );
+               $oldid = $this->getOldID();
                $diff = $wgRequest->getVal( 'diff' );
                $rcid = $wgRequest->getVal( 'rcid' );
+               $rdfrom = $wgRequest->getVal( 'rdfrom' );
 
                $wgOut->setArticleFlag( true );
                $wgOut->setRobotpolicy( 'index,follow' );
@@ -663,22 +585,24 @@ class Article {
                if ( !is_null( $diff ) ) {
                        require_once( 'DifferenceEngine.php' );
                        $wgOut->setPageTitle( $this->mTitle->getPrefixedText() );
-                       $de = new DifferenceEngine( intval($oldid), intval($diff), intval($rcid) );
+                       $de = new DifferenceEngine( $oldid, $diff, $rcid );
                        $de->showDiffPage();
-                       wfProfileOut( $fname );
                        if( $diff == 0 ) {
                                # Run view updates for current revision only
                                $this->viewUpdates();
                        }
+                       wfProfileOut( $fname );
                        return;
                }
                if ( empty( $oldid ) && $this->checkTouched() ) {
                        if( $wgOut->checkLastModified( $this->mTouched ) ){
+                               wfProfileOut( $fname );
                                return;
                        } else if ( $this->tryFileCache() ) {
                                # tell wgOut that output is taken care of
                                $wgOut->disable();
                                $this->viewUpdates();
+                               wfProfileOut( $fname );
                                return;
                        }
                }
@@ -709,7 +633,7 @@ class Article {
                        # We're looking at an old revision
 
                        if ( !empty( $oldid ) ) {
-                               $this->setOldSubtitle();
+                               $this->setOldSubtitle( isset($this->mOldId) ? $this->mOldId : $oldid );
                                $wgOut->setRobotpolicy( 'noindex,follow' );
                        }
                        if ( '' != $this->mRedirectedFrom ) {
@@ -721,6 +645,11 @@ class Article {
 
                                # Can't cache redirects
                                $pcache = false;
+                       } elseif ( !empty( $rdfrom ) ) {
+                               $sk = $wgUser->getSkin();
+                               $redir = $sk->makeExternalLink( $rdfrom, $rdfrom );
+                               $s = wfMsg( 'redirectedfrom', $redir );
+                               $wgOut->setSubtitle( $s );
                        }
 
                        # wrap user css and user js in pre and don't parse
@@ -733,12 +662,14 @@ class Article {
                                $wgOut->addHTML( '<pre>'.htmlspecialchars($this->mContent)."\n</pre>" );
                        } else if ( $rt = Title::newFromRedirect( $text ) ) {
                                # Display redirect
-                               $imageUrl = $wgStylePath.'/images/redirect.png';
+                               $imageUrl = $wgStylePath.'/common/images/redirect.png';
                                $targetUrl = $rt->escapeLocalURL();
                                $titleText = htmlspecialchars( $rt->getPrefixedText() );
                                $link = $sk->makeLinkObj( $rt );
-                               $wgOut->addHTML( '<img valign="center" src="'.$imageUrl.'">' .
+
+                               $wgOut->addHTML( '<img valign="center" src="'.$imageUrl.'" alt="#REDIRECT" />' .
                                  '<span class="redirectText">'.$link.'</span>' );
+
                        } else if ( $pcache ) {
                                # Display content and save to parser cache
                                $wgOut->addPrimaryWikiText( $text, $this );
@@ -750,8 +681,8 @@ class Article {
                $wgOut->setPageTitle( $this->mTitle->getPrefixedText() );
                # If we have been passed an &rcid= parameter, we want to give the user a
                # chance to mark this new article as patrolled.
-               if ( $wgUseRCPatrol && !is_null ( $rcid ) && $rcid != 0 && $wgUser->getID() != 0 &&
-                    ( $wgUser->isSysop() || !$wgOnlySysopsCanPatrol ) )
+               if ( $wgUseRCPatrol && !is_null ( $rcid ) && $rcid != 0 && $wgUser->isLoggedIn() &&
+                    ( $wgUser->isAllowed('patrol') || !$wgOnlySysopsCanPatrol ) )
                {
                        $wgOut->addHTML( wfMsg ( 'markaspatrolledlink',
                                $sk->makeKnownLinkObj ( $this->mTitle, wfMsg ( 'markaspatrolledtext' ),
@@ -764,18 +695,22 @@ class Article {
 
                # Add link titles as META keywords
                $wgOut->addMetaTags() ;
-                       
+
                $this->viewUpdates();
                wfProfileOut( $fname );
-       }
 
-       # Theoretically we could defer these whole insert and update
-       # functions for after display, but that's taking a big leap
-       # of faith, and we want to be able to report database
-       # errors at some point.
+               $wgUser->clearNotification( $this->mTitle );
+       }
 
-       /* private */ function insertNewArticle( $text, $summary, $isminor, $watchthis ) {
-               global $wgOut, $wgUser, $wgMwRedir;
+       /**
+        * Theoretically we could defer these whole insert and update
+        * functions for after display, but that's taking a big leap
+        * of faith, and we want to be able to report database
+        * errors at some point.
+        * @private
+        */
+       function insertNewArticle( $text, $summary, $isminor, $watchthis ) {
+               global $wgOut, $wgUser;
                global $wgUseSquid, $wgDeferredUpdateList, $wgInternalServer;
 
                $fname = 'Article::insertNewArticle';
@@ -785,39 +720,52 @@ class Article {
                $ns = $this->mTitle->getNamespace();
                $ttl = $this->mTitle->getDBkey();
                $text = $this->preSaveTransform( $text );
-               if ( $wgMwRedir->matchStart( $text ) ) { $redir = 1; }
+               if ( $this->isRedirect( $text ) ) { $redir = 1; }
                else { $redir = 0; }
 
                $now = wfTimestampNow();
-               $won = wfInvertTimestamp( $now );
                wfSeedRandom();
-               $rand = number_format( mt_rand() / mt_getrandmax(), 12, '.', '' );
+               $rand = wfRandom();
+               $isminor = ( $isminor && $wgUser->isLoggedIn() ) ? 1 : 0;
+               
+               $mungedText = $text;
+               $flags = Revision::compressRevisionText( $mungedText );
+
                $dbw =& wfGetDB( DB_MASTER );
 
-               $cur_id = $dbw->nextSequenceValue( 'cur_cur_id_seq' );
-
-               $isminor = ( $isminor && $wgUser->getID() ) ? 1 : 0;
-
-               $dbw->insertArray( 'cur', array(
-                       'cur_id' => $cur_id,
-                       'cur_namespace' => $ns,
-                       'cur_title' => $ttl,
-                       'cur_text' => $text,
-                       'cur_comment' => $summary,
-                       'cur_user' => $wgUser->getID(),
-                       'cur_timestamp' => $dbw->timestamp($now),
-                       'cur_minor_edit' => $isminor,
-                       'cur_counter' => 0,
-                       'cur_restrictions' => '',
-                       'cur_user_text' => $wgUser->getName(),
-                       'cur_is_redirect' => $redir,
-                       'cur_is_new' => 1,
-                       'cur_random' => $rand,
-                       'cur_touched' => $dbw->timestamp($now),
-                       'inverse_timestamp' => $won,
+               $old_id = $dbw->nextSequenceValue( 'text_old_id_seq' );
+               $dbw->insert( 'text', array(
+                       'old_id' => $old_id,
+                       'old_text' => $mungedText,
+                       'old_flags' => $flags,
+               ), $fname );
+               $revisionId = $dbw->insertId();
+               
+               $page_id = $dbw->nextSequenceValue( 'page_page_id_seq' );
+               $dbw->insert( 'page', array(
+                       'page_id' => $page_id,
+                       'page_namespace' => $ns,
+                       'page_title' => $ttl,
+                       'page_counter' => 0,
+                       'page_restrictions' => '',
+                       'page_is_redirect' => $redir,
+                       'page_is_new' => 1,
+                       'page_random' => $rand,
+                       'page_touched' => $dbw->timestamp($now),
+                       'page_latest' => $revisionId,
                ), $fname );
-
                $newid = $dbw->insertId();
+
+               $dbw->insert( 'revision', array(
+                       'rev_page' => $newid,
+                       'rev_id' => $revisionId,
+                       'rev_comment' => $summary,
+                       'rev_user' => $wgUser->getID(),
+                       'rev_timestamp' => $dbw->timestamp($now),
+                       'rev_minor_edit' => $isminor,
+                       'rev_user_text' => $wgUser->getName(),
+               ), $fname );
+
                $this->mTitle->resetArticleID( $newid );
 
                Article::onArticleCreate( $this->mTitle );
@@ -833,20 +781,56 @@ class Article {
 
                # The talk page isn't in the regular link tables, so we need to update manually:
                $talkns = $ns ^ 1; # talk -> normal; normal -> talk
-               $dbw->updateArray( 'cur', array('cur_touched' => $dbw->timestamp($now) ), 
-                       array(  'cur_namespace' => $talkns, 'cur_title' => $ttl ), $fname );
+               $dbw->update( 'page',
+                       array( 'page_touched' => $dbw->timestamp($now) ),
+                       array( 'page_namespace' => $talkns,
+                              'page_title' => $ttl ),
+                       $fname );
 
                # standard deferred updates
-               $this->editUpdates( $text );
+               $this->editUpdates( $text, $summary, $isminor, $now );
 
-               $this->showArticle( $text, wfMsg( 'newarticle' ) );
+               $oldid = 0; # new article
+               $this->showArticle( $text, wfMsg( 'newarticle' ), false, $isminor, $now, $summary, $oldid );
        }
 
-
-       /* Side effects: loads last edit */
-       function getTextOfLastEditWithSectionReplacedOrAdded($section, $text, $summary = '') {
-               $this->loadLastEdit();
-               $oldtext = $this->getContent( true );
+       /**
+        * Fetch and uncompress the text for a given revision.
+        * Can ask by rev_id number or timestamp (set $field)
+        */
+       function fetchRevisionText( $revId = null, $field = 'rev_id' ) {
+               $fname = 'Article::fetchRevisionText';
+               $dbw =& wfGetDB( DB_MASTER );
+               if( $revId ) {
+                       $rev = $dbw->addQuotes( $revId );
+               } else {
+                       $rev = 'page_latest';
+               }
+               $result = $dbw->query(
+                       sprintf( "SELECT old_text, old_flags
+                               FROM %s,%s,%s
+                               WHERE old_id=rev_id AND rev_page=page_id AND page_id=%d
+                               AND %s=%s",
+                               $dbw->tableName( 'page' ),
+                               $dbw->tableName( 'revision' ),
+                               $dbw->tableName( 'text' ),
+                               IntVal( $this->mTitle->getArticleId() ),
+                               $field,
+                               $rev ),
+                       $fname );
+               $obj = $dbw->fetchObject( $result );
+               $dbw->freeResult( $result );
+               $oldtext = Revision::getRevisionText( $obj );
+               return $oldtext;
+       }
+       
+       function getTextOfLastEditWithSectionReplacedOrAdded($section, $text, $summary = '', $edittime = NULL) {
+               $fname = 'Article::getTextOfLastEditWithSectionReplacedOrAdded';
+               if( is_null( $edittime ) ) {
+                       $oldtext = $this->fetchRevisionText();
+               } else {
+                       $oldtext = $this->fetchRevisionText( $edittime, 'rev_timestamp' );
+               }
                if ($section != '') {
                        if($section=='new') {
                                if($summary) $subject="== {$summary} ==\n\n";
@@ -919,19 +903,32 @@ class Article {
                return $text;
        }
 
+       /**
+        * Change an existing article. Puts the previous version back into the old table, updates RC 
+        * and all necessary caches, mostly via the deferred update array.
+        *
+        * It is possible to call this function from a command-line script, but note that you should 
+        * first set $wgUser, and clean up $wgDeferredUpdates after each edit.
+        */
        function updateArticle( $text, $summary, $minor, $watchthis, $forceBot = false, $sectionanchor = '' ) {
                global $wgOut, $wgUser;
                global $wgDBtransactions, $wgMwRedir;
-               global $wgUseSquid, $wgInternalServer;
+               global $wgUseSquid, $wgInternalServer, $wgPostCommitUpdateList;
 
                $fname = 'Article::updateArticle';
                $good = true;
 
                if ( $this->mMinorEdit ) { $me1 = 1; } else { $me1 = 0; }
-               if ( $minor && $wgUser->getID() ) { $me2 = 1; } else { $me2 = 0; }
-               if ( preg_match( "/^((" . $wgMwRedir->getBaseRegex() . ')[^\\n]+)/i', $text, $m ) ) {
-                       $redir = 1;
-                       $text = $m[1] . "\n"; # Remove all content but redirect
+               if ( $minor && $wgUser->isLoggedIn() ) { $me2 = 1; } else { $me2 = 0; }
+               if ( $this->isRedirect( $text ) ) {
+                       # Remove all content but redirect
+                       # This could be done by reconstructing the redirect from a title given by 
+                       # Title::newFromRedirect(), but then we wouldn't know which synonym the user
+                       # wants to see
+                       if ( preg_match( "/^((" . $wgMwRedir->getBaseRegex() . ')[^\\n]+)/i', $text, $m ) ) {
+                               $redir = 1;
+                               $text = $m[1] . "\n";
+                       }
                }
                else { $redir = 0; }
 
@@ -955,24 +952,47 @@ class Article {
                        $this->mCountAdjustment = $this->isCountable( $text )
                          - $this->isCountable( $oldtext );
                        $now = wfTimestampNow();
-                       $won = wfInvertTimestamp( $now );
 
-                       # First update the cur row
-                       $dbw->updateArray( 'cur',
+                       $mungedText = $text;
+                       $flags = Revision::compressRevisionText( $mungedText );
+                       
+                       $lastRevision = $dbw->selectField(
+                               'page', 'page_latest', array( 'page_id' => $this->getId() ) );
+                       
+                       # Record the text to the text table
+                       $old_id = $dbw->nextSequenceValue( 'text_old_id_val' );
+                       $dbw->insert( 'text',
+                               array(
+                                       'old_id' => $old_id,
+                                       'old_text' => $mungedText,
+                                       'old_flags' => $flags,
+                               ), $fname
+                       );
+                       $revisionId = $dbw->insertId();
+                       
+                       # Record the edit in revisions
+                       $dbw->insert( 'revision',
+                               array(
+                                       'rev_id' => $revisionId,
+                                       'rev_page' => $this->getID(),
+                                       'rev_comment' => $summary,
+                                       'rev_minor_edit' => $me2,
+                                       'rev_user' => $wgUser->getID(),
+                                       'rev_user_text' => $wgUser->getName(),
+                                       'rev_timestamp' => $dbw->timestamp( $now ),
+                               ), $fname
+                       );
+                       
+                       # Update page
+                       $dbw->update( 'page',
                                array( /* SET */
-                                       'cur_text' => $text,
-                                       'cur_comment' => $summary,
-                                       'cur_minor_edit' => $me2,
-                                       'cur_user' => $wgUser->getID(),
-                                       'cur_timestamp' => $dbw->timestamp($now),
-                                       'cur_user_text' => $wgUser->getName(),
-                                       'cur_is_redirect' => $redir,
-                                       'cur_is_new' => 0,
-                                       'cur_touched' => $dbw->timestamp($now),
-                                       'inverse_timestamp' => $won
+                                       'page_latest' => $revisionId,
+                                       'page_touched' => $dbw->timestamp( $now ),
+                                       'page_is_new' => 0,
+                                       'page_is_redirect' => $redir,
                                ), array( /* WHERE */
-                                       'cur_id' => $this->getID(),
-                                       'cur_timestamp' => $dbw->timestamp($this->getTimestamp())
+                                       'page_id' => $this->getID(),
+                                       'page_latest' => $lastRevision, # Paranoia
                                ), $fname
                        );
 
@@ -980,32 +1000,10 @@ class Article {
                                /* Belated edit conflict! Run away!! */
                                $good = false;
                        } else {
-                               # Now insert the previous revision into old
-
-                               # This overwrites $oldtext if revision compression is on
-                               $flags = Article::compressRevisionText( $oldtext );
-
-                               $dbw->insertArray( 'old',
-                                       array(
-                                               'old_id' => $dbw->nextSequenceValue( 'old_old_id_seq' ),
-                                               'old_namespace' => $this->mTitle->getNamespace(),
-                                               'old_title' => $this->mTitle->getDBkey(),
-                                               'old_text' => $oldtext,
-                                               'old_comment' => $this->getComment(),
-                                               'old_user' => $this->getUser(),
-                                               'old_user_text' => $this->getUserText(),
-                                               'old_timestamp' => $dbw->timestamp($this->getTimestamp()),
-                                               'old_minor_edit' => $me1,
-                                               'inverse_timestamp' => wfInvertTimestamp( $this->getTimestamp() ),
-                                               'old_flags' => $flags,
-                                       ), $fname
-                               );
-
-                               $oldid = $dbw->insertId();
-
+                               # Update recentchanges and purge cache and whatnot
                                $bot = (int)($wgUser->isBot() || $forceBot);
                                RecentChange::notifyEdit( $now, $this->mTitle, $me2, $wgUser, $summary,
-                                       $oldid, $this->getTimestamp(), $bot );
+                                       $lastRevision, $this->getTimestamp(), $bot );
                                Article::onArticleEdit( $this->mTitle );
                        }
                }
@@ -1025,7 +1023,7 @@ class Article {
                        }
                }
                # standard deferred updates
-               $this->editUpdates( $text );
+               $this->editUpdates( $text, $summary, $minor, $now );
 
 
                $urls = array();
@@ -1045,20 +1043,20 @@ class Article {
                if ( $wgUseSquid ) {
                        $urls = array_merge( $urls, $this->mTitle->getSquidURLs() );
                        $u = new SquidUpdate( $urls );
-                       $u->doUpdate();
+                       array_push( $wgPostCommitUpdateList, $u );
                }
 
-               $this->showArticle( $text, wfMsg( 'updated' ), $sectionanchor );
+               $this->showArticle( $text, wfMsg( 'updated' ), $sectionanchor, $me2, $now, $summary, $lastRevision );
                }
                return $good;
        }
 
-       # After we've either updated or inserted the article, update
-       # the link tables and redirect to the new page.
-
-       function showArticle( $text, $subtitle , $sectionanchor = '' ) {
-               global $wgOut, $wgUser, $wgLinkCache;
-               global $wgMwRedir;
+       /**
+        * After we've either updated or inserted the article, update
+        * the link tables and redirect to the new page.
+        */
+       function showArticle( $text, $subtitle , $sectionanchor = '', $me2, $now, $summary, $oldid ) {
+               global $wgOut, $wgUser, $wgLinkCache, $wgEnotif;
 
                $wgLinkCache = new LinkCache();
                # Select for update
@@ -1075,33 +1073,23 @@ class Article {
                # Look up the links in the DB and add them to the link cache
                $wgOut->transformBuffer( RLH_FOR_UPDATE );
 
-               if( $wgMwRedir->matchStart( $text ) )
+               if( $this->isRedirect( $text ) )
                        $r = 'redirect=no';
                else
                        $r = '';
                $wgOut->redirect( $this->mTitle->getFullURL( $r ).$sectionanchor );
-       }
 
-       # Validate article
+               # this call would better fit into RecentChange::notifyEdit and RecentChange::notifyNew .
+               # this will be improved later (to-do)
 
-       function validate () {
-               global $wgOut, $wgUseValidation;
-               if( $wgUseValidation ) {
-                       require_once ( 'SpecialValidate.php' ) ;
-                       $wgOut->setPagetitle( wfMsg( 'validate' ) . ': ' . $this->mTitle->getPrefixedText() );
-                       $wgOut->setRobotpolicy( 'noindex,follow' );
-                       if( $this->mTitle->getNamespace() != 0 ) {
-                               $wgOut->addHTML( wfMsg( 'val_validate_article_namespace_only' ) );
-                               return;
-                       }
-                       $v = new Validation;
-                       $v->validate_form( $this->mTitle->getDBkey() );
-               } else {
-                       $wgOut->errorpage( 'nosuchaction', 'nosuchactiontext' );
-               }
+               include_once( "UserMailer.php" );
+               $wgEnotif = new EmailNotification ();
+               $wgEnotif->NotifyOnPageChange( $wgUser->getID(), $this->mTitle->getDBkey(), $this->mTitle->getNamespace(),$now, $summary, $me2, $oldid );
        }
 
-       # Mark this particular edit as patrolled
+       /**
+        * Mark this particular edit as patrolled
+        */
        function markpatrolled() {
                global $wgOut, $wgRequest, $wgOnlySysopsCanPatrol, $wgUseRCPatrol, $wgUser;
                $wgOut->setRobotpolicy( 'noindex,follow' );
@@ -1111,12 +1099,12 @@ class Article {
                        $wgOut->errorpage( 'rcpatroldisabled', 'rcpatroldisabledtext' );
                        return;
                }
-               if ( $wgUser->getID() == 0 )
+               if ( $wgUser->isAnon() )
                {
                        $wgOut->loginToUse();
                        return;
                }
-               if ( $wgOnlySysopsCanPatrol && !$wgUser->isSysop() )
+               if ( $wgOnlySysopsCanPatrol && !$wgUser->isAllowed('patrol') )
                {
                        $wgOut->sysopRequired();
                        return;
@@ -1127,7 +1115,9 @@ class Article {
                        RecentChange::markPatrolled( $rcid );
                        $wgOut->setPagetitle( wfMsg( 'markedaspatrolled' ) );
                        $wgOut->addWikiText( wfMsg( 'markedaspatrolledtext' ) );
-                       $wgOut->returnToMain( true, $this->mTitle->getPrefixedText() );
+
+                       $rcTitle = Title::makeTitle( NS_SPECIAL, 'Recentchanges' );
+                       $wgOut->returnToMain( false, $rcTitle->getPrefixedText() );
                }
                else
                {
@@ -1136,13 +1126,15 @@ class Article {
        }
 
 
-       # Add this page to my watchlist
+       /**
+        * Add this page to $wgUser's watchlist
+        */
        
-       function watch( $add = true ) {
-               global $wgUser, $wgOut, $wgLang;
-               global $wgDeferredUpdateList;
+       function watch() {
+               
+               global $wgUser, $wgOut;
 
-               if ( 0 == $wgUser->getID() ) {
+               if ( $wgUser->isAnon() ) {
                        $wgOut->errorpage( 'watchnologin', 'watchnologintext' );
                        return;
                }
@@ -1150,39 +1142,67 @@ class Article {
                        $wgOut->readOnlyPage();
                        return;
                }
-               if( $add )
-                       $wgUser->addWatch( $this->mTitle );
-               else
-                       $wgUser->removeWatch( $this->mTitle );
-
-               $wgOut->setPagetitle( wfMsg( $add ? 'addedwatch' : 'removedwatch' ) );
-               $wgOut->setRobotpolicy( 'noindex,follow' );
 
-               $sk = $wgUser->getSkin() ;
-               $link = $this->mTitle->getPrefixedText();
+               if (wfRunHooks('WatchArticle', $wgUser, $this)) {
+                       
+                       $wgUser->addWatch( $this->mTitle );
+                       $wgUser->saveSettings();
 
-               if($add)
+                       wfRunHooks('WatchArticleComplete', $wgUser, $this);
+                       
+                       $wgOut->setPagetitle( wfMsg( 'addedwatch' ) );
+                       $wgOut->setRobotpolicy( 'noindex,follow' );
+                       
+                       $link = $this->mTitle->getPrefixedText();
                        $text = wfMsg( 'addedwatchtext', $link );
-               else
-                       $text = wfMsg( 'removedwatchtext', $link );
-               $wgOut->addWikiText( $text );
-
-               $up = new UserUpdate();
-               array_push( $wgDeferredUpdateList, $up );
-
+                       $wgOut->addWikiText( $text );
+               }
+               
                $wgOut->returnToMain( true, $this->mTitle->getPrefixedText() );
        }
 
+       /**
+        * Stop watching a page
+        */
+       
        function unwatch() {
-               $this->watch( false );
-       }
 
-       # protect a page
+               global $wgUser, $wgOut;
+
+               if ( $wgUser->isAnon() ) {
+                       $wgOut->errorpage( 'watchnologin', 'watchnologintext' );
+                       return;
+               }
+               if ( wfReadOnly() ) {
+                       $wgOut->readOnlyPage();
+                       return;
+               }
+
+               if (wfRunHooks('UnwatchArticle', $wgUser, $this)) {
+                       
+                       $wgUser->removeWatch( $this->mTitle );
+                       $wgUser->saveSettings();
+                       
+                       wfRunHooks('UnwatchArticleComplete', $wgUser, $this);
+                       
+                       $wgOut->setPagetitle( wfMsg( 'removedwatch' ) );
+                       $wgOut->setRobotpolicy( 'noindex,follow' );
+                       
+                       $link = $this->mTitle->getPrefixedText();
+                       $text = wfMsg( 'removedwatchtext', $link );
+                       $wgOut->addWikiText( $text );
+               }
+               
+               $wgOut->returnToMain( true, $this->mTitle->getPrefixedText() );
+       }
 
+       /**
+        * protect a page
+        */
        function protect( $limit = 'sysop' ) {
                global $wgUser, $wgOut, $wgRequest;
 
-               if ( ! $wgUser->isSysop() ) {
+               if ( ! $wgUser->isAllowed('protect') ) {
                        $wgOut->sysopRequired();
                        return;
                }
@@ -1196,27 +1216,49 @@ class Article {
                        return;
                }
 
-               $confirm = $wgRequest->getBool( 'wpConfirmProtect' ) && $wgRequest->wasPosted();
+               $confirm = $wgRequest->getBool( 'wpConfirmProtect' ) &&
+                       $wgRequest->wasPosted() &&
+                       $wgUser->matchEditToken( $wgRequest->getVal( 'wpEditToken' ) );
+               $moveonly = $wgRequest->getBool( 'wpMoveOnly' );
                $reason = $wgRequest->getText( 'wpReasonProtect' );
 
                if ( $confirm ) {
                        $dbw =& wfGetDB( DB_MASTER );
-                       $dbw->updateArray( 'cur',
+                       $dbw->update( 'page',
                                array( /* SET */
-                                       'cur_touched' => $dbw->timestamp(),
-                                       'cur_restrictions' => (string)$limit
+                                       'page_touched' => $dbw->timestamp(),
+                                       'page_restrictions' => (string)$limit
                                ), array( /* WHERE */
-                                       'cur_id' => $id
+                                       'page_id' => $id
                                ), 'Article::protect'
                        );
 
-                       $log = new LogPage( 'protect' );
-                       if ( $limit === '' ) {
+                       $restrictions = "move=" . $limit;
+                       if( !$moveonly ) {
+                               $restrictions .= ":edit=" . $limit;
+                       }
+                       if (wfRunHooks('ArticleProtect', $this, $wgUser, $limit == 'sysop', $reason, $moveonly)) {
+                               
+                               $dbw =& wfGetDB( DB_MASTER );
+                               $dbw->update( 'page',
+                                                         array( /* SET */
+                                                                        'page_touched' => $dbw->timestamp(),
+                                                                        'page_restrictions' => $restrictions
+                                                                        ), array( /* WHERE */
+                                                                                          'page_id' => $id
+                                                                                          ), 'Article::protect'
+                                                         );
+                               
+                               wfRunHooks('ArticleProtectComplete', $this, $wgUser, $limit == 'sysop', $reason, $moveonly);
+                               
+                               $log = new LogPage( 'protect' );
+                               if ( $limit === '' ) {
                                        $log->addEntry( 'unprotect', $this->mTitle, $reason );
-                       } else {
+                               } else {
                                        $log->addEntry( 'protect', $this->mTitle, $reason );
+                               }
+                               $wgOut->redirect( $this->mTitle->getFullURL() );
                        }
-                       $wgOut->redirect( $this->mTitle->getFullURL() );
                        return;
                } else {
                        $reason = htmlspecialchars( wfMsg( 'protectreason' ) );
@@ -1224,10 +1266,11 @@ class Article {
                }
        }
 
-       # Output protection confirmation dialog
-       
+       /**
+        * Output protection confirmation dialog
+        */
        function confirmProtect( $par, $reason, $limit = 'sysop'  ) {
-               global $wgOut;
+               global $wgOut, $wgUser;
 
                wfDebug( "Article::confirmProtect\n" );
 
@@ -1236,6 +1279,7 @@ class Article {
 
                $check = '';
                $protcom = '';
+               $moveonly = '';
 
                if ( $limit === '' ) {
                        $wgOut->setPageTitle( wfMsg( 'confirmunprotect' ) );
@@ -1249,11 +1293,13 @@ class Article {
                        $wgOut->setSubtitle( wfMsg( 'protectsub', $sub ) );
                        $wgOut->addWikiText( wfMsg( 'confirmprotecttext' ) );
                        $check = htmlspecialchars( wfMsg( 'confirmprotect' ) );
+                       $moveonly = htmlspecialchars( wfMsg( 'protectmoveonly' ) );
                        $protcom = htmlspecialchars( wfMsg( 'protectcomment' ) );
                        $formaction = $this->mTitle->escapeLocalURL( 'action=protect' . $par );
                }
 
                $confirm = htmlspecialchars( wfMsg( 'confirm' ) );
+               $token = htmlspecialchars( $wgUser->editToken() );
 
                $wgOut->addHTML( "
 <form id='protectconfirm' method='post' action=\"{$formaction}\">
@@ -1276,7 +1322,19 @@ class Article {
                        <td>
                                <label for='wpConfirmProtect'>{$check}</label>
                        </td>
-               </tr>
+               </tr> " );
+               if($moveonly != '') {
+                       $wgOut->AddHTML( "
+               <tr>
+                       <td align='right'>
+                               <input type='checkbox' name='wpMoveOnly' value='1' id='wpMoveOnly' />
+                       </td>
+                       <td>
+                               <label for='wpMoveOnly'>{$moveonly}</label>
+                       </td>
+               </tr> " );
+               }
+               $wgOut->addHTML( "
                <tr>
                        <td>&nbsp;</td>
                        <td>
@@ -1284,27 +1342,34 @@ class Article {
                        </td>
                </tr>
        </table>
+       <input type='hidden' name='wpEditToken' value=\"{$token}\" />
 </form>\n" );
 
                $wgOut->returnToMain( false );
        }
 
+       /**
+        * Unprotect the pages
+        */
        function unprotect() {
                return $this->protect( '' );
        }
 
-       # UI entry point for page deletion
-       
+       /*
+        * UI entry point for page deletion
+        */
        function delete() {
                global $wgUser, $wgOut, $wgMessageCache, $wgRequest;
                $fname = 'Article::delete';
-               $confirm = $wgRequest->getBool( 'wpConfirm' ) && $wgRequest->wasPosted();
+               $confirm = $wgRequest->getBool( 'wpConfirm' ) &&
+                       $wgRequest->wasPosted() &&
+                       $wgUser->matchEditToken( $wgRequest->getVal( 'wpEditToken' ) );
                $reason = $wgRequest->getText( 'wpReason' );
 
                # This code desperately needs to be totally rewritten
 
                # Check permissions
-               if ( ( ! $wgUser->isSysop() ) ) {
+               if ( ( ! $wgUser->isAllowed('delete') ) ) {
                        $wgOut->sysopRequired();
                        return;
                }
@@ -1332,26 +1397,28 @@ class Article {
                $dbr =& $this->getDB();
                $ns = $this->mTitle->getNamespace();
                $title = $this->mTitle->getDBkey();
-               $old = $dbr->getArray( 'old',
-                       array( 'old_text', 'old_flags' ),
+               $revisions = $dbr->select( array( 'page', 'revision' ),
+                       array( 'rev_id' ),
                        array(
-                               'old_namespace' => $ns,
-                               'old_title' => $title,
-                       ), $fname, $this->getSelectOptions( array( 'ORDER BY' => 'inverse_timestamp' ) )
+                               'page_namespace' => $ns,
+                               'page_title' => $title,
+                               'rev_page = page_id'
+                       ), $fname, $this->getSelectOptions( array( 'ORDER BY' => 'rev_timestamp DESC' ) )
                );
 
-               if( $old !== false && !$confirm ) {
+               if( $dbr->numRows( $revisions ) > 1 && !$confirm ) {
                        $skin=$wgUser->getSkin();
                        $wgOut->addHTML('<b>'.wfMsg('historywarning'));
                        $wgOut->addHTML( $skin->historyLink() .'</b>');
                }
 
                # Fetch cur_text
-               $s = $dbr->getArray( 'cur',
-                       array( 'cur_text' ),
+               $s = $dbr->selectRow( array( 'page', 'text' ),
+                       array( 'old_text' ),
                        array(
-                               'cur_namespace' => $ns,
-                               'cur_title' => $title,
+                               'page_namespace' => $ns,
+                               'page_title' => $title,
+                               'page_latest = old_id'
                        ), $fname, $this->getSelectOptions()
                );
 
@@ -1360,11 +1427,11 @@ class Article {
 
                        #if this is empty, an earlier revision may contain "useful" text
                        $blanked = false;
-                       if($s->cur_text != '') {
-                               $text=$s->cur_text;
+                       if($s->old_text != '') {
+                               $text=$s->old_text;
                        } else {
-                               if($old) {
-                                       $text = Article::getRevisionText( $old );
+                               if($old) { # TODO
+                                       $text = Revision::getRevisionText( $old );
                                        $blanked = true;
                                }
 
@@ -1401,10 +1468,11 @@ class Article {
                return $this->confirmDelete( '', $reason );
        }
 
-       # Output deletion confirmation dialog
-       
+       /**
+        * Output deletion confirmation dialog
+        */
        function confirmDelete( $par, $reason ) {
-               global $wgOut;
+               global $wgOut, $wgUser;
 
                wfDebug( "Article::confirmDelete\n" );
 
@@ -1418,6 +1486,7 @@ class Article {
                $confirm = htmlspecialchars( wfMsg( 'confirm' ) );
                $check = htmlspecialchars( wfMsg( 'confirmcheck' ) );
                $delcom = htmlspecialchars( wfMsg( 'deletecomment' ) );
+               $token = htmlspecialchars( $wgUser->editToken() );
 
                $wgOut->addHTML( "
 <form id='deleteconfirm' method='post' action=\"{$formaction}\">
@@ -1448,44 +1517,52 @@ class Article {
                        </td>
                </tr>
        </table>
+       <input type='hidden' name='wpEditToken' value=\"{$token}\" />
 </form>\n" );
 
                $wgOut->returnToMain( false );
        }
 
 
-       # Perform a deletion and output success or failure messages
-       
+       /**
+        * Perform a deletion and output success or failure messages
+        */
        function doDelete( $reason ) {
-               global $wgOut, $wgUser, $wgLang;
+               global $wgOut, $wgUser, $wgContLang;
                $fname = 'Article::doDelete';
                wfDebug( $fname."\n" );
 
-               if ( $this->doDeleteArticle( $reason ) ) {
-                       $deleted = $this->mTitle->getPrefixedText();
-
-                       $wgOut->setPagetitle( wfMsg( 'actioncomplete' ) );
-                       $wgOut->setRobotpolicy( 'noindex,nofollow' );
-
-                       $sk = $wgUser->getSkin();
-                       $loglink = $sk->makeKnownLink( $wgLang->getNsText( NS_WIKIPEDIA ) .
-                         ':' . wfMsg( 'dellogpage' ), wfMsg( 'deletionlog' ) );
-
-                       $text = wfMsg( 'deletedtext', $deleted, $loglink );
-
-                       $wgOut->addHTML( '<p>' . $text . "</p>\n" );
-                       $wgOut->returnToMain( false );
-               } else {
-                       $wgOut->fatalError( wfMsg( 'cannotdelete' ) );
+               if (wfRunHooks('ArticleDelete', $this, $wgUser, $reason)) {
+                       if ( $this->doDeleteArticle( $reason ) ) {
+                               $deleted = $this->mTitle->getPrefixedText();
+                               
+                               $wgOut->setPagetitle( wfMsg( 'actioncomplete' ) );
+                               $wgOut->setRobotpolicy( 'noindex,nofollow' );
+                               
+                               $sk = $wgUser->getSkin();
+                               $loglink = $sk->makeKnownLink( $wgContLang->getNsText( NS_PROJECT ) .
+                                                                                          ':' . wfMsgForContent( 'dellogpage' ),
+                                                                                          wfMsg( 'deletionlog' ) );
+                               
+                               $text = wfMsg( 'deletedtext', $deleted, $loglink );
+                               
+                               $wgOut->addHTML( '<p>' . $text . "</p>\n" );
+                               $wgOut->returnToMain( false );
+                               wfRunHooks('ArticleDeleteComplete', $this, $wgUser, $reason);
+                       } else {
+                               $wgOut->fatalError( wfMsg( 'cannotdelete' ) );
+                       }
                }
        }
 
-       # Back-end article deletion
-       # Deletes the article with database consistency, writes logs, purges caches
-       # Returns success
+       /**
+        * Back-end article deletion
+        * Deletes the article with database consistency, writes logs, purges caches
+        * Returns success
+        */
        function doDeleteArticle( $reason ) {
-               global $wgUser, $wgLang;
-               global  $wgUseSquid, $wgDeferredUpdateList, $wgInternalServer;
+               global $wgUser;
+               global  $wgUseSquid, $wgDeferredUpdateList, $wgInternalServer, $wgPostCommitUpdateList;
 
                $fname = 'Article::doDeleteArticle';
                wfDebug( $fname."\n" );
@@ -1515,7 +1592,7 @@ class Article {
                        }
 
                        $u = new SquidUpdate( $urls );
-                       array_push( $wgDeferredUpdateList, $u );
+                       array_push( $wgPostCommitUpdateList, $u );
 
                }
 
@@ -1523,51 +1600,31 @@ class Article {
                Title::touchArray( $linksTo );
 
                # Move article and history to the "archive" table
-               $archiveTable = $dbw->tableName( 'archive' );
-               $oldTable = $dbw->tableName( 'old' );
-               $curTable = $dbw->tableName( 'cur' );
-               $recentchangesTable = $dbw->tableName( 'recentchanges' );
-               $linksTable = $dbw->tableName( 'links' );
-               $brokenlinksTable = $dbw->tableName( 'brokenlinks' );
-
-               $dbw->insertSelect( 'archive', 'cur',
-                       array(
-                               'ar_namespace' => 'cur_namespace',
-                               'ar_title' => 'cur_title',
-                               'ar_text' => 'cur_text',
-                               'ar_comment' => 'cur_comment',
-                               'ar_user' => 'cur_user',
-                               'ar_user_text' => 'cur_user_text',
-                               'ar_timestamp' => 'cur_timestamp',
-                               'ar_minor_edit' => 'cur_minor_edit',
-                               'ar_flags' => 0,
-                       ), array(
-                               'cur_namespace' => $ns,
-                               'cur_title' => $t,
-                       ), $fname
-               );
 
-               $dbw->insertSelect( 'archive', 'old',
+               $dbw->insertSelect( 'archive', array( 'page','revision', 'text' ),
                        array(
-                               'ar_namespace' => 'old_namespace',
-                               'ar_title' => 'old_title',
+                               'ar_namespace' => 'page_namespace',
+                               'ar_title' => 'page_title',
                                'ar_text' => 'old_text',
-                               'ar_comment' => 'old_comment',
-                               'ar_user' => 'old_user',
-                               'ar_user_text' => 'old_user_text',
-                               'ar_timestamp' => 'old_timestamp',
-                               'ar_minor_edit' => 'old_minor_edit',
-                               'ar_flags' => 'old_flags'
+                               'ar_comment' => 'rev_comment',
+                               'ar_user' => 'rev_user',
+                               'ar_user_text' => 'rev_user_text',
+                               'ar_timestamp' => 'rev_timestamp',
+                               'ar_minor_edit' => 'rev_minor_edit',
+                               'ar_flags' => 0,
                        ), array(
-                               'old_namespace' => $ns,
-                               'old_title' => $t,
+                               'page_namespace' => $ns,
+                               'page_title' => $t,
+                               'page_id = rev_page AND old_id = rev_id'
                        ), $fname
                );
 
                # Now that it's safely backed up, delete it
 
-               $dbw->delete( 'cur', array( 'cur_namespace' => $ns, 'cur_title' => $t ), $fname );
-               $dbw->delete( 'old', array( 'old_namespace' => $ns, 'old_title' => $t ), $fname );
+               $dbw->deleteJoin( 'text', 'revision', 'old_id', 'rev_id', array( "rev_page = {$id}" ), $fname );
+               $dbw->delete( 'revision', array( 'rev_page' => $id ), $fname );
+               $dbw->delete( 'page', array( 'page_id' => $id ), $fname);
                $dbw->delete( 'recentchanges', array( 'rc_namespace' => $ns, 'rc_title' => $t ), $fname );
 
                # Finally, clean up the link tables
@@ -1601,11 +1658,14 @@ class Article {
                return true;
        }
 
+       /**
+        * Revert a modification
+        */
        function rollback() {
-               global $wgUser, $wgLang, $wgOut, $wgRequest;
+               global $wgUser, $wgOut, $wgRequest;
                $fname = 'Article::rollback';
 
-               if ( ! $wgUser->isSysop() ) {
+               if ( ! $wgUser->isAllowed('rollback') ) {
                        $wgOut->sysopRequired();
                        return;
                }
@@ -1613,6 +1673,13 @@ class Article {
                        $wgOut->readOnlyPage( $this->getContent( true ) );
                        return;
                }
+               if( !$wgUser->matchEditToken( $wgRequest->getVal( 'token' ),
+                       array( $this->mTitle->getPrefixedText(),
+                               $wgRequest->getVal( 'from' ) )  ) ) {
+                       $wgOut->setPageTitle( wfMsg( 'rollbackfailed' ) );
+                       $wgOut->addWikiText( wfMsg( 'sessionfailure' ) );
+                       return;
+               }
                $dbw =& wfGetDB( DB_MASTER );
 
                # Enhanced rollback, marks edits rc_bot=1
@@ -1623,9 +1690,9 @@ class Article {
                $n = $this->mTitle->getNamespace();
 
                # Get the last editor, lock table exclusively
-               $s = $dbw->getArray( 'cur',
-                       array( 'cur_id','cur_user','cur_user_text','cur_comment' ),
-                       array( 'cur_title' => $tt, 'cur_namespace' => $n ),
+               $s = $dbw->selectRow( array( 'page', 'revision' ),
+                       array( 'page_id','rev_user','rev_user_text','rev_comment' ),
+                       array( 'page_title' => $tt, 'page_namespace' => $n ),
                        $fname, 'FOR UPDATE'
                );
                if( $s === false ) {
@@ -1634,26 +1701,26 @@ class Article {
                        return;
                }
                $ut = $dbw->strencode( $s->cur_user_text );
-               $uid = $s->cur_user;
-               $pid = $s->cur_id;
+               $uid = $s->rev_user;
+               $pid = $s->page_id;
 
                $from = str_replace( '_', ' ', $wgRequest->getVal( 'from' ) );
-               if( $from != $s->cur_user_text ) {
+               if( $from != $s->rev_user_text ) {
                        $wgOut->setPageTitle(wfmsg('rollbackfailed'));
                        $wgOut->addWikiText( wfMsg( 'alreadyrolled',
                                htmlspecialchars( $this->mTitle->getPrefixedText()),
                                htmlspecialchars( $from ),
-                               htmlspecialchars( $s->cur_user_text ) ) );
-                       if($s->cur_comment != '') {
+                               htmlspecialchars( $s->rev_user_text ) ) );
+                       if($s->rev_comment != '') {
                                $wgOut->addHTML(
                                        wfMsg('editcomment',
-                                       htmlspecialchars( $s->cur_comment ) ) );
+                                       htmlspecialchars( $s->rev_comment ) ) );
                        }
                        return;
                }
 
                # Get the last edit not by this guy
-               $s = $dbw->getArray( 'old',
+               $s = $dbw->selectRow( 'old',
                        array( 'old_text','old_user','old_user_text','old_timestamp','old_flags' ),
                        array(
                                'old_namespace' => $n,
@@ -1670,7 +1737,7 @@ class Article {
 
                if ( $bot ) {
                        # Mark all reverted edits as bot
-                       $dbw->updateArray( 'recentchanges',
+                       $dbw->update( 'recentchanges',
                                array( /* SET */
                                        'rc_bot' => 1
                                ), array( /* WHERE */
@@ -1684,17 +1751,20 @@ class Article {
                $newcomment = wfMsg( 'revertpage', $s->old_user_text, $from );
                $wgOut->setPagetitle( wfMsg( 'actioncomplete' ) );
                $wgOut->setRobotpolicy( 'noindex,nofollow' );
-               $wgOut->addHTML( '<h2>' . $newcomment . "</h2>\n<hr />\n" );
-               $this->updateArticle( Article::getRevisionText( $s ), $newcomment, 1, $this->mTitle->userIsWatching(), $bot );
+               $wgOut->addHTML( '<h2>' . htmlspecialchars( $newcomment ) . "</h2>\n<hr />\n" );
+               $this->updateArticle( Revision::getRevisionText( $s ), $newcomment, 1, $this->mTitle->userIsWatching(), $bot );
                Article::onArticleEdit( $this->mTitle );
                $wgOut->returnToMain( false );
        }
 
 
-       # Do standard deferred updates after page view
-
-       /* private */ function viewUpdates() {
+       /**
+        * Do standard deferred updates after page view
+        * @private
+        */
+       function viewUpdates() {
                global $wgDeferredUpdateList;
+               
                if ( 0 != $this->getID() ) {
                        global $wgDisableCounters;
                        if( !$wgDisableCounters ) {
@@ -1703,23 +1773,36 @@ class Article {
                                array_push( $wgDeferredUpdateList, $u );
                        }
                }
-               $u = new UserTalkUpdate( 0, $this->mTitle->getNamespace(),
-                 $this->mTitle->getDBkey() );
-               array_push( $wgDeferredUpdateList, $u );
-       }
+               
+               # Update newtalk status if user is reading their own
+               # talk page
 
-       # Do standard deferred updates after page edit.
-       # Every 1000th edit, prune the recent changes table.
+               global $wgUser;
+               if ($this->mTitle->getNamespace() == NS_USER_TALK &&
+                       $this->mTitle->getText() == $wgUser->getName()) {
+                       require_once( 'UserTalkUpdate.php' );
+                       $u = new UserTalkUpdate( 0, $this->mTitle->getNamespace(), $this->mTitle->getDBkey(), false, false, false );
+               }
+       }
 
-       /* private */ function editUpdates( $text ) {
+       /**
+        * Do standard deferred updates after page edit.
+        * Every 1000th edit, prune the recent changes table.
+        * @private
+        * @param string $text
+        */
+       function editUpdates( $text, $summary, $minoredit, $timestamp_of_pagechange) {
                global $wgDeferredUpdateList, $wgDBname, $wgMemc;
-               global $wgMessageCache;
+               global $wgMessageCache, $wgUser;
 
                wfSeedRandom();
                if ( 0 == mt_rand( 0, 999 ) ) {
+                       # Periodically flush old entries from the recentchanges table.
+                       global $wgRCMaxAge;
                        $dbw =& wfGetDB( DB_MASTER );
-                       $cutoff = wfUnix2Timestamp( time() - ( 7 * 86400 ) );
-                       $sql = "DELETE FROM recentchanges WHERE rc_timestamp < '{$cutoff}'";
+                       $cutoff = $dbw->timestamp( time() - $wgRCMaxAge );
+                       $recentchanges = $dbw->tableName( 'recentchanges' );
+                       $sql = "DELETE FROM $recentchanges WHERE rc_timestamp < '{$cutoff}'";
                        $dbw->query( $sql );
                }
                $id = $this->getID();
@@ -1736,8 +1819,14 @@ class Article {
                        $u = new SearchUpdate( $id, $title, $text );
                        array_push( $wgDeferredUpdateList, $u );
 
-                       $u = new UserTalkUpdate( 1, $this->mTitle->getNamespace(), $shortTitle );
-                       array_push( $wgDeferredUpdateList, $u );
+                       # If this is another user's talk page,
+                       # create a watchlist entry for this page
+                       
+                       if ($this->mTitle->getNamespace() == NS_USER_TALK &&
+                               $shortTitle != $wgUser->getName()) {
+                               require_once( 'UserTalkUpdate.php' );
+                               $u = new UserTalkUpdate( 1, $this->mTitle->getNamespace(), $shortTitle, $summary, $minoredit, $timestamp_of_pagechange);
+                       }
 
                        if ( $this->mTitle->getNamespace() == NS_MEDIAWIKI ) {
                                $wgMessageCache->replace( $shortTitle, $text );
@@ -1745,17 +1834,29 @@ class Article {
                }
        }
 
-       /* private */ function setOldSubtitle() {
-               global $wgLang, $wgOut;
+       /**
+        * @todo document this function
+        * @private
+        * @param string $oldid         Revision ID of this article revision
+        */
+       function setOldSubtitle( $oldid=0 ) {
+               global $wgLang, $wgOut, $wgUser;
 
                $td = $wgLang->timeanddate( $this->mTimestamp, true );
-               $r = wfMsg( 'revisionasof', $td );
-               $wgOut->setSubtitle( "({$r})" );
+               $sk = $wgUser->getSkin();
+               $lnk = $sk->makeKnownLinkObj ( $this->mTitle, wfMsg( 'currentrevisionlink' ) );
+               $prevlink = $sk->makeKnownLinkObj( $this->mTitle, wfMsg( 'previousrevision' ), 'direction=prev&oldid='.$oldid );
+               $nextlink = $sk->makeKnownLinkObj( $this->mTitle, wfMsg( 'nextrevision' ), 'direction=next&oldid='.$oldid );
+               $r = wfMsg( 'revisionasofwithlink', $td, $lnk, $prevlink, $nextlink );
+               $wgOut->setSubtitle( $r );
        }
 
-       # This function is called right before saving the wikitext,
-       # so we can do things like signatures and links-in-context.
-
+       /**
+        * This function is called right before saving the wikitext,
+        * so we can do things like signatures and links-in-context.
+        *
+        * @param string $text
+        */
        function preSaveTransform( $text ) {
                global $wgParser, $wgUser;
                return $wgParser->preSaveTransform( $text, $this->mTitle, $wgUser, ParserOptions::newFromUser( $wgUser ) );
@@ -1763,9 +1864,11 @@ class Article {
 
        /* Caching functions */
 
-       # checkLastModified returns true if it has taken care of all
-       # output to the client that is necessary for this request.
-       # (that is, it has sent a cached version of the page)
+       /**
+        * checkLastModified returns true if it has taken care of all
+        * output to the client that is necessary for this request.
+        * (that is, it has sent a cached version of the page)
+        */
        function tryFileCache() {
                static $called = false;
                if( $called ) {
@@ -1775,11 +1878,6 @@ class Article {
                $called = true;
                if($this->isFileCacheable()) {
                        $touched = $this->mTouched;
-                       if( $this->mTitle->getPrefixedDBkey() == wfMsg( 'mainpage' ) ) {
-                               # Expire the main page quicker
-                               $expire = wfUnix2Timestamp( time() - 3600 );
-                               $touched = max( $expire, $touched );
-                       }
                        $cache = new CacheManager( $this->mTitle );
                        if($cache->isFileCacheGood( $touched )) {
                                global $wgOut;
@@ -1794,7 +1892,11 @@ class Article {
                        wfDebug( " tryFileCache() - not cacheable\n" );
                }
        }
-
+       
+       /**
+        * Check if the page can be cached
+        * @return bool
+        */
        function isFileCacheable() {
                global $wgUser, $wgUseFileCache, $wgShowIPinHeader, $wgRequest;
                extract( $wgRequest->getValues( 'action', 'oldid', 'diff', 'redirect', 'printable' ) );
@@ -1802,7 +1904,7 @@ class Article {
                return $wgUseFileCache
                        and (!$wgShowIPinHeader)
                        and ($this->getID() != 0)
-                       and ($wgUser->getId() == 0)
+                       and ($wgUser->isAnon())
                        and (!$wgUser->getNewtalk())
                        and ($this->mTitle->getNamespace() != NS_SPECIAL )
                        and (empty( $action ) || $action == 'view')
@@ -1813,92 +1915,90 @@ class Article {
                        and (!$this->mRedirectedFrom);
        }
 
-       # Loads cur_touched and returns a value indicating if it should be used
+       /**
+        * Loads cur_touched and returns a value indicating if it should be used
+        *
+        */
        function checkTouched() {
                $fname = 'Article::checkTouched';
                $id = $this->getID();
                $dbr =& $this->getDB();
-               $s = $dbr->getArray( 'cur', array( 'cur_touched', 'cur_is_redirect' ),
-                       array( 'cur_id' => $id ), $fname, $this->getSelectOptions() );
+               $s = $dbr->selectRow( 'page', array( 'page_touched', 'page_is_redirect' ),
+                       array( 'page_id' => $id ), $fname, $this->getSelectOptions() );
                if( $s !== false ) {
-                       $this->mTouched = wfTimestamp(TS_MW,$s->cur_touched);
-                       return !$s->cur_is_redirect;
+                       $this->mTouched = wfTimestamp( TS_MW, $s->page_touched );
+                       return !$s->page_is_redirect;
                } else {
                        return false;
                }
        }
 
-       # Edit an article without doing all that other stuff
+       /**
+        * Edit an article without doing all that other stuff
+        *
+        * @param string $text text submitted
+        * @param string $comment comment submitted
+        * @param integer $minor whereas it's a minor modification
+        */
        function quickEdit( $text, $comment = '', $minor = 0 ) {
-               global $wgUser, $wgMwRedir;
+               global $wgUser;
                $fname = 'Article::quickEdit';
+
+               #wfDebugDieBacktrace( "$fname called." );
+
                wfProfileIn( $fname );
 
                $dbw =& wfGetDB( DB_MASTER );
                $ns = $this->mTitle->getNamespace();
                $dbkey = $this->mTitle->getDBkey();
                $encDbKey = $dbw->strencode( $dbkey );
-               $timestamp = wfTimestampNow();
-
-               # Save to history
-               $dbw->insertSelect( 'old', 'cur',
-                       array(
-                               'old_namespace' => 'cur_namespace',
-                               'old_title' => 'cur_title',
-                               'old_text' => 'cur_text',
-                               'old_comment' => 'cur_comment',
-                               'old_user' => 'cur_user',
-                               'old_user_text' => 'cur_user_text',
-                               'old_timestamp' => 'cur_timestamp',
-                               'inverse_timestamp' => '99999999999999-cur_timestamp',
-                       ), array(
-                               'cur_namespace' => $ns,
-                               'cur_title' => $dbkey,
-                       ), $fname
-               );
-
-               # Use the affected row count to determine if the article is new
-               $numRows = $dbw->affectedRows();
-
-               # Make an array of fields to be inserted
-               $fields = array(
-                       'cur_text' => $text,
-                       'cur_timestamp' => $timestamp,
-                       'cur_user' => $wgUser->getID(),
-                       'cur_user_text' => $wgUser->getName(),
-                       'inverse_timestamp' => wfInvertTimestamp( $timestamp ),
-                       'cur_comment' => $comment,
-                       'cur_is_redirect' => $wgMwRedir->matchStart( $text ) ? 1 : 0,
-                       'cur_minor_edit' => intval($minor),
-                       'cur_touched' => $dbw->timestamp($timestamp),
-               );
-
-               if ( $numRows ) {
-                       # Update article
-                       $fields['cur_is_new'] = 0;
-                       $dbw->updateArray( 'cur', $fields, array( 'cur_namespace' => $ns, 'cur_title' => $dbkey ), $fname );
-               } else {
-                       # Insert new article
-                       $fields['cur_is_new'] = 1;
-                       $fields['cur_namespace'] = $ns;
-                       $fields['cur_title'] = $dbkey;
-                       $fields['cur_random'] = $rand = number_format( mt_rand() / mt_getrandmax(), 12, '.', '' );
-                       $dbw->insertArray( 'cur', $fields, $fname );
-               }
+               $timestamp = $dbw->timestamp();
+               # insert new text
+               $dbw->insert( 'text', array(
+                               'old_text' => $text,
+                               'old_flags' => "" ), $fname );
+               $text_id = $dbw->insertID();
+
+               # update page
+               $dbw->update( 'page', array(
+                       'page_is_new' => 0,
+                       'page_touched' => $timestamp,
+                       'page_is_redirect' => $this->isRedirect( $text ) ? 1 : 0,
+                       'page_latest' => $text_id ),
+                       array( 'page_namespace' => $ns, 'page_title' => $dbkey ), $fname );
+               # Retrieve page ID
+               $page_id = $dbw->selectField( 'page', 'page_id', array( 'page_namespace' => $ns, 'page_title' => $dbkey ), $fname );
+
+               # update revision
+               $dbw->insert( 'revision', array(
+                       'rev_id' => $text_id,
+                       'rev_page' => $page_id,
+                       'rev_comment' => $comment,
+                       'rev_user' => $wgUser->getID(),
+                       'rev_user_text' => $wgUser->getName(),
+                       'rev_timestamp' => $timestamp,
+                       'rev_minor_edit' => intval($minor) ),
+                       $fname );
                wfProfileOut( $fname );
        }
 
-       /* static */ function incViewCount( $id ) {
+       /**
+        * Used to increment the view counter
+        *
+        * @static
+        * @param integer $id article id
+        */
+       function incViewCount( $id ) {
                $id = intval( $id );
                global $wgHitcounterUpdateFreq;
 
                $dbw =& wfGetDB( DB_MASTER );
-               $curTable = $dbw->tableName( 'cur' );
+               $pageTable = $dbw->tableName( 'page' );
                $hitcounterTable = $dbw->tableName( 'hitcounter' );
                $acchitsTable = $dbw->tableName( 'acchits' );
 
                if( $wgHitcounterUpdateFreq <= 1 ){ //
-                       $dbw->query( "UPDATE $curTable SET cur_counter = cur_counter + 1 WHERE cur_id = $id" );
+                       $dbw->query( "UPDATE $pageTable SET page_counter = page_counter + 1 WHERE page_id = $id" );
                        return;
                }
 
@@ -1937,15 +2037,20 @@ class Article {
                $dbw->ignoreErrors( $oldignore );
        }
 
-       # The onArticle*() functions are supposed to be a kind of hooks
-       # which should be called whenever any of the specified actions
-       # are done.
-       #
-       # This is a good place to put code to clear caches, for instance.
-
-       # This is called on page move and undelete, as well as edit
-       /* static */ function onArticleCreate($title_obj) {
-               global $wgUseSquid, $wgDeferredUpdateList;
+       /**#@+
+        * The onArticle*() functions are supposed to be a kind of hooks
+        * which should be called whenever any of the specified actions
+        * are done.
+        * 
+        * This is a good place to put code to clear caches, for instance.
+        * 
+        * This is called on page move and undelete, as well as edit
+        * @static
+        * @param $title_obj a title object
+        */
+
+       function onArticleCreate($title_obj) {
+               global $wgUseSquid, $wgPostCommitUpdateList;
 
                $titles = $title_obj->getBrokenLinksTo();
 
@@ -1956,26 +2061,29 @@ class Article {
                                $urls[] = $linkTitle->getInternalURL();
                        }
                        $u = new SquidUpdate( $urls );
-                       array_push( $wgDeferredUpdateList, $u );
+                       array_push( $wgPostCommitUpdateList, $u );
                }
 
                # Clear persistent link cache
                LinkCache::linksccClearBrokenLinksTo( $title_obj->getPrefixedDBkey() );
        }
 
-       /* static */ function onArticleDelete($title_obj) {
+       function onArticleDelete($title_obj) {
                LinkCache::linksccClearLinksTo( $title_obj->getArticleID() );
        }
-
-       /* static */ function onArticleEdit($title_obj) {
+       function onArticleEdit($title_obj) {
                LinkCache::linksccClearPage( $title_obj->getArticleID() );
        }
-
-
-       # Info about this page
-
+       /**#@-*/
+
+       /**
+        * Info about this page
+        * Called for ?action=info when $wgAllowPageInfo is on.
+        *
+        * @access public
+        */
        function info() {
-               global $wgUser, $wgTitle, $wgOut, $wgLang, $wgAllowPageInfo;
+               global $wgLang, $wgOut, $wgAllowPageInfo;
                $fname = 'Article::info';
 
                if ( !$wgAllowPageInfo ) {
@@ -1983,67 +2091,107 @@ class Article {
                        return;
                }
 
-               $dbr =& $this->getDB();
-
-               $basenamespace = $wgTitle->getNamespace() & (~1);
-               $cur_clause = array( 'cur_title' => $wgTitle->getDBkey(), 'cur_namespace' => $basenamespace );
-               $old_clause = array( 'old_title' => $wgTitle->getDBkey(), 'old_namespace' => $basenamespace );
-               $wl_clause  = array( 'wl_title' => $wgTitle->getDBkey(), 'wl_namespace' => $basenamespace );
-               $fullTitle = $wgTitle->makeName($basenamespace, $wgTitle->getDBKey());
-               $wgOut->setPagetitle(  $fullTitle );
+               $page = $this->mTitle->getSubjectPage();
+               
+               $wgOut->setPagetitle( $page->getPrefixedText() );
                $wgOut->setSubtitle( wfMsg( 'infosubtitle' ));
 
                # first, see if the page exists at all.
-               $exists = $dbr->selectField( 'cur', 'COUNT(*)', $cur_clause, $fname, $this->getSelectOptions() );
-               if ($exists < 1) {
+               $exists = $page->getArticleId() != 0;
+               if( !$exists ) {
                        $wgOut->addHTML( wfMsg('noarticletext') );
                } else {
-                       $numwatchers = $dbr->selectField( 'watchlist', 'COUNT(*)', $wl_clause, $fname, 
-                               $this->getSelectOptions() );
-                       $wgOut->addHTML( "<ul><li>" . wfMsg("numwatchers", $numwatchers) . '</li>' );
-                       $old = $dbr->selectField( 'old', 'COUNT(*)', $old_clause, $fname, $this->getSelectOptions() );
-                       $wgOut->addHTML( "<li>" . wfMsg('numedits', $old + 1) . '</li>');
-
-                       # to find number of distinct authors, we need to do some
-                       # funny stuff because of the cur/old table split:
-                       # - first, find the name of the 'cur' author
-                       # - then, find the number of *other* authors in 'old'
-
-                       # find 'cur' author
-                       $cur_author = $dbr->selectField( 'cur', 'cur_user_text', $cur_clause, $fname, 
+                       $dbr =& $this->getDB( DB_SLAVE );       
+                       $wl_clause = array(
+                               'wl_title'     => $page->getDBkey(),
+                               'wl_namespace' => $page->getNamespace() );
+                       $numwatchers = $dbr->selectField(
+                               'watchlist',
+                               'COUNT(*)',
+                               $wl_clause,
+                               $fname,
                                $this->getSelectOptions() );
 
-                       # find number of 'old' authors excluding 'cur' author
-                       $authors = $dbr->selectField( 'old', 'COUNT(DISTINCT old_user_text)',
-                               $old_clause + array( 'old_user_text<>' . $dbr->addQuotes( $cur_author ) ), $fname,
-                               $this->getSelectOptions() ) + 1;
-
-                       # now for the Talk page ...
-                       $cur_clause = array( 'cur_title' => $wgTitle->getDBkey(), 'cur_namespace' => $basenamespace+1 );
-                       $old_clause = array( 'old_title' => $wgTitle->getDBkey(), 'old_namespace' => $basenamespace+1 );
-
-                       # does it exist?
-                       $exists = $dbr->selectField( 'cur', 'COUNT(*)', $cur_clause, $fname, $this->getSelectOptions() );
-
-                       # number of edits
-                       if ($exists > 0) {
-                               $old = $dbr->selectField( 'old', 'COUNT(*)', $old_clause, $fname, $this->getSelectOptions() );
-                               $wgOut->addHTML( '<li>' . wfMsg("numtalkedits", $old + 1) . '</li>');
+                       $pageInfo = $this->pageCountInfo( $page );
+                       $talkInfo = $this->pageCountInfo( $page->getTalkPage() );
+                       
+                       $wgOut->addHTML( "<ul><li>" . wfMsg("numwatchers", $wgLang->formatNum( $numwatchers ) ) . '</li>' );
+                       $wgOut->addHTML( "<li>" . wfMsg('numedits', $wgLang->formatNum( $pageInfo['edits'] ) ) . '</li>');
+                       if( $talkInfo ) {
+                               $wgOut->addHTML( '<li>' . wfMsg("numtalkedits", $wgLang->formatNum( $talkInfo['edits'] ) ) . '</li>');
                        }
-                       $wgOut->addHTML( '<li>' . wfMsg("numauthors", $authors) . '</li>' );
+                       $wgOut->addHTML( '<li>' . wfMsg("numauthors", $wgLang->formatNum( $pageInfo['authors'] ) ) . '</li>' );
+                       if( $talkInfo ) {
+                               $wgOut->addHTML( '<li>' . wfMsg('numtalkauthors', $wgLang->formatNum( $talkInfo['authors'] ) ) . '</li>' );
+                       }
+                       $wgOut->addHTML( '</ul>' );
 
-                       # number of authors
-                       if ($exists > 0) {
-                               $cur_author = $dbr->selectField( 'cur', 'cur_user_text', $cur_clause, $fname,
-                                       $this->getSelectOptions() );
-                               $authors = $dbr->selectField( 'cur', 'COUNT(DISTINCT old_user_text)',
-                                       $old_clause + array( 'old_user_text<>' . $dbr->addQuotes( $cur_author ) ), 
-                                       $fname, $this->getSelectOptions() );
+               }
+       }
+       
+       /**
+        * Return the total number of edits and number of unique editors
+        * on a given page. If page does not exist, returns false.
+        *
+        * @param Title $title
+        * @return array
+        * @access private
+        */
+       function pageCountInfo( $title ) {
+               $id = $title->getArticleId();
+               if( $id == 0 ) {
+                       return false;
+               }
+               
+               $dbr =& $this->getDB( DB_SLAVE );       
+
+               $rev_clause = array( 'rev_page' => $id );
+               $fname = 'Article::pageCountInfo';
+
+               $edits = $dbr->selectField(
+                       'revision',
+                       'COUNT(rev_page)',
+                       $rev_clause,
+                       $fname,
+                       $this->getSelectOptions() );
+
+               $authors = $dbr->selectField(
+                       'revision',
+                       'COUNT(DISTINCT rev_user_text)',
+                       $rev_clause,
+                       $fname,
+                       $this->getSelectOptions() );
+               
+               return array( 'edits' => $edits, 'authors' => $authors );
+       }
+}
 
-                               $wgOut->addHTML( '<li>' . wfMsg('numtalkauthors', $authors) . '</li></ul>' );
-                       }
+/**
+ * Check whether an article is a stub
+ *
+ * @public
+ * @param integer $articleID   ID of the article that is to be checked
+ */
+function wfArticleIsStub( $articleID ) {
+       global $wgUser;
+       $fname = 'wfArticleIsStub';
+
+       wfDebugDieBacktrace( 'This function seems to be unused. Pending removal.' );
+
+       $threshold = $wgUser->getOption('stubthreshold') ;
+       if ( $threshold > 0 ) {
+               $dbr =& wfGetDB( DB_SLAVE );
+               $s = $dbr->selectRow( array('page', 'text'),
+                       array( 'LENGTH(old_text) AS len', 'page_namespace', 'page_is_redirect' ),
+                       array( 'page_id' => $articleID, "page.page_latest=text.old_id" ),
+                       $fname ) ;
+               if ( $s == false OR $s->page_is_redirect OR $s->page_namespace != NS_MAIN ) {
+                       return false;
                }
+               $size = $s->len;
+               return ( $size < $threshold );
        }
+       return false;
 }
 
 ?>