Spaces to tabs
[lhc/web/wiklou.git] / includes / Title.php
index d0fee24..106eabe 100644 (file)
@@ -4,30 +4,34 @@
  * @file
  */
 
+/**
+ * @todo:  determine if it is really necessary to load this.  Appears to be left over from pre-autoloader versions, and
+ *   is only really needed to provide access to constant UTF8_REPLACEMENT, which actually resides in UtfNormalDefines.php
+ *   and is loaded by UtfNormalUtil.php, which is loaded by UtfNormal.php.
+ */
 if ( !class_exists( 'UtfNormal' ) ) {
-       require_once( dirname(__FILE__) . '/normal/UtfNormal.php' );
+       require_once( dirname( __FILE__ ) . '/normal/UtfNormal.php' );
 }
 
-define ( 'GAID_FOR_UPDATE', 1 );
-
-
 /**
- * Constants for pr_cascade bitfield
+ * @deprecated This used to be a define, but was moved to
+ * Title::GAID_FOR_UPDATE in 1.17. This will probably be removed in 1.18
  */
-define( 'CASCADE', 1 );
+define( 'GAID_FOR_UPDATE', Title::GAID_FOR_UPDATE );
 
 /**
  * Represents a title within MediaWiki.
  * Optionally may contain an interwiki designation or namespace.
  * @note This class can fetch various kinds of data from the database;
  *       however, it does so inefficiently.
+ *
+ * @internal documentation reviewed 15 Mar 2010
  */
 class Title {
        /** @name Static cache variables */
-       //@{
-       static private $titleCache=array();
-       static private $interwikiCache=array();
-       //@}
+       // @{
+       static private $titleCache = array();
+       // @}
 
        /**
         * Title::newFromText maintains a cache to avoid expensive re-normalization of
@@ -36,79 +40,89 @@ class Title {
         */
        const CACHE_MAX = 1000;
 
+       /**
+        * Used to be GAID_FOR_UPDATE define. Used with getArticleId() and friends
+        * to use the master DB
+        */
+       const GAID_FOR_UPDATE = 1;
+
 
        /**
         * @name Private member variables
         * Please use the accessor functions instead.
         * @private
         */
-       //@{
-
-       var $mTextform = '';              ///< Text form (spaces not underscores) of the main part
-       var $mUrlform = '';               ///< URL-encoded form of the main part
-       var $mDbkeyform = '';             ///< Main part with underscores
-       var $mUserCaseDBKey;              ///< DB key with the initial letter in the case specified by the user
-       var $mNamespace = NS_MAIN;        ///< Namespace index, i.e. one of the NS_xxxx constants
-       var $mInterwiki = '';             ///< Interwiki prefix (or null string)
-       var $mFragment;                   ///< Title fragment (i.e. the bit after the #)
-       var $mArticleID = -1;             ///< Article ID, fetched from the link cache on demand
-       var $mLatestID = false;           ///< ID of most recent revision
-       var $mRestrictions = array();     ///< Array of groups allowed to edit this article
+       // @{
+
+       var $mTextform = '';              // /< Text form (spaces not underscores) of the main part
+       var $mUrlform = '';               // /< URL-encoded form of the main part
+       var $mDbkeyform = '';             // /< Main part with underscores
+       var $mUserCaseDBKey;              // /< DB key with the initial letter in the case specified by the user
+       var $mNamespace = NS_MAIN;        // /< Namespace index, i.e. one of the NS_xxxx constants
+       var $mInterwiki = '';             // /< Interwiki prefix (or null string)
+       var $mFragment;                   // /< Title fragment (i.e. the bit after the #)
+       var $mArticleID = -1;             // /< Article ID, fetched from the link cache on demand
+       var $mLatestID = false;           // /< ID of most recent revision
+       var $mRestrictions = array();     // /< Array of groups allowed to edit this article
        var $mOldRestrictions = false;
        var $mCascadeRestriction;         ///< Cascade restrictions on this page to included templates and images?
+       var $mCascadingRestrictions;      // Caching the results of getCascadeProtectionSources
        var $mRestrictionsExpiry = array(); ///< When do the restrictions on this page expire?
        var $mHasCascadingRestrictions;   ///< Are cascading restrictions in effect on this page?
        var $mCascadeSources;             ///< Where are the cascading restrictions coming from on this page?
        var $mRestrictionsLoaded = false; ///< Boolean for initialisation on demand
        var $mPrefixedText;               ///< Text form including namespace/interwiki, initialised on demand
+       var $mTitleProtection;            ///< Cached value for getTitleProtection (create protection)
        # Don't change the following default, NS_MAIN is hardcoded in several
        # places.  See bug 696.
-       var $mDefaultNamespace = NS_MAIN; ///< Namespace index when there is no namespace
+       var $mDefaultNamespace = NS_MAIN; // /< Namespace index when there is no namespace
                                          # Zero except in {{transclusion}} tags
-       var $mWatched = null;             ///< Is $wgUser watching this page? null if unfilled, accessed through userIsWatching()
-       var $mLength = -1;                ///< The page length, 0 for special pages
-       var $mRedirect = null;            ///< Is the article at this title a redirect?
-       var $mNotificationTimestamp = array(); ///< Associative array of user ID -> timestamp/false
-       var $mBacklinkCache = null;       ///< Cache of links to this title
-       //@}
+       var $mWatched = null;             // /< Is $wgUser watching this page? null if unfilled, accessed through userIsWatching()
+       var $mLength = -1;                // /< The page length, 0 for special pages
+       var $mRedirect = null;            // /< Is the article at this title a redirect?
+       var $mNotificationTimestamp = array(); // /< Associative array of user ID -> timestamp/false
+       var $mBacklinkCache = null;       // /< Cache of links to this title
+       // @}
 
 
        /**
         * Constructor
         * @private
         */
-       /* private */ function __construct() {}
+       /* private */ function __construct() { }
 
        /**
         * Create a new Title from a prefixed DB key
-        * @param $key \type{\string} The database key, which has underscores
+        *
+        * @param $key String The database key, which has underscores
         *      instead of spaces, possibly including namespace and
         *      interwiki prefixes
-        * @return \type{Title} the new object, or NULL on an error
+        * @return Title the new object, or NULL on an error
         */
        public static function newFromDBkey( $key ) {
                $t = new Title();
                $t->mDbkeyform = $key;
-               if( $t->secureAndSplit() )
+               if ( $t->secureAndSplit() ) {
                        return $t;
-               else
+               } else {
                        return null;
+               }
        }
 
        /**
         * Create a new Title from text, such as what one would find in a link. De-
         * codes any HTML entities in the text.
         *
-        * @param $text             string  The link text; spaces, prefixes, and an
+        * @param $text String  The link text; spaces, prefixes, and an
         *   initial ':' indicating the main namespace are accepted.
-        * @param $defaultNamespace int     The namespace to use if none is speci-
+        * @param $defaultNamespace int The namespace to use if none is speci-
         *   fied by a prefix.  If you want to force a specific namespace even if
         *   $text might begin with a namespace prefix, use makeTitle() or
         *   makeTitleSafe().
         * @return Title  The new object, or null on an error.
         */
        public static function newFromText( $text, $defaultNamespace = NS_MAIN ) {
-               if( is_object( $text ) ) {
+               if ( is_object( $text ) ) {
                        throw new MWException( 'Title::newFromText given an object' );
                }
 
@@ -120,26 +134,26 @@ class Title {
                 *
                 * In theory these are value objects and won't get changed...
                 */
-               if( $defaultNamespace == NS_MAIN && isset( Title::$titleCache[$text] ) ) {
+               if ( $defaultNamespace == NS_MAIN && isset( Title::$titleCache[$text] ) ) {
                        return Title::$titleCache[$text];
                }
 
                /**
-                * Convert things like &eacute; &#257; or &#x3017; into real text...
+                * Convert things like &eacute; &#257; or &#x3017; into normalized (bug 14952) text
                 */
-               $filteredText = Sanitizer::decodeCharReferences( $text );
+               $filteredText = Sanitizer::decodeCharReferencesAndNormalize( $text );
 
                $t = new Title();
                $t->mDbkeyform = str_replace( ' ', '_', $filteredText );
                $t->mDefaultNamespace = $defaultNamespace;
 
                static $cachedcount = 0 ;
-               if( $t->secureAndSplit() ) {
-                       if( $defaultNamespace == NS_MAIN ) {
-                               if( $cachedcount >= self::CACHE_MAX ) {
+               if ( $t->secureAndSplit() ) {
+                       if ( $defaultNamespace == NS_MAIN ) {
+                               if ( $cachedcount >= self::CACHE_MAX ) {
                                        # Avoid memory leaks on mass operations...
                                        Title::$titleCache = array();
-                                       $cachedcount=0;
+                                       $cachedcount = 0;
                                }
                                $cachedcount++;
                                Title::$titleCache[$text] =& $t;
@@ -162,8 +176,9 @@ class Title {
         *
         * Create a new Title from URL-encoded text. Ensures that
         * the given title's length does not exceed the maximum.
-        * @param $url \type{\string} the title, as might be taken from a URL
-        * @return \type{Title} the new object, or NULL on an error
+        *
+        * @param $url String the title, as might be taken from a URL
+        * @return Title the new object, or NULL on an error
         */
        public static function newFromURL( $url ) {
                global $wgLegalTitleChars;
@@ -177,7 +192,7 @@ class Title {
                }
 
                $t->mDbkeyform = str_replace( ' ', '_', $url );
-               if( $t->secureAndSplit() ) {
+               if ( $t->secureAndSplit() ) {
                        return $t;
                } else {
                        return null;
@@ -187,14 +202,14 @@ class Title {
        /**
         * Create a new Title from an article ID
         *
-        * @param $id \type{\int} the page_id corresponding to the Title to create
-        * @param $flags \type{\int} use GAID_FOR_UPDATE to use master
-        * @return \type{Title} the new object, or NULL on an error
+        * @param $id Int the page_id corresponding to the Title to create
+        * @param $flags Int use Title::GAID_FOR_UPDATE to use master
+        * @return Title the new object, or NULL on an error
         */
        public static function newFromID( $id, $flags = 0 ) {
-               $db = ($flags & GAID_FOR_UPDATE) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
+               $db = ( $flags & self::GAID_FOR_UPDATE ) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
                $row = $db->selectRow( 'page', '*', array( 'page_id' => $id ), __METHOD__ );
-               if( $row !== false ) {
+               if ( $row !== false ) {
                        $title = Title::newFromRow( $row );
                } else {
                        $title = null;
@@ -204,36 +219,46 @@ class Title {
 
        /**
         * Make an array of titles from an array of IDs
-        * @param $ids \type{\arrayof{\int}} Array of IDs
-        * @return \type{\arrayof{Title}} Array of Titles
+        *
+        * @param $ids Array of Int Array of IDs
+        * @return Array of Titles
         */
        public static function newFromIDs( $ids ) {
                if ( !count( $ids ) ) {
                        return array();
                }
                $dbr = wfGetDB( DB_SLAVE );
-               $res = $dbr->select( 'page', array( 'page_namespace', 'page_title' ),
-                       'page_id IN (' . $dbr->makeList( $ids ) . ')', __METHOD__ );
+               
+               $res = $dbr->select(
+                       'page',
+                       array(
+                               'page_namespace', 'page_title', 'page_id',
+                               'page_len', 'page_is_redirect', 'page_latest',
+                       ),
+                       array( 'page_id' => $ids ),
+                       __METHOD__
+               );
 
                $titles = array();
-               foreach( $res as $row ) {
-                       $titles[] = Title::makeTitle( $row->page_namespace, $row->page_title );
+               foreach ( $res as $row ) {
+                       $titles[] = Title::newFromRow( $row );
                }
                return $titles;
        }
 
        /**
         * Make a Title object from a DB row
-        * @param $row \type{Row} (needs at least page_title,page_namespace)
-        * @return \type{Title} corresponding Title
+        *
+        * @param $row Object database row (needs at least page_title,page_namespace)
+        * @return Title corresponding Title
         */
        public static function newFromRow( $row ) {
                $t = self::makeTitle( $row->page_namespace, $row->page_title );
 
-               $t->mArticleID = isset($row->page_id) ? intval($row->page_id) : -1;
-               $t->mLength = isset($row->page_len) ? intval($row->page_len) : -1;
-               $t->mRedirect = isset($row->page_is_redirect) ? (bool)$row->page_is_redirect : null;
-               $t->mLatestID = isset($row->page_latest) ? $row->page_latest : false;
+               $t->mArticleID = isset( $row->page_id ) ? intval( $row->page_id ) : -1;
+               $t->mLength = isset( $row->page_len ) ? intval( $row->page_len ) : -1;
+               $t->mRedirect = isset( $row->page_is_redirect ) ? (bool)$row->page_is_redirect : null;
+               $t->mLatestID = isset( $row->page_latest ) ? intval( $row->page_latest ) : false;
 
                return $t;
        }
@@ -245,14 +270,15 @@ class Title {
         * For convenience, spaces are converted to underscores so that
         * eg user_text fields can be used directly.
         *
-        * @param $ns \type{\int} the namespace of the article
-        * @param $title \type{\string} the unprefixed database key form
-        * @param $fragment \type{\string} The link fragment (after the "#")
-        * @return \type{Title} the new object
+        * @param $ns Int the namespace of the article
+        * @param $title String the unprefixed database key form
+        * @param $fragment String The link fragment (after the "#")
+        * @param $interwiki String The interwiki prefix
+        * @return Title the new object
         */
-       public static function &makeTitle( $ns, $title, $fragment = '' ) {
+       public static function &makeTitle( $ns, $title, $fragment = '', $interwiki = '' ) {
                $t = new Title();
-               $t->mInterwiki = '';
+               $t->mInterwiki = $interwiki;
                $t->mFragment = $fragment;
                $t->mNamespace = $ns = intval( $ns );
                $t->mDbkeyform = str_replace( ' ', '_', $title );
@@ -267,15 +293,16 @@ class Title {
         * The parameters will be checked for validity, which is a bit slower
         * than makeTitle() but safer for user-provided data.
         *
-        * @param $ns \type{\int} the namespace of the article
-        * @param $title \type{\string} the database key form
-        * @param $fragment \type{\string} The link fragment (after the "#")
-        * @return \type{Title} the new object, or NULL on an error
+        * @param $ns Int the namespace of the article
+        * @param $title String the database key form
+        * @param $fragment String The link fragment (after the "#")
+        * @param $interwiki String The interwiki prefix
+        * @return Title the new object, or NULL on an error
         */
-       public static function makeTitleSafe( $ns, $title, $fragment = '' ) {
+       public static function makeTitleSafe( $ns, $title, $fragment = '', $interwiki = '' ) {
                $t = new Title();
-               $t->mDbkeyform = Title::makeName( $ns, $title, $fragment );
-               if( $t->secureAndSplit() ) {
+               $t->mDbkeyform = Title::makeName( $ns, $title, $fragment, $interwiki );
+               if ( $t->secureAndSplit() ) {
                        return $t;
                } else {
                        return null;
@@ -284,7 +311,8 @@ class Title {
 
        /**
         * Create a new Title for the Main Page
-        * @return \type{Title} the new object
+        *
+        * @return Title the new object
         */
        public static function newMainPage() {
                $title = Title::newFromText( wfMsgForContent( 'mainpage' ) );
@@ -301,8 +329,8 @@ class Title {
         * This will only return the very next target, useful for
         * the redirect table and other checks that don't need full recursion
         *
-        * @param $text \type{\string} Text with possible redirect
-        * @return \type{Title} The corresponding Title
+        * @param $text String: Text with possible redirect
+        * @return Title: The corresponding Title
         */
        public static function newFromRedirect( $text ) {
                return self::newFromRedirectInternal( $text );
@@ -314,8 +342,8 @@ class Title {
         * This will recurse down $wgMaxRedirects times or until a non-redirect target is hit
         * in order to provide (hopefully) the Title of the final destination instead of another redirect
         *
-        * @param $text \type{\string} Text with possible redirect
-        * @return \type{Title} The corresponding Title
+        * @param $text String Text with possible redirect
+        * @return Title The corresponding Title
         */
        public static function newFromRedirectRecurse( $text ) {
                $titles = self::newFromRedirectArray( $text );
@@ -328,29 +356,31 @@ class Title {
         * The last element in the array is the final destination after all redirects
         * have been resolved (up to $wgMaxRedirects times)
         *
-        * @param $text \type{\string} Text with possible redirect
-        * @return \type{\array} Array of Titles, with the destination last
+        * @param $text String Text with possible redirect
+        * @return Array of Titles, with the destination last
         */
        public static function newFromRedirectArray( $text ) {
                global $wgMaxRedirects;
                // are redirects disabled?
-               if( $wgMaxRedirects < 1 )
+               if ( $wgMaxRedirects < 1 ) {
                        return null;
+               }
                $title = self::newFromRedirectInternal( $text );
-               if( is_null( $title ) )
+               if ( is_null( $title ) ) {
                        return null;
+               }
                // recursive check to follow double redirects
                $recurse = $wgMaxRedirects;
                $titles = array( $title );
-               while( --$recurse > 0 ) {
-                       if( $title->isRedirect() ) {
+               while ( --$recurse > 0 ) {
+                       if ( $title->isRedirect() ) {
                                $article = new Article( $title, 0 );
                                $newtitle = $article->getRedirectTarget();
                        } else {
                                break;
                        }
                        // Redirects to some special pages are not permitted
-                       if( $newtitle instanceOf Title && $newtitle->isValidRedirectTarget() ) {
+                       if ( $newtitle instanceOf Title && $newtitle->isValidRedirectTarget() ) {
                                // the new title passes the checks, so make that our current title so that further recursion can be checked
                                $title = $newtitle;
                                $titles[] = $newtitle;
@@ -365,29 +395,27 @@ class Title {
         * Really extract the redirect destination
         * Do not call this function directly, use one of the newFromRedirect* functions above
         *
-        * @param $text \type{\string} Text with possible redirect
-        * @return \type{Title} The corresponding Title
+        * @param $text String Text with possible redirect
+        * @return Title The corresponding Title
         */
        protected static function newFromRedirectInternal( $text ) {
                $redir = MagicWord::get( 'redirect' );
-               $text = trim($text);
-               if( $redir->matchStartAndRemove( $text ) ) {
+               $text = trim( $text );
+               if ( $redir->matchStartAndRemove( $text ) ) {
                        // Extract the first link and see if it's usable
                        // Ensure that it really does come directly after #REDIRECT
                        // Some older redirects included a colon, so don't freak about that!
                        $m = array();
-                       if( preg_match( '!^\s*:?\s*\[{2}(.*?)(?:\|.*?)?\]{2}!', $text, $m ) ) {
+                       if ( preg_match( '!^\s*:?\s*\[{2}(.*?)(?:\|.*?)?\]{2}!', $text, $m ) ) {
                                // Strip preceding colon used to "escape" categories, etc.
                                // and URL-decode links
-                               if( strpos( $m[1], '%' ) !== false ) {
+                               if ( strpos( $m[1], '%' ) !== false ) {
                                        // Match behavior of inline link parsing here;
-                                       // don't interpret + as " " most of the time!
-                                       // It might be safe to just use rawurldecode instead, though.
-                                       $m[1] = urldecode( ltrim( $m[1], ':' ) );
+                                       $m[1] = rawurldecode( ltrim( $m[1], ':' ) );
                                }
                                $title = Title::newFromText( $m[1] );
                                // If the title is a redirect to bad special pages or is invalid, return null
-                               if( !$title instanceof Title || !$title->isValidRedirectTarget() ) {
+                               if ( !$title instanceof Title || !$title->isValidRedirectTarget() ) {
                                        return null;
                                }
                                return $title;
@@ -396,24 +424,29 @@ class Title {
                return null;
        }
 
-#----------------------------------------------------------------------------
+# ----------------------------------------------------------------------------
 #      Static functions
-#----------------------------------------------------------------------------
+# ----------------------------------------------------------------------------
 
        /**
         * Get the prefixed DB key associated with an ID
-        * @param $id \type{\int} the page_id of the article
-        * @return \type{Title} an object representing the article, or NULL
+        *
+        * @param $id Int the page_id of the article
+        * @return Title an object representing the article, or NULL
         *  if no such article was found
         */
        public static function nameOf( $id ) {
                $dbr = wfGetDB( DB_SLAVE );
 
-               $s = $dbr->selectRow( 'page',
-                       array( 'page_namespace','page_title' ),
+               $s = $dbr->selectRow(
+                       'page',
+                       array( 'page_namespace', 'page_title' ),
                        array( 'page_id' => $id ),
-                       __METHOD__ );
-               if ( $s === false ) { return null; }
+                       __METHOD__
+               );
+               if ( $s === false ) {
+                       return null;
+               }
 
                $n = self::makeName( $s->page_namespace, $s->page_title );
                return $n;
@@ -421,7 +454,8 @@ class Title {
 
        /**
         * Get a regex character class describing the legal characters in a link
-        * @return \type{\string} the list of characters, not delimited
+        *
+        * @return String the list of characters, not delimited
         */
        public static function legalChars() {
                global $wgLegalTitleChars;
@@ -432,16 +466,16 @@ class Title {
         * Get a string representation of a title suitable for
         * including in a search index
         *
-        * @param $ns \type{\int} a namespace index
-        * @param $title \type{\string} text-form main part
-        * @return \type{\string} a stripped-down title string ready for the
+        * @param $ns Int a namespace index
+        * @param $title String text-form main part
+        * @return String a stripped-down title string ready for the
         *  search index
         */
        public static function indexTitle( $ns, $title ) {
                global $wgContLang;
 
                $lc = SearchEngine::legalSearchChars() . '&#;';
-               $t = $wgContLang->stripForSearch( $title );
+               $t = $wgContLang->normalizeForSearch( $title );
                $t = preg_replace( "/[^{$lc}]+/", ' ', $t );
                $t = $wgContLang->lc( $t );
 
@@ -459,16 +493,21 @@ class Title {
 
        /**
         * Make a prefixed DB key from a DB key and a namespace index
-        * @param $ns \type{\int} numerical representation of the namespace
-        * @param $title \type{\string} the DB key form the title
-        * @param $fragment \type{\string} The link fragment (after the "#")
-        * @return \type{\string} the prefixed form of the title
+        *
+        * @param $ns Int numerical representation of the namespace
+        * @param $title String the DB key form the title
+        * @param $fragment String The link fragment (after the "#")
+        * @param $interwiki String The interwiki prefix
+        * @return String the prefixed form of the title
         */
-       public static function makeName( $ns, $title, $fragment = '' ) {
+       public static function makeName( $ns, $title, $fragment = '', $interwiki = '' ) {
                global $wgContLang;
 
                $namespace = $wgContLang->getNsText( $ns );
                $name = $namespace == '' ? $title : "$namespace:$title";
+               if ( strval( $interwiki ) != '' ) {
+                       $name = "$interwiki:$name";
+               }
                if ( strval( $fragment ) != '' ) {
                        $name .= '#' . $fragment;
                }
@@ -479,7 +518,7 @@ class Title {
         * Determine whether the object refers to a page within
         * this project.
         *
-        * @return \type{\bool} TRUE if this is an in-project interwiki link
+        * @return Bool TRUE if this is an in-project interwiki link
         *      or a wikilink, FALSE otherwise
         */
        public function isLocal() {
@@ -494,116 +533,154 @@ class Title {
         * Determine whether the object refers to a page within
         * this project and is transcludable.
         *
-        * @return \type{\bool} TRUE if this is transcludable
+        * @return Bool TRUE if this is transcludable
         */
        public function isTrans() {
-               if ($this->mInterwiki == '')
+               if ( $this->mInterwiki == '' ) {
                        return false;
+               }
 
                return Interwiki::fetch( $this->mInterwiki )->isTranscludable();
        }
 
+       /**
+        * Returns the DB name of the distant wiki 
+        * which owns the object.
+        *
+        * @return String the DB name
+        */
+       public function getTransWikiID() {
+               if ( $this->mInterwiki == '' ) {
+                       return false;
+               }
+
+               return Interwiki::fetch( $this->mInterwiki )->getWikiID();
+       }
+
        /**
         * Escape a text fragment, say from a link, for a URL
+        *
+        * @param $fragment string containing a URL or link fragment (after the "#")
+        * @return String: escaped string
         */
        static function escapeFragmentForURL( $fragment ) {
-               global $wgEnforceHtmlIds;
                # Note that we don't urlencode the fragment.  urlencoded Unicode
                # fragments appear not to work in IE (at least up to 7) or in at least
                # one version of Opera 9.x.  The W3C validator, for one, doesn't seem
                # to care if they aren't encoded.
-               return Sanitizer::escapeId( $fragment,
-                       $wgEnforceHtmlIds ? 'noninitial' : 'xml' );
+               return Sanitizer::escapeId( $fragment, 'noninitial' );
        }
 
-#----------------------------------------------------------------------------
+# ----------------------------------------------------------------------------
 #      Other stuff
-#----------------------------------------------------------------------------
+# ----------------------------------------------------------------------------
 
        /** Simple accessors */
        /**
         * Get the text form (spaces not underscores) of the main part
-        * @return \type{\string} Main part of the title
+        *
+        * @return String Main part of the title
         */
        public function getText() { return $this->mTextform; }
+
        /**
         * Get the URL-encoded form of the main part
-        * @return \type{\string} Main part of the title, URL-encoded
+        *
+        * @return String Main part of the title, URL-encoded
         */
        public function getPartialURL() { return $this->mUrlform; }
+
        /**
         * Get the main part with underscores
-        * @return \type{\string} Main part of the title, with underscores
+        *
+        * @return String: Main part of the title, with underscores
         */
        public function getDBkey() { return $this->mDbkeyform; }
+
        /**
         * Get the namespace index, i.e.\ one of the NS_xxxx constants.
-        * @return \type{\int} Namespace index
+        *
+        * @return Integer: Namespace index
         */
        public function getNamespace() { return $this->mNamespace; }
+
        /**
         * Get the namespace text
-        * @return \type{\string} Namespace text
+        *
+        * @return String: Namespace text
         */
        public function getNsText() {
                global $wgContLang;
 
-               if ( '' != $this->mInterwiki ) {
+               if ( $this->mInterwiki != '' ) {
                        // This probably shouldn't even happen. ohh man, oh yuck.
                        // But for interwiki transclusion it sometimes does.
                        // Shit. Shit shit shit.
                        //
                        // Use the canonical namespaces if possible to try to
                        // resolve a foreign namespace.
-                       if( MWNamespace::exists( $this->mNamespace ) ) {
+                       if ( MWNamespace::exists( $this->mNamespace ) ) {
                                return MWNamespace::getCanonicalName( $this->mNamespace );
                        }
                }
                return $wgContLang->getNsText( $this->mNamespace );
        }
+
        /**
         * Get the DB key with the initial letter case as specified by the user
-        * @return \type{\string} DB key
+        *
+        * @return String DB key
         */
        function getUserCaseDBKey() {
                return $this->mUserCaseDBKey;
        }
+
        /**
         * Get the namespace text of the subject (rather than talk) page
-        * @return \type{\string} Namespace text
+        *
+        * @return String Namespace text
         */
        public function getSubjectNsText() {
                global $wgContLang;
                return $wgContLang->getNsText( MWNamespace::getSubject( $this->mNamespace ) );
        }
+
        /**
         * Get the namespace text of the talk page
-        * @return \type{\string} Namespace text
+        *
+        * @return String Namespace text
         */
        public function getTalkNsText() {
                global $wgContLang;
                return( $wgContLang->getNsText( MWNamespace::getTalk( $this->mNamespace ) ) );
        }
+
        /**
         * Could this title have a corresponding talk page?
-        * @return \type{\bool} TRUE or FALSE
+        *
+        * @return Bool TRUE or FALSE
         */
        public function canTalk() {
                return( MWNamespace::canTalk( $this->mNamespace ) );
        }
+
        /**
         * Get the interwiki prefix (or null string)
-        * @return \type{\string} Interwiki prefix
+        *
+        * @return String Interwiki prefix
         */
        public function getInterwiki() { return $this->mInterwiki; }
+
        /**
         * Get the Title fragment (i.e.\ the bit after the #) in text form
-        * @return \type{\string} Title fragment
+        *
+        * @return String Title fragment
         */
        public function getFragment() { return $this->mFragment; }
+
        /**
         * Get the fragment in URL form, including the "#" character if there is one
-        * @return \type{\string} Fragment in URL form
+        * @return String Fragment in URL form
         */
        public function getFragmentForURL() {
                if ( $this->mFragment == '' ) {
@@ -612,15 +689,18 @@ class Title {
                        return '#' . Title::escapeFragmentForURL( $this->mFragment );
                }
        }
+
        /**
         * Get the default namespace index, for when there is no namespace
-        * @return \type{\int} Default namespace index
+        *
+        * @return Int Default namespace index
         */
        public function getDefaultNamespace() { return $this->mDefaultNamespace; }
 
        /**
         * Get title for search index
-        * @return \type{\string} a stripped-down title string ready for the
+        *
+        * @return String a stripped-down title string ready for the
         *  search index
         */
        public function getIndexTitle() {
@@ -629,7 +709,8 @@ class Title {
 
        /**
         * Get the prefixed database key form
-        * @return \type{\string} the prefixed title, with underscores and
+        *
+        * @return String the prefixed title, with underscores and
         *  any interwiki and namespace prefixes
         */
        public function getPrefixedDBkey() {
@@ -641,7 +722,8 @@ class Title {
        /**
         * Get the prefixed title with spaces.
         * This is the form usually used for display
-        * @return \type{\string} the prefixed title, with spaces
+        *
+        * @return String the prefixed title, with spaces
         */
        public function getPrefixedText() {
                if ( empty( $this->mPrefixedText ) ) { // FIXME: bad usage of empty() ?
@@ -655,12 +737,13 @@ class Title {
        /**
         * Get the prefixed title with spaces, plus any fragment
         * (part beginning with '#')
-        * @return \type{\string} the prefixed title, with spaces and
+        *
+        * @return String the prefixed title, with spaces and
         *  the fragment, including '#'
         */
        public function getFullText() {
                $text = $this->getPrefixedText();
-               if( '' != $this->mFragment ) {
+               if ( $this->mFragment != '' ) {
                        $text .= '#' . $this->mFragment;
                }
                return $text;
@@ -668,35 +751,39 @@ class Title {
 
        /**
         * Get the base name, i.e. the leftmost parts before the /
-        * @return \type{\string} Base name
+        *
+        * @return String Base name
         */
        public function getBaseText() {
-               if( !MWNamespace::hasSubpages( $this->mNamespace ) ) {
+               if ( !MWNamespace::hasSubpages( $this->mNamespace ) ) {
                        return $this->getText();
                }
 
                $parts = explode( '/', $this->getText() );
                # Don't discard the real title if there's no subpage involved
-               if( count( $parts ) > 1 )
-                       unset( $parts[ count( $parts ) - 1 ] );
+               if ( count( $parts ) > 1 ) {
+                       unset( $parts[count( $parts ) - 1] );
+               }
                return implode( '/', $parts );
        }
 
        /**
         * Get the lowest-level subpage name, i.e. the rightmost part after /
-        * @return \type{\string} Subpage name
+        *
+        * @return String Subpage name
         */
        public function getSubpageText() {
-               if( !MWNamespace::hasSubpages( $this->mNamespace ) ) {
+               if ( !MWNamespace::hasSubpages( $this->mNamespace ) ) {
                        return( $this->mTextform );
                }
                $parts = explode( '/', $this->mTextform );
-               return( $parts[ count( $parts ) - 1 ] );
+               return( $parts[count( $parts ) - 1] );
        }
 
        /**
         * Get a URL-encoded form of the subpage text
-        * @return \type{\string} URL-encoded subpage name
+        *
+        * @return String URL-encoded subpage name
         */
        public function getSubpageUrlForm() {
                $text = $this->getSubpageText();
@@ -706,7 +793,8 @@ class Title {
 
        /**
         * Get a URL-encoded title (not an actual URL) including interwiki
-        * @return \type{\string} the URL-encoded form
+        *
+        * @return String the URL-encoded form
         */
        public function getPrefixedURL() {
                $s = $this->prefix( $this->mDbkeyform );
@@ -721,13 +809,13 @@ class Title {
         * @param $query \twotypes{\string,\array} an optional query string, not used for interwiki
         *   links. Can be specified as an associative array as well, e.g.,
         *   array( 'action' => 'edit' ) (keys and values will be URL-escaped).
-        * @param $variant \type{\string} language variant of url (for sr, zh..)
-        * @return \type{\string} the URL
+        * @param $variant String language variant of url (for sr, zh..)
+        * @return String the URL
         */
        public function getFullURL( $query = '', $variant = false ) {
-               global $wgContLang, $wgServer, $wgRequest;
+               global $wgServer, $wgRequest;
 
-               if( is_array( $query ) ) {
+               if ( is_array( $query ) ) {
                        $query = wfArrayToCGI( $query );
                }
 
@@ -737,14 +825,14 @@ class Title {
 
                        // Ugly quick hack to avoid duplicate prefixes (bug 4571 etc)
                        // Correct fix would be to move the prepending elsewhere.
-                       if ($wgRequest->getVal('action') != 'render') {
+                       if ( $wgRequest->getVal( 'action' ) != 'render' ) {
                                $url = $wgServer . $url;
                        }
                } else {
-                       $baseUrl = $interwiki->getURL( );
+                       $baseUrl = $interwiki->getURL();
 
                        $namespace = wfUrlencode( $this->getNsText() );
-                       if ( '' != $namespace ) {
+                       if ( $namespace != '' ) {
                                # Can this actually happen? Interwikis shouldn't be parsed.
                                # Yes! It can in interwiki transclusion. But... it probably shouldn't.
                                $namespace .= ':';
@@ -763,28 +851,22 @@ class Title {
        /**
         * Get a URL with no fragment or server name.  If this page is generated
         * with action=render, $wgServer is prepended.
-        * @param mixed $query an optional query string; if not specified,
+        *
+        * @param $query Mixed: an optional query string; if not specified,
         *   $wgArticlePath will be used.  Can be specified as an associative array
         *   as well, e.g., array( 'action' => 'edit' ) (keys and values will be
         *   URL-escaped).
-        * @param $variant \type{\string} language variant of url (for sr, zh..)
-        * @return \type{\string} the URL
+        * @param $variant String language variant of url (for sr, zh..)
+        * @return String the URL
         */
        public function getLocalURL( $query = '', $variant = false ) {
                global $wgArticlePath, $wgScript, $wgServer, $wgRequest;
-               global $wgVariantArticlePath, $wgContLang, $wgUser;
+               global $wgVariantArticlePath, $wgContLang;
 
-               if( is_array( $query ) ) {
+               if ( is_array( $query ) ) {
                        $query = wfArrayToCGI( $query );
                }
 
-               // internal links should point to same variant as current page (only anonymous users)
-               if($variant == false && $wgContLang->hasVariants() && !$wgUser->isLoggedIn()){
-                       $pref = $wgContLang->getPreferredVariant(false);
-                       if($pref != $wgContLang->getCode())
-                               $variant = $pref;
-               }
-
                if ( $this->isExternal() ) {
                        $url = $this->getFullURL();
                        if ( $query ) {
@@ -797,8 +879,8 @@ class Title {
                } else {
                        $dbkey = wfUrlencode( $this->getPrefixedDBkey() );
                        if ( $query == '' ) {
-                               if( $variant != false && $wgContLang->hasVariants() ) {
-                                       if( $wgVariantArticlePath == false ) {
+                               if ( $variant != false && $wgContLang->hasVariants() ) {
+                                       if ( !$wgVariantArticlePath ) {
                                                $variantArticlePath =  "$wgScript?title=$1&variant=$2"; // default
                                        } else {
                                                $variantArticlePath = $wgVariantArticlePath;
@@ -812,15 +894,17 @@ class Title {
                                global $wgActionPaths;
                                $url = false;
                                $matches = array();
-                               if( !empty( $wgActionPaths ) &&
+                               if ( !empty( $wgActionPaths ) &&
                                        preg_match( '/^(.*&|)action=([^&]*)(&(.*)|)$/', $query, $matches ) )
                                {
                                        $action = urldecode( $matches[2] );
-                                       if( isset( $wgActionPaths[$action] ) ) {
+                                       if ( isset( $wgActionPaths[$action] ) ) {
                                                $query = $matches[1];
-                                               if( isset( $matches[4] ) ) $query .= $matches[4];
+                                               if ( isset( $matches[4] ) ) {
+                                                       $query .= $matches[4];
+                                               }
                                                $url = str_replace( '$1', $dbkey, $wgActionPaths[$action] );
-                                               if( $query != '' ) {
+                                               if ( $query != '' ) {
                                                        $url = wfAppendQuery( $url, $query );
                                                }
                                        }
@@ -835,7 +919,7 @@ class Title {
 
                        // FIXME: this causes breakage in various places when we
                        // actually expected a local URL and end up with dupe prefixes.
-                       if ($wgRequest->getVal('action') == 'render') {
+                       if ( $wgRequest->getVal( 'action' ) == 'render' ) {
                                $url = $wgServer . $url;
                        }
                }
@@ -853,18 +937,18 @@ class Title {
         * The result obviously should not be URL-escaped, but does need to be
         * HTML-escaped if it's being output in HTML.
         *
-        * @param $query \type{\arrayof{\string}} An associative array of key => value pairs for the
+        * @param $query Array of Strings An associative array of key => value pairs for the
         *   query string.  Keys and values will be escaped.
-        * @param $variant \type{\string} Language variant of URL (for sr, zh..).  Ignored
+        * @param $variant String Language variant of URL (for sr, zh..).  Ignored
         *   for external links.  Default is "false" (same variant as current page,
         *   for anonymous users).
-        * @return \type{\string} the URL
+        * @return String the URL
         */
        public function getLinkUrl( $query = array(), $variant = false ) {
                wfProfileIn( __METHOD__ );
-               if( $this->isExternal() ) {
+               if ( $this->isExternal() ) {
                        $ret = $this->getFullURL( $query );
-               } elseif( $this->getPrefixedText() === '' && $this->getFragment() !== '' ) {
+               } elseif ( $this->getPrefixedText() === '' && $this->getFragment() !== '' ) {
                        $ret = $this->getFragmentForURL();
                } else {
                        $ret = $this->getLocalURL( $query, $variant ) . $this->getFragmentForURL();
@@ -876,8 +960,9 @@ class Title {
        /**
         * Get an HTML-escaped version of the URL form, suitable for
         * using in a link, without a server name or fragment
-        * @param $query \type{\string} an optional query string
-        * @return \type{\string} the URL
+        *
+        * @param $query String an optional query string
+        * @return String the URL
         */
        public function escapeLocalURL( $query = '' ) {
                return htmlspecialchars( $this->getLocalURL( $query ) );
@@ -887,8 +972,8 @@ class Title {
         * Get an HTML-escaped version of the URL form, suitable for
         * using in a link, including the server name and fragment
         *
-        * @param $query \type{\string} an optional query string
-        * @return \type{\string} the URL
+        * @param $query String an optional query string
+        * @return String the URL
         */
        public function escapeFullURL( $query = '' ) {
                return htmlspecialchars( $this->getFullURL( $query ) );
@@ -899,9 +984,9 @@ class Title {
         * - Used in various Squid-related code, in case we have a different
         * internal hostname for the server from the exposed one.
         *
-        * @param $query \type{\string} an optional query string
-        * @param $variant \type{\string} language variant of url (for sr, zh..)
-        * @return \type{\string} the URL
+        * @param $query String an optional query string
+        * @param $variant String language variant of url (for sr, zh..)
+        * @return String the URL
         */
        public function getInternalURL( $query = '', $variant = false ) {
                global $wgInternalServer;
@@ -912,11 +997,14 @@ class Title {
 
        /**
         * Get the edit URL for this Title
-        * @return \type{\string} the URL, or a null string if this is an
+        *
+        * @return String the URL, or a null string if this is an
         *  interwiki link
         */
        public function getEditURL() {
-               if ( '' != $this->mInterwiki ) { return ''; }
+               if ( $this->mInterwiki != '' ) {
+                       return '';
+               }
                $s = $this->getLocalURL( 'action=edit' );
 
                return $s;
@@ -925,7 +1013,8 @@ class Title {
        /**
         * Get the HTML-escaped displayable text form.
         * Used for the title field in <a> tags.
-        * @return \type{\string} the text, including any prefixes
+        *
+        * @return String the text, including any prefixes
         */
        public function getEscapedText() {
                return htmlspecialchars( $this->getPrefixedText() );
@@ -933,23 +1022,27 @@ class Title {
 
        /**
         * Is this Title interwiki?
-        * @return \type{\bool}
+        *
+        * @return Bool
         */
-       public function isExternal() { return ( '' != $this->mInterwiki ); }
+       public function isExternal() {
+               return ( $this->mInterwiki != '' );
+       }
 
        /**
         * Is this page "semi-protected" - the *only* protection is autoconfirm?
         *
-        * @param @action \type{\string} Action to check (default: edit)
-        * @return \type{\bool}
+        * @param $action String Action to check (default: edit)
+        * @return Bool
         */
        public function isSemiProtected( $action = 'edit' ) {
-               if( $this->exists() ) {
+               if ( $this->exists() ) {
                        $restrictions = $this->getRestrictions( $action );
-                       if( count( $restrictions ) > 0 ) {
-                               foreach( $restrictions as $restriction ) {
-                                       if( strtolower( $restriction ) != 'autoconfirmed' )
+                       if ( count( $restrictions ) > 0 ) {
+                               foreach ( $restrictions as $restriction ) {
+                                       if ( strtolower( $restriction ) != 'autoconfirmed' ) {
                                                return false;
+                                       }
                                }
                        } else {
                                # Not protected
@@ -964,9 +1057,10 @@ class Title {
 
        /**
         * Does the title correspond to a protected article?
-        * @param $what \type{\string} the action the page is protected from,
+        *
+        * @param $action String the action the page is protected from,
         * by default checks all actions.
-        * @return \type{\bool}
+        * @return Bool
         */
        public function isProtected( $action = '' ) {
                global $wgRestrictionLevels;
@@ -974,15 +1068,16 @@ class Title {
                $restrictionTypes = $this->getRestrictionTypes();
 
                # Special pages have inherent protection
-               if( $this->getNamespace() == NS_SPECIAL )
+               if( $this->getNamespace() == NS_SPECIAL ) {
                        return true;
+               }
 
                # Check regular protection levels
-               foreach( $restrictionTypes as $type ){
-                       if( $action == $type || $action == '' ) {
+               foreach ( $restrictionTypes as $type ) {
+                       if ( $action == $type || $action == '' ) {
                                $r = $this->getRestrictions( $type );
-                               foreach( $wgRestrictionLevels as $level ) {
-                                       if( in_array( $level, $r ) && $level != '' ) {
+                               foreach ( $wgRestrictionLevels as $level ) {
+                                       if ( in_array( $level, $r ) && $level != '' ) {
                                                return true;
                                        }
                                }
@@ -992,15 +1087,33 @@ class Title {
                return false;
        }
 
+       /**
+        * Is this a conversion table for the LanguageConverter?
+        *
+        * @return Bool
+        */
+       public function isConversionTable() {
+               if(
+                       $this->getNamespace() == NS_MEDIAWIKI &&
+                       strpos( $this->getText(), 'Conversiontable' ) !== false
+               )
+               {
+                       return true;
+               }
+
+               return false;
+       }
+
        /**
         * Is $wgUser watching this page?
-        * @return \type{\bool}
+        *
+        * @return Bool
         */
        public function userIsWatching() {
                global $wgUser;
 
                if ( is_null( $this->mWatched ) ) {
-                       if ( NS_SPECIAL == $this->mNamespace || !$wgUser->isLoggedIn()) {
+                       if ( NS_SPECIAL == $this->mNamespace || !$wgUser->isLoggedIn() ) {
                                $this->mWatched = false;
                        } else {
                                $this->mWatched = $wgUser->isWatched( $this );
@@ -1019,25 +1132,33 @@ class Title {
         *
         * May provide false positives, but should never provide a false negative.
         *
-        * @param $action \type{\string} action that permission needs to be checked for
-        * @return \type{\bool}
+        * @param $action String action that permission needs to be checked for
+        * @return Bool
         */
        public function quickUserCan( $action ) {
                return $this->userCan( $action, false );
        }
 
        /**
-        * Determines if $wgUser is unable to edit this page because it has been protected
+        * Determines if $user is unable to edit this page because it has been protected
         * by $wgNamespaceProtection.
         *
-        * @return \type{\bool}
+        * @param $user User object, $wgUser will be used if not passed
+        * @return Bool
         */
-       public function isNamespaceProtected() {
-               global $wgNamespaceProtection, $wgUser;
-               if( isset( $wgNamespaceProtection[ $this->mNamespace ] ) ) {
-                       foreach( (array)$wgNamespaceProtection[ $this->mNamespace ] as $right ) {
-                               if( $right != '' && !$wgUser->isAllowed( $right ) )
+       public function isNamespaceProtected( User $user = null ) {
+               global $wgNamespaceProtection;
+
+               if ( $user === null ) {
+                       global $wgUser;
+                       $user = $wgUser;
+               }
+
+               if ( isset( $wgNamespaceProtection[$this->mNamespace] ) ) {
+                       foreach ( (array)$wgNamespaceProtection[$this->mNamespace] as $right ) {
+                               if ( $right != '' && !$user->isAllowed( $right ) ) {
                                        return true;
+                               }
                        }
                }
                return false;
@@ -1045,13 +1166,14 @@ class Title {
 
        /**
         * Can $wgUser perform $action on this page?
-        * @param $action \type{\string} action that permission needs to be checked for
-        * @param $doExpensiveQueries \type{\bool} Set this to false to avoid doing unnecessary queries.
-        * @return \type{\bool}
+        *
+        * @param $action String action that permission needs to be checked for
+        * @param $doExpensiveQueries Bool Set this to false to avoid doing unnecessary queries.
+        * @return Bool
         */
        public function userCan( $action, $doExpensiveQueries = true ) {
                global $wgUser;
-               return ($this->getUserPermissionsErrorsInternal( $action, $wgUser, $doExpensiveQueries, true ) === array());
+               return ( $this->getUserPermissionsErrorsInternal( $action, $wgUser, $doExpensiveQueries, true ) === array() );
        }
 
        /**
@@ -1059,86 +1181,21 @@ class Title {
         *
         * FIXME: This *does not* check throttles (User::pingLimiter()).
         *
-        * @param $action \type{\string}action that permission needs to be checked for
-        * @param $user \type{User} user to check
-        * @param $doExpensiveQueries \type{\bool} Set this to false to avoid doing unnecessary queries.
-        * @param $ignoreErrors \type{\arrayof{\string}} Set this to a list of message keys whose corresponding errors may be ignored.
-        * @return \type{\array} Array of arrays of the arguments to wfMsg to explain permissions problems.
+        * @param $action String action that permission needs to be checked for
+        * @param $user User to check
+        * @param $doExpensiveQueries Bool Set this to false to avoid doing unnecessary queries.
+        * @param $ignoreErrors Array of Strings Set this to a list of message keys whose corresponding errors may be ignored.
+        * @return Array of arrays of the arguments to wfMsg to explain permissions problems.
         */
        public function getUserPermissionsErrors( $action, $user, $doExpensiveQueries = true, $ignoreErrors = array() ) {
-               if( !StubObject::isRealObject( $user ) ) {
-                       //Since StubObject is always used on globals, we can unstub $wgUser here and set $user = $wgUser
-                       global $wgUser;
-                       $wgUser->_unstub( '', 5 );
-                       $user = $wgUser;
-               }
                $errors = $this->getUserPermissionsErrorsInternal( $action, $user, $doExpensiveQueries );
 
-               global $wgContLang;
-               global $wgLang;
-               global $wgEmailConfirmToEdit;
-
-               if ( $wgEmailConfirmToEdit && !$user->isEmailConfirmed() && $action != 'createaccount' ) {
-                       $errors[] = array( 'confirmedittext' );
-               }
-
-               // Edit blocks should not affect reading. Account creation blocks handled at userlogin.
-               if ( $action != 'read' && $action != 'createaccount' && $user->isBlockedFrom( $this ) ) {
-                       $block = $user->mBlock;
-
-                       // This is from OutputPage::blockedPage
-                       // Copied at r23888 by werdna
-
-                       $id = $user->blockedBy();
-                       $reason = $user->blockedFor();
-                       if( $reason == '' ) {
-                               $reason = wfMsg( 'blockednoreason' );
-                       }
-                       $ip = wfGetIP();
-
-                       if ( is_numeric( $id ) ) {
-                               $name = User::whoIs( $id );
-                       } else {
-                               $name = $id;
-                       }
-
-                       $link = '[[' . $wgContLang->getNsText( NS_USER ) . ":{$name}|{$name}]]";
-                       $blockid = $block->mId;
-                       $blockExpiry = $user->mBlock->mExpiry;
-                       $blockTimestamp = $wgLang->timeanddate( wfTimestamp( TS_MW, $user->mBlock->mTimestamp ), true );
-
-                       if ( $blockExpiry == 'infinity' ) {
-                               // Entry in database (table ipblocks) is 'infinity' but 'ipboptions' uses 'infinite' or 'indefinite'
-                               $scBlockExpiryOptions = wfMsg( 'ipboptions' );
-
-                               foreach ( explode( ',', $scBlockExpiryOptions ) as $option ) {
-                                       if ( strpos( $option, ':' ) == false )
-                                               continue;
-
-                                       list ($show, $value) = explode( ":", $option );
-
-                                       if ( $value == 'infinite' || $value == 'indefinite' ) {
-                                               $blockExpiry = $show;
-                                               break;
-                                       }
-                               }
-                       } else {
-                               $blockExpiry = $wgLang->timeanddate( wfTimestamp( TS_MW, $blockExpiry ), true );
-                       }
-
-                       $intended = $user->mBlock->mAddress;
-
-                       $errors[] = array( ($block->mAuto ? 'autoblockedtext' : 'blockedtext'), $link, $reason, $ip, $name,
-                               $blockid, $blockExpiry, $intended, $blockTimestamp );
-               }
-
                // Remove the errors being ignored.
+               foreach ( $errors as $index => $error ) {
+                       $error_key = is_array( $error ) ? $error[0] : $error;
 
-               foreach( $errors as $index => $error ) {
-                       $error_key = is_array($error) ? $error[0] : $error;
-
-                       if (in_array( $error_key, $ignoreErrors )) {
-                               unset($errors[$index]);
+                       if ( in_array( $error_key, $ignoreErrors ) ) {
+                               unset( $errors[$index] );
                        }
                }
 
@@ -1146,36 +1203,35 @@ class Title {
        }
 
        /**
-        * Can $user perform $action on this page? This is an internal function,
-        * which checks ONLY that previously checked by userCan (i.e. it leaves out
-        * checks on wfReadOnly() and blocks)
+        * Permissions checks that fail most often, and which are easiest to test.
         *
-        * @param $action \type{\string} action that permission needs to be checked for
-        * @param $user \type{User} user to check
-        * @param $doExpensiveQueries \type{\bool} Set this to false to avoid doing unnecessary queries.
-        * @param $short \type{\bool} Set this to true to stop after the first permission error.
-        * @return \type{\array} Array of arrays of the arguments to wfMsg to explain permissions problems.
+        * @param $action String the action to check
+        * @param $user User user to check
+        * @param $errors Array list of current errors
+        * @param $doExpensiveQueries Boolean whether or not to perform expensive queries
+        * @param $short Boolean short circuit on first error
+        *
+        * @return Array list of errors
         */
-       private function getUserPermissionsErrorsInternal( $action, $user, $doExpensiveQueries=true, $short=false ) {
-               wfProfileIn( __METHOD__ );
-
-               $errors = array();
-
-               // First stop is permissions checks, which fail most often, and which are easiest to test.
-               if ( $action == 'move' ) {
-                       if( !$user->isAllowed( 'move-rootuserpages' )
-                                       && $this->getNamespace() == NS_USER && !$this->isSubpage() )
-                       {
+       private function checkQuickPermissions( $action, $user, $errors, $doExpensiveQueries, $short ) {
+               if ( $action == 'create' ) {
+                       if ( ( $this->isTalkPage() && !$user->isAllowed( 'createtalk' ) ) ||
+                                ( !$this->isTalkPage() && !$user->isAllowed( 'createpage' ) ) ) {
+                               $errors[] = $user->isAnon() ? array( 'nocreatetext' ) : array( 'nocreate-loggedin' );
+                       }
+               } elseif ( $action == 'move' ) {
+                       if ( !$user->isAllowed( 'move-rootuserpages' )
+                                       && $this->mNamespace == NS_USER && !$this->isSubpage() ) {
                                // Show user page-specific message only if the user can move other pages
                                $errors[] = array( 'cant-move-user-page' );
                        }
 
                        // Check if user is allowed to move files if it's a file
-                       if( $this->getNamespace() == NS_FILE && !$user->isAllowed( 'movefile' ) ) {
+                       if ( $this->mNamespace == NS_FILE && !$user->isAllowed( 'movefile' ) ) {
                                $errors[] = array( 'movenotallowedfile' );
                        }
 
-                       if( !$user->isAllowed( 'move' ) ) {
+                       if ( !$user->isAllowed( 'move' ) ) {
                                // User can't move anything
                                global $wgGroupPermissions;
                                $userCanMove = false;
@@ -1188,135 +1244,183 @@ class Title {
                                }
                                if ( $user->isAnon() && ( $userCanMove || $autoconfirmedCanMove ) ) {
                                        // custom message if logged-in users without any special rights can move
-                                       $errors[] = array ( 'movenologintext' );
+                                       $errors[] = array( 'movenologintext' );
                                } else {
-                                       $errors[] = array ('movenotallowed');
+                                       $errors[] = array( 'movenotallowed' );
                                }
                        }
-               } elseif ( $action == 'create' ) {
-                       if( ( $this->isTalkPage() && !$user->isAllowed( 'createtalk' ) ) ||
-                               ( !$this->isTalkPage() && !$user->isAllowed( 'createpage' ) ) )
-                       {
-                               $errors[] = $user->isAnon() ? array ('nocreatetext') : array ('nocreate-loggedin');
-                       }
-               } elseif( $action == 'move-target' ) {
-                       if( !$user->isAllowed( 'move' ) ) {
+               } elseif ( $action == 'move-target' ) {
+                       if ( !$user->isAllowed( 'move' ) ) {
                                // User can't move anything
-                               $errors[] = array ('movenotallowed');
-                       } elseif( !$user->isAllowed( 'move-rootuserpages' )
-                               && $this->getNamespace() == NS_USER && !$this->isSubpage() )
-                       {
+                               $errors[] = array( 'movenotallowed' );
+                       } elseif ( !$user->isAllowed( 'move-rootuserpages' )
+                                       && $this->mNamespace == NS_USER && !$this->isSubpage() ) {
                                // Show user page-specific message only if the user can move other pages
                                $errors[] = array( 'cant-move-to-user-page' );
                        }
-               } elseif( !$user->isAllowed( $action ) ) {
-                       $return = null;
-
+               } elseif ( !$user->isAllowed( $action ) ) {
                        // We avoid expensive display logic for quickUserCan's and such
                        $groups = false;
-                       if (!$short) {
+                       if ( !$short ) {
                                $groups = array_map( array( 'User', 'makeGroupLinkWiki' ),
                                        User::getGroupsWithPermission( $action ) );
                        }
 
-                       if( $groups ) {
-                               $return = array( 'badaccess-groups',
-                                       array( implode( ', ', $groups ), count( $groups ) ) );
+                       if ( $groups ) {
+                               global $wgLang;
+                               $return = array(
+                                       'badaccess-groups',
+                                       $wgLang->commaList( $groups ),
+                                       count( $groups )
+                               );
                        } else {
-                               $return = array( "badaccess-group0" );
+                               $return = array( 'badaccess-group0' );
                        }
                        $errors[] = $return;
                }
 
-               # Short-circuit point
-               if( $short && count($errors) > 0 ) {
-                       wfProfileOut( __METHOD__ );
-                       return $errors;
+               return $errors;
+       }
+
+       /**
+        * Add the resulting error code to the errors array
+        *
+        * @param $errors Array list of current errors
+        * @param $result Mixed result of errors
+        *
+        * @return Array list of errors
+        */
+       private function resultToError( $errors, $result ) {
+               if ( is_array( $result ) && count( $result ) && !is_array( $result[0] ) ) {
+                       // A single array representing an error
+                       $errors[] = $result;
+               } else if ( is_array( $result ) && is_array( $result[0] ) ) {
+                       // A nested array representing multiple errors
+                       $errors = array_merge( $errors, $result );
+               } else if ( $result !== '' && is_string( $result ) ) {
+                       // A string representing a message-id
+                       $errors[] = array( $result );
+               } else if ( $result === false ) {
+                       // a generic "We don't want them to do that"
+                       $errors[] = array( 'badaccess-group0' );
                }
+               return $errors;
+       }
 
+       /**
+        * Check various permission hooks
+        *
+        * @param $action String the action to check
+        * @param $user User user to check
+        * @param $errors Array list of current errors
+        * @param $doExpensiveQueries Boolean whether or not to perform expensive queries
+        * @param $short Boolean short circuit on first error
+        *
+        * @return Array list of errors
+        */
+       private function checkPermissionHooks( $action, $user, $errors, $doExpensiveQueries, $short ) {
                // Use getUserPermissionsErrors instead
-               if( !wfRunHooks( 'userCan', array( &$this, &$user, $action, &$result ) ) ) {
-                       wfProfileOut( __METHOD__ );
+               $result = '';
+               if ( !wfRunHooks( 'userCan', array( &$this, &$user, $action, &$result ) ) ) {
                        return $result ? array() : array( array( 'badaccess-group0' ) );
                }
                // Check getUserPermissionsErrors hook
-               if( !wfRunHooks( 'getUserPermissionsErrors', array(&$this,&$user,$action,&$result) ) ) {
-                       if( is_array($result) && count($result) && !is_array($result[0]) )
-                               $errors[] = $result; # A single array representing an error
-                       else if( is_array($result) && is_array($result[0]) )
-                               $errors = array_merge( $errors, $result ); # A nested array representing multiple errors
-                       else if( $result !== '' && is_string($result) )
-                               $errors[] = array($result); # A string representing a message-id
-                       else if( $result === false )
-                               $errors[] = array('badaccess-group0'); # a generic "We don't want them to do that"
-               }
-               # Short-circuit point
-               if( $short && count($errors) > 0 ) {
-                       wfProfileOut( __METHOD__ );
-                       return $errors;
+               if ( !wfRunHooks( 'getUserPermissionsErrors', array( &$this, &$user, $action, &$result ) ) ) {
+                       $errors = $this->resultToError( $errors, $result );
                }
                // Check getUserPermissionsErrorsExpensive hook
-               if( $doExpensiveQueries && !wfRunHooks( 'getUserPermissionsErrorsExpensive', array(&$this,&$user,$action,&$result) ) ) {
-                       if( is_array($result) && count($result) && !is_array($result[0]) )
-                               $errors[] = $result; # A single array representing an error
-                       else if( is_array($result) && is_array($result[0]) )
-                               $errors = array_merge( $errors, $result ); # A nested array representing multiple errors
-                       else if( $result !== '' && is_string($result) )
-                               $errors[] = array($result); # A string representing a message-id
-                       else if( $result === false )
-                               $errors[] = array('badaccess-group0'); # a generic "We don't want them to do that"
-               }
-               # Short-circuit point
-               if( $short && count($errors) > 0 ) {
-                       wfProfileOut( __METHOD__ );
-                       return $errors;
+               if ( $doExpensiveQueries && !( $short && count( $errors ) > 0 ) &&
+                        !wfRunHooks( 'getUserPermissionsErrorsExpensive', array( &$this, &$user, $action, &$result ) ) ) {
+                       $errors = $this->resultToError( $errors, $result );
                }
 
+               return $errors;
+       }
+
+       /**
+        * Check permissions on special pages & namespaces
+        *
+        * @param $action String the action to check
+        * @param $user User user to check
+        * @param $errors Array list of current errors
+        * @param $doExpensiveQueries Boolean whether or not to perform expensive queries
+        * @param $short Boolean short circuit on first error
+        *
+        * @return Array list of errors
+        */
+       private function checkSpecialsAndNSPermissions( $action, $user, $errors, $doExpensiveQueries, $short ) {
                # Only 'createaccount' and 'execute' can be performed on
                # special pages, which don't actually exist in the DB.
                $specialOKActions = array( 'createaccount', 'execute' );
-               if( NS_SPECIAL == $this->mNamespace && !in_array( $action, $specialOKActions) ) {
-                       $errors[] = array('ns-specialprotected');
+               if ( NS_SPECIAL == $this->mNamespace && !in_array( $action, $specialOKActions ) ) {
+                       $errors[] = array( 'ns-specialprotected' );
                }
 
                # Check $wgNamespaceProtection for restricted namespaces
-               if( $this->isNamespaceProtected() ) {
-                       $ns = $this->getNamespace() == NS_MAIN ?
+               if ( $this->isNamespaceProtected( $user ) ) {
+                       $ns = $this->mNamespace == NS_MAIN ?
                                wfMsg( 'nstab-main' ) : $this->getNsText();
-                       $errors[] = NS_MEDIAWIKI == $this->mNamespace ?
-                               array('protectedinterface') : array( 'namespaceprotected',  $ns );
+                       $errors[] = $this->mNamespace == NS_MEDIAWIKI ?
+                               array( 'protectedinterface' ) : array( 'namespaceprotected',  $ns );
                }
 
+               return $errors;
+       }
+
+       /**
+        * Check CSS/JS sub-page permissions
+        *
+        * @param $action String the action to check
+        * @param $user User user to check
+        * @param $errors Array list of current errors
+        * @param $doExpensiveQueries Boolean whether or not to perform expensive queries
+        * @param $short Boolean short circuit on first error
+        *
+        * @return Array list of errors
+        */
+       private function checkCSSandJSPermissions( $action, $user, $errors, $doExpensiveQueries, $short ) {
                # Protect css/js subpages of user pages
                # XXX: this might be better using restrictions
                # XXX: Find a way to work around the php bug that prevents using $this->userCanEditCssSubpage()
                #      and $this->userCanEditJsSubpage() from working
                # XXX: right 'editusercssjs' is deprecated, for backward compatibility only
-               if( $this->isCssSubpage() && !( $user->isAllowed('editusercssjs') || $user->isAllowed('editusercss') )
-                       && $action != 'patrol'
-                       && !preg_match('/^'.preg_quote($user->getName(), '/').'\//', $this->mTextform) )
-               {
-                       $errors[] = array('customcssjsprotected');
-               } else if( $this->isJsSubpage() && !( $user->isAllowed('editusercssjs') || $user->isAllowed('edituserjs') )
-                       && $action != 'patrol'
-                       && !preg_match('/^'.preg_quote($user->getName(), '/').'\//', $this->mTextform) )
-               {
-                       $errors[] = array('customcssjsprotected');
+               if ( $action != 'patrol' && !$user->isAllowed( 'editusercssjs' )
+                               && !preg_match( '/^' . preg_quote( $user->getName(), '/' ) . '\//', $this->mTextform ) ) {
+                       if ( $this->isCssSubpage() && !$user->isAllowed( 'editusercss' ) ) {
+                               $errors[] = array( 'customcssjsprotected' );
+                       } else if ( $this->isJsSubpage() && !$user->isAllowed( 'edituserjs' ) ) {
+                               $errors[] = array( 'customcssjsprotected' );
+                       }
                }
 
-               # Check against page_restrictions table requirements on this
-               # page. The user must possess all required rights for this action.
-               foreach( $this->getRestrictions($action) as $right ) {
+               return $errors;
+       }
+
+       /**
+        * Check against page_restrictions table requirements on this
+        * page. The user must possess all required rights for this
+        * action.
+        *
+        * @param $action String the action to check
+        * @param $user User user to check
+        * @param $errors Array list of current errors
+        * @param $doExpensiveQueries Boolean whether or not to perform expensive queries
+        * @param $short Boolean short circuit on first error
+        *
+        * @return Array list of errors
+        */
+       private function checkPageRestrictions( $action, $user, $errors, $doExpensiveQueries, $short ) {
+               foreach ( $this->getRestrictions( $action ) as $right ) {
                        // Backwards compatibility, rewrite sysop -> protect
-                       if( $right == 'sysop' ) {
+                       if ( $right == 'sysop' ) {
                                $right = 'protect';
                        }
-                       if( '' != $right && !$user->isAllowed( $right ) ) {
+                       if ( $right != '' && !$user->isAllowed( $right ) ) {
                                // Users with 'editprotected' permission can edit protected pages
-                               if( $action=='edit' && $user->isAllowed( 'editprotected' ) ) {
+                               if ( $action == 'edit' && $user->isAllowed( 'editprotected' ) ) {
                                        // Users with 'editprotected' permission cannot edit protected pages
                                        // with cascading option turned on.
-                                       if( $this->mCascadeRestriction ) {
+                                       if ( $this->mCascadeRestriction ) {
                                                $errors[] = array( 'protectedpagetext', $right );
                                        }
                                } else {
@@ -1324,74 +1428,200 @@ class Title {
                                }
                        }
                }
-               # Short-circuit point
-               if( $short && count($errors) > 0 ) {
-                       wfProfileOut( __METHOD__ );
-                       return $errors;
-               }
 
-               if( $doExpensiveQueries && !$this->isCssJsSubpage() ) {
-                       # We /could/ use the protection level on the source page, but it's fairly ugly
-                       #  as we have to establish a precedence hierarchy for pages included by multiple
-                       #  cascade-protected pages. So just restrict it to people with 'protect' permission,
-                       #  as they could remove the protection anyway.
+               return $errors;
+       }
+
+       /**
+        * Check restrictions on cascading pages.
+        * 
+        * @param $action String the action to check
+        * @param $user User user to check
+        * @param $errors Array list of current errors
+        * @param $doExpensiveQueries Boolean whether or not to perform expensive queries
+        * @param $short Boolean short circuit on first error
+        *
+        * @return Array list of errors
+        */
+       private function checkCascadingSourcesRestrictions( $action, $user, $errors, $doExpensiveQueries, $short ) {
+               if ( $doExpensiveQueries && !$this->isCssJsSubpage() ) {
+                       # We /could/ use the protection level on the source page, but it's
+                       # fairly ugly as we have to establish a precedence hierarchy for pages
+                       # included by multiple cascade-protected pages. So just restrict
+                       # it to people with 'protect' permission, as they could remove the
+                       # protection anyway.
                        list( $cascadingSources, $restrictions ) = $this->getCascadeProtectionSources();
                        # Cascading protection depends on more than this page...
                        # Several cascading protected pages may include this page...
                        # Check each cascading level
                        # This is only for protection restrictions, not for all actions
-                       if( $cascadingSources > 0 && isset($restrictions[$action]) ) {
-                               foreach( $restrictions[$action] as $right ) {
+                       if ( isset( $restrictions[$action] ) ) {
+                               foreach ( $restrictions[$action] as $right ) {
                                        $right = ( $right == 'sysop' ) ? 'protect' : $right;
-                                       if( '' != $right && !$user->isAllowed( $right ) ) {
+                                       if ( $right != '' && !$user->isAllowed( $right ) ) {
                                                $pages = '';
-                                               foreach( $cascadingSources as $page )
+                                               foreach ( $cascadingSources as $page )
                                                        $pages .= '* [[:' . $page->getPrefixedText() . "]]\n";
                                                $errors[] = array( 'cascadeprotected', count( $cascadingSources ), $pages );
                                        }
                                }
                        }
                }
-               # Short-circuit point
-               if( $short && count($errors) > 0 ) {
-                       wfProfileOut( __METHOD__ );
-                       return $errors;
-               }
 
-               if( $action == 'protect' ) {
-                       if( $this->getUserPermissionsErrors('edit', $user) != array() ) {
-                               $errors[] = array( 'protect-cantedit' ); // If they can't edit, they shouldn't protect.
-                       }
-               }
+               return $errors;
+       }
 
-               if( $action == 'create' ) {
+       /**
+        * Check action permissions not already checked in checkQuickPermissions
+        *
+        * @param $action String the action to check
+        * @param $user User user to check
+        * @param $errors Array list of current errors
+        * @param $doExpensiveQueries Boolean whether or not to perform expensive queries
+        * @param $short Boolean short circuit on first error
+        *
+        * @return Array list of errors
+        */
+       private function checkActionPermissions( $action, $user, $errors, $doExpensiveQueries, $short ) {
+               if ( $action == 'protect' ) {
+                       if ( $this->getUserPermissionsErrors( 'edit', $user ) != array() ) {
+                               // If they can't edit, they shouldn't protect.
+                               $errors[] = array( 'protect-cantedit' );
+                       }
+               } elseif ( $action == 'create' ) {
                        $title_protection = $this->getTitleProtection();
-                       if( is_array($title_protection) ) {
-                               extract($title_protection); // is this extract() really needed?
-
-                               if( $pt_create_perm == 'sysop' ) {
-                                       $pt_create_perm = 'protect'; // B/C
+                       if( $title_protection ) {
+                               if( $title_protection['pt_create_perm'] == 'sysop' ) {
+                                       $title_protection['pt_create_perm'] = 'protect'; // B/C
                                }
-                               if( $pt_create_perm == '' || !$user->isAllowed($pt_create_perm) ) {
-                                       $errors[] = array( 'titleprotected', User::whoIs($pt_user), $pt_reason );
+                               if( $title_protection['pt_create_perm'] == '' || !$user->isAllowed( $title_protection['pt_create_perm'] ) ) {
+                                       $errors[] = array( 'titleprotected', User::whoIs( $title_protection['pt_user'] ), $title_protection['pt_reason'] );
                                }
                        }
-               } elseif( $action == 'move' ) {
+               } elseif ( $action == 'move' ) {
                        // Check for immobile pages
-                       if( !MWNamespace::isMovable( $this->getNamespace() ) ) {
+                       if ( !MWNamespace::isMovable( $this->mNamespace ) ) {
                                // Specific message for this case
                                $errors[] = array( 'immobile-source-namespace', $this->getNsText() );
-                       } elseif( !$this->isMovable() ) {
+                       } elseif ( !$this->isMovable() ) {
                                // Less specific message for rarer cases
                                $errors[] = array( 'immobile-page' );
                        }
-               } elseif( $action == 'move-target' ) {
-                       if( !MWNamespace::isMovable( $this->getNamespace() ) ) {
+               } elseif ( $action == 'move-target' ) {
+                       if ( !MWNamespace::isMovable( $this->mNamespace ) ) {
                                $errors[] = array( 'immobile-target-namespace', $this->getNsText() );
-                       } elseif( !$this->isMovable() ) {
+                       } elseif ( !$this->isMovable() ) {
                                $errors[] = array( 'immobile-target-page' );
                        }
                }
+               return $errors;
+       }
+
+       /**
+        * Check that the user isn't blocked from editting.
+        *
+        * @param $action String the action to check
+        * @param $user User user to check
+        * @param $errors Array list of current errors
+        * @param $doExpensiveQueries Boolean whether or not to perform expensive queries
+        * @param $short Boolean short circuit on first error
+        *
+        * @return Array list of errors
+        */
+       private function checkUserBlock( $action, $user, $errors, $doExpensiveQueries, $short ) {
+               if( $short && count( $errors ) > 0 ) {
+                       return $errors;
+               }
+
+               global $wgContLang, $wgLang, $wgEmailConfirmToEdit;
+
+               if ( $wgEmailConfirmToEdit && !$user->isEmailConfirmed() && $action != 'createaccount' ) {
+                       $errors[] = array( 'confirmedittext' );
+               }
+
+               // Edit blocks should not affect reading. Account creation blocks handled at userlogin.
+               if ( $action != 'read' && $action != 'createaccount' && $user->isBlockedFrom( $this ) ) {
+                       $block = $user->mBlock;
+
+                       // This is from OutputPage::blockedPage
+                       // Copied at r23888 by werdna
+
+                       $id = $user->blockedBy();
+                       $reason = $user->blockedFor();
+                       if ( $reason == '' ) {
+                               $reason = wfMsg( 'blockednoreason' );
+                       }
+                       $ip = wfGetIP();
+
+                       if ( is_numeric( $id ) ) {
+                               $name = User::whoIs( $id );
+                       } else {
+                               $name = $id;
+                       }
+
+                       $link = '[[' . $wgContLang->getNsText( NS_USER ) . ":{$name}|{$name}]]";
+                       $blockid = $block->mId;
+                       $blockExpiry = $user->mBlock->mExpiry;
+                       $blockTimestamp = $wgLang->timeanddate( wfTimestamp( TS_MW, $user->mBlock->mTimestamp ), true );
+                       if ( $blockExpiry == 'infinity' ) {
+                               // Entry in database (table ipblocks) is 'infinity' but 'ipboptions' uses 'infinite' or 'indefinite'
+                               $scBlockExpiryOptions = wfMsg( 'ipboptions' );
+
+                               foreach ( explode( ',', $scBlockExpiryOptions ) as $option ) {
+                                       if ( !strpos( $option, ':' ) )
+                                               continue;
+
+                                       list( $show, $value ) = explode( ':', $option );
+
+                                       if ( $value == 'infinite' || $value == 'indefinite' ) {
+                                               $blockExpiry = $show;
+                                               break;
+                                       }
+                               }
+                       } else {
+                               $blockExpiry = $wgLang->timeanddate( wfTimestamp( TS_MW, $blockExpiry ), true );
+                       }
+
+                       $intended = $user->mBlock->mAddress;
+
+                       $errors[] = array( ( $block->mAuto ? 'autoblockedtext' : 'blockedtext' ), $link, $reason, $ip, $name,
+                               $blockid, $blockExpiry, $intended, $blockTimestamp );
+               }
+
+               return $errors;
+       }
+
+       /**
+        * Can $user perform $action on this page? This is an internal function,
+        * which checks ONLY that previously checked by userCan (i.e. it leaves out
+        * checks on wfReadOnly() and blocks)
+        *
+        * @param $action String action that permission needs to be checked for
+        * @param $user User to check
+        * @param $doExpensiveQueries Bool Set this to false to avoid doing unnecessary queries.
+        * @param $short Bool Set this to true to stop after the first permission error.
+        * @return Array of arrays of the arguments to wfMsg to explain permissions problems.
+        */
+       protected function getUserPermissionsErrorsInternal( $action, $user, $doExpensiveQueries = true, $short = false ) {
+               wfProfileIn( __METHOD__ );
+
+               $errors = array();
+               $checks = array(
+                       'checkQuickPermissions',
+                       'checkPermissionHooks',
+                       'checkSpecialsAndNSPermissions',
+                       'checkCSSandJSPermissions',
+                       'checkPageRestrictions',
+                       'checkCascadingSourcesRestrictions',
+                       'checkActionPermissions',
+                       'checkUserBlock'
+               );
+
+               while( count( $checks ) > 0 &&
+                          !( $short && count( $errors ) > 0 ) ) {
+                       $method = array_shift( $checks );
+                       $errors = $this->$method( $action, $user, $errors, $doExpensiveQueries, $short );
+               }
 
                wfProfileOut( __METHOD__ );
                return $errors;
@@ -1399,7 +1629,9 @@ class Title {
 
        /**
         * Is this title subject to title protection?
-        * @return \type{\mixed} An associative array representing any existent title
+        * Title protection is the one applied against creation of such title.
+        *
+        * @return Mixed An associative array representing any existent title
         *   protection, or false if there's none.
         */
        private function getTitleProtection() {
@@ -1409,75 +1641,78 @@ class Title {
                }
 
                // Can't protect pages that exist.
-               if ($this->exists()) {
+               if ( $this->exists() ) {
                        return false;
                }
 
-               $dbr = wfGetDB( DB_SLAVE );
-               $res = $dbr->select( 'protected_titles', '*',
-                       array( 'pt_namespace' => $this->getNamespace(), 'pt_title' => $this->getDBkey() ),
-                       __METHOD__ );
+               if ( !isset( $this->mTitleProtection ) ) {
+                       $dbr = wfGetDB( DB_SLAVE );
+                       $res = $dbr->select( 'protected_titles', '*',
+                               array( 'pt_namespace' => $this->getNamespace(), 'pt_title' => $this->getDBkey() ),
+                               __METHOD__ );
 
-               if ($row = $dbr->fetchRow( $res )) {
-                       return $row;
-               } else {
-                       return false;
+                       // fetchRow returns false if there are no rows.
+                       $this->mTitleProtection = $dbr->fetchRow( $res );
                }
+               return $this->mTitleProtection;
        }
 
        /**
         * Update the title protection status
-        * @param $create_perm \type{\string} Permission required for creation
-        * @param $reason \type{\string} Reason for protection
-        * @param $expiry \type{\string} Expiry timestamp
+        *
+        * @param $create_perm String Permission required for creation
+        * @param $reason String Reason for protection
+        * @param $expiry String Expiry timestamp
+        * @return boolean true
         */
        public function updateTitleProtection( $create_perm, $reason, $expiry ) {
-               global $wgUser,$wgContLang;
+               global $wgUser, $wgContLang;
 
-               if ($create_perm == implode(',',$this->getRestrictions('create'))
-                       && $expiry == $this->mRestrictionsExpiry['create']) {
+               if ( $create_perm == implode( ',', $this->getRestrictions( 'create' ) )
+                       && $expiry == $this->mRestrictionsExpiry['create'] ) {
                        // No change
                        return true;
                }
 
-               list ($namespace, $title) = array( $this->getNamespace(), $this->getDBkey() );
+               list ( $namespace, $title ) = array( $this->getNamespace(), $this->getDBkey() );
 
                $dbw = wfGetDB( DB_MASTER );
 
-               $encodedExpiry = Block::encodeExpiry($expiry, $dbw );
+               $encodedExpiry = Block::encodeExpiry( $expiry, $dbw );
 
                $expiry_description = '';
                if ( $encodedExpiry != 'infinity' ) {
-                       $expiry_description = ' (' . wfMsgForContent( 'protect-expiring',$wgContLang->timeanddate( $expiry ),
-                               $wgContLang->date( $expiry ) , $wgContLang->time( $expiry ) ).')';
-               }
-               else {
-                       $expiry_description .= ' (' . wfMsgForContent( 'protect-expiry-indefinite' ).')';
+                       $expiry_description = ' (' . wfMsgForContent( 'protect-expiring', $wgContLang->timeanddate( $expiry ),
+                               $wgContLang->date( $expiry ) , $wgContLang->time( $expiry ) ) . ')';
+               } else {
+                       $expiry_description .= ' (' . wfMsgForContent( 'protect-expiry-indefinite' ) . ')';
                }
 
                # Update protection table
-               if ($create_perm != '' ) {
-                       $dbw->replace( 'protected_titles', array(array('pt_namespace', 'pt_title')),
-                               array(
+               if ( $create_perm != '' ) {
+                       $this->mTitleProtection = array(
                                        'pt_namespace' => $namespace,
                                        'pt_title' => $title,
                                        'pt_create_perm' => $create_perm,
-                                       'pt_timestamp' => Block::encodeExpiry(wfTimestampNow(), $dbw),
+                                       'pt_timestamp' => Block::encodeExpiry( wfTimestampNow(), $dbw ),
                                        'pt_expiry' => $encodedExpiry,
                                        'pt_user' => $wgUser->getId(),
                                        'pt_reason' => $reason,
-                               ), __METHOD__
-                       );
+                               );
+                       $dbw->replace( 'protected_titles', array( array( 'pt_namespace', 'pt_title' ) ),
+                               $this->mTitleProtection, __METHOD__     );
                } else {
                        $dbw->delete( 'protected_titles', array( 'pt_namespace' => $namespace,
                                'pt_title' => $title ), __METHOD__ );
+                       $this->mTitleProtection = false;
                }
+
                # Update the protection log
-               if( $dbw->affectedRows() ) {
+               if ( $dbw->affectedRows() ) {
                        $log = new LogPage( 'protect' );
 
-                       if( $create_perm ) {
-                               $params = array("[create=$create_perm] $expiry_description",'');
+                       if ( $create_perm ) {
+                               $params = array( "[create=$create_perm] $expiry_description", '' );
                                $log->addEntry( ( isset( $this->mRestrictions['create'] ) && $this->mRestrictions['create'] ) ? 'modify' : 'protect', $this, trim( $reason ), $params );
                        } else {
                                $log->addEntry( 'unprotect', $this, $reason );
@@ -1493,16 +1728,19 @@ class Title {
        public function deleteTitleProtection() {
                $dbw = wfGetDB( DB_MASTER );
 
-               $dbw->delete( 'protected_titles',
+               $dbw->delete(
+                       'protected_titles',
                        array( 'pt_namespace' => $this->getNamespace(), 'pt_title' => $this->getDBkey() ),
-                       __METHOD__ );
+                       __METHOD__
+               );
+               $this->mTitleProtection = false;
        }
 
        /**
         * Would anybody with sufficient privileges be able to move this page?
         * Some pages just aren't movable.
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool TRUE or FALSE
         */
        public function isMovable() {
                return MWNamespace::isMovable( $this->getNamespace() ) && $this->getInterwiki() == '';
@@ -1510,7 +1748,8 @@ class Title {
 
        /**
         * Can $wgUser read this page?
-        * @return \type{\bool} TRUE or FALSE
+        *
+        * @return Bool
         * @todo fold these checks into userCan()
         */
        public function userCanRead() {
@@ -1519,21 +1758,21 @@ class Title {
                static $useShortcut = null;
 
                # Initialize the $useShortcut boolean, to determine if we can skip quite a bit of code below
-               if( is_null( $useShortcut ) ) {
+               if ( is_null( $useShortcut ) ) {
                        global $wgRevokePermissions;
                        $useShortcut = true;
-                       if( empty( $wgGroupPermissions['*']['read'] ) ) {
+                       if ( empty( $wgGroupPermissions['*']['read'] ) ) {
                                # Not a public wiki, so no shortcut
                                $useShortcut = false;
-                       } elseif( !empty( $wgRevokePermissions ) ) {
+                       } elseif ( !empty( $wgRevokePermissions ) ) {
                                /*
                                 * Iterate through each group with permissions being revoked (key not included since we don't care
                                 * what the group name is), then check if the read permission is being revoked. If it is, then
                                 * we don't use the shortcut below since the user might not be able to read, even though anon
                                 * reading is allowed.
                                 */
-                               foreach( $wgRevokePermissions as $perms ) {
-                                       if( !empty( $perms['read'] ) ) {
+                               foreach ( $wgRevokePermissions as $perms ) {
+                                       if ( !empty( $perms['read'] ) ) {
                                                # We might be removing the read right from the user, so no shortcut
                                                $useShortcut = false;
                                                break;
@@ -1549,10 +1788,11 @@ class Title {
                }
 
                # Shortcut for public wikis, allows skipping quite a bit of code
-               if ( $useShortcut )
+               if ( $useShortcut ) {
                        return true;
+               }
 
-               if( $wgUser->isAllowed( 'read' ) ) {
+               if ( $wgUser->isAllowed( 'read' ) ) {
                        return true;
                } else {
                        global $wgWhitelistRead;
@@ -1561,14 +1801,14 @@ class Title {
                         * Always grant access to the login page.
                         * Even anons need to be able to log in.
                        */
-                       if( $this->isSpecial( 'Userlogin' ) || $this->isSpecial( 'Resetpass' ) ) {
+                       if ( $this->isSpecial( 'Userlogin' ) || $this->isSpecial( 'Resetpass' ) ) {
                                return true;
                        }
 
                        /**
                         * Bail out if there isn't whitelist
                         */
-                       if( !is_array($wgWhitelistRead) ) {
+                       if ( !is_array( $wgWhitelistRead ) ) {
                                return false;
                        }
 
@@ -1578,33 +1818,35 @@ class Title {
                        $name = $this->getPrefixedText();
                        $dbName = $this->getPrefixedDBKey();
                        // Check with and without underscores
-                       if( in_array($name,$wgWhitelistRead,true) || in_array($dbName,$wgWhitelistRead,true) )
+                       if ( in_array( $name, $wgWhitelistRead, true ) || in_array( $dbName, $wgWhitelistRead, true ) )
                                return true;
 
                        /**
                         * Old settings might have the title prefixed with
                         * a colon for main-namespace pages
                         */
-                       if( $this->getNamespace() == NS_MAIN ) {
-                               if( in_array( ':' . $name, $wgWhitelistRead ) )
+                       if ( $this->getNamespace() == NS_MAIN ) {
+                               if ( in_array( ':' . $name, $wgWhitelistRead ) ) {
                                        return true;
+                               }
                        }
 
                        /**
                         * If it's a special page, ditch the subpage bit
                         * and check again
                         */
-                       if( $this->getNamespace() == NS_SPECIAL ) {
+                       if ( $this->getNamespace() == NS_SPECIAL ) {
                                $name = $this->getDBkey();
-                               list( $name, /* $subpage */) = SpecialPage::resolveAliasWithSubpage( $name );
+                               list( $name, /* $subpage */ ) = SpecialPage::resolveAliasWithSubpage( $name );
                                if ( $name === false ) {
                                        # Invalid special page, but we show standard login required message
                                        return false;
                                }
 
                                $pure = SpecialPage::getTitleFor( $name )->getPrefixedText();
-                               if( in_array( $pure, $wgWhitelistRead, true ) )
+                               if ( in_array( $pure, $wgWhitelistRead, true ) ) {
                                        return true;
+                               }
                        }
 
                }
@@ -1613,7 +1855,8 @@ class Title {
 
        /**
         * Is this a talk page of some sort?
-        * @return \type{\bool} TRUE or FALSE
+        *
+        * @return Bool
         */
        public function isTalkPage() {
                return MWNamespace::isTalk( $this->getNamespace() );
@@ -1621,7 +1864,8 @@ class Title {
 
        /**
         * Is this a subpage?
-        * @return \type{\bool} TRUE or FALSE
+        *
+        * @return Bool
         */
        public function isSubpage() {
                return MWNamespace::hasSubpages( $this->mNamespace )
@@ -1631,10 +1875,11 @@ class Title {
 
        /**
         * Does this have subpages?  (Warning, usually requires an extra DB query.)
-        * @return \type{\bool} TRUE or FALSE
+        *
+        * @return Bool
         */
        public function hasSubpages() {
-               if( !MWNamespace::hasSubpages( $this->mNamespace ) ) {
+               if ( !MWNamespace::hasSubpages( $this->mNamespace ) ) {
                        # Duh
                        return false;
                }
@@ -1643,32 +1888,36 @@ class Title {
                # alone to cache the result.  There's no point in having it hanging
                # around uninitialized in every Title object; therefore we only add it
                # if needed and don't declare it statically.
-               if( isset( $this->mHasSubpages ) ) {
+               if ( isset( $this->mHasSubpages ) ) {
                        return $this->mHasSubpages;
                }
 
                $subpages = $this->getSubpages( 1 );
-               if( $subpages instanceof TitleArray )
+               if ( $subpages instanceof TitleArray ) {
                        return $this->mHasSubpages = (bool)$subpages->count();
+               }
                return $this->mHasSubpages = false;
        }
 
        /**
         * Get all subpages of this page.
-        * @param $limit Maximum number of subpages to fetch; -1 for no limit
+        *
+        * @param $limit Int maximum number of subpages to fetch; -1 for no limit
         * @return mixed TitleArray, or empty array if this page's namespace
         *  doesn't allow subpages
         */
        public function getSubpages( $limit = -1 ) {
-               if( !MWNamespace::hasSubpages( $this->getNamespace() ) )
+               if ( !MWNamespace::hasSubpages( $this->getNamespace() ) ) {
                        return array();
+               }
 
                $dbr = wfGetDB( DB_SLAVE );
                $conds['page_namespace'] = $this->getNamespace();
                $conds[] = 'page_title ' . $dbr->buildLike( $this->getDBkey() . '/', $dbr->anyString() );
                $options = array();
-               if( $limit > -1 )
+               if ( $limit > -1 ) {
                        $options['LIMIT'] = $limit;
+               }
                return $this->mSubpages = TitleArray::newFromResult(
                        $dbr->select( 'page',
                                array( 'page_id', 'page_namespace', 'page_title', 'page_is_redirect' ),
@@ -1683,7 +1932,7 @@ class Title {
         * Could this page contain custom CSS or JavaScript, based
         * on the title?
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool
         */
        public function isCssOrJsPage() {
                return $this->mNamespace == NS_MEDIAWIKI
@@ -1692,75 +1941,81 @@ class Title {
 
        /**
         * Is this a .css or .js subpage of a user page?
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool
         */
        public function isCssJsSubpage() {
-               return ( NS_USER == $this->mNamespace and preg_match("/\\/.*\\.(?:css|js)$/", $this->mTextform ) );
+               return ( NS_USER == $this->mNamespace and preg_match( "/\\/.*\\.(?:css|js)$/", $this->mTextform ) );
        }
+
        /**
         * Is this a *valid* .css or .js subpage of a user page?
-        * Check that the corresponding skin exists
-        * @return \type{\bool} TRUE or FALSE
+        *
+        * @return Bool
+        * @deprecated
         */
        public function isValidCssJsSubpage() {
-               if ( $this->isCssJsSubpage() ) {
-                       $skinNames = Skin::getSkinNames();
-                       return array_key_exists( $this->getSkinFromCssJsSubpage(), $skinNames );
-               } else {
-                       return false;
-               }
+               return $this->isCssJsSubpage();
        }
+
        /**
         * Trim down a .css or .js subpage title to get the corresponding skin name
+        *
+        * @return string containing skin name from .css or .js subpage title
         */
        public function getSkinFromCssJsSubpage() {
                $subpage = explode( '/', $this->mTextform );
                $subpage = $subpage[ count( $subpage ) - 1 ];
                return( str_replace( array( '.css', '.js' ), array( '', '' ), $subpage ) );
        }
+
        /**
         * Is this a .css subpage of a user page?
-        * @return \type{\bool} TRUE or FALSE
+        *
+        * @return Bool
         */
        public function isCssSubpage() {
-               return ( NS_USER == $this->mNamespace && preg_match("/\\/.*\\.css$/", $this->mTextform ) );
+               return ( NS_USER == $this->mNamespace && preg_match( "/\\/.*\\.css$/", $this->mTextform ) );
        }
+
        /**
         * Is this a .js subpage of a user page?
-        * @return \type{\bool} TRUE or FALSE
+        *
+        * @return Bool
         */
        public function isJsSubpage() {
-               return ( NS_USER == $this->mNamespace && preg_match("/\\/.*\\.js$/", $this->mTextform ) );
+               return ( NS_USER == $this->mNamespace && preg_match( "/\\/.*\\.js$/", $this->mTextform ) );
        }
+
        /**
         * Protect css subpages of user pages: can $wgUser edit
         * this page?
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool
         * @todo XXX: this might be better using restrictions
         */
        public function userCanEditCssSubpage() {
                global $wgUser;
-               return ( ( $wgUser->isAllowed('editusercssjs') && $wgUser->isAllowed('editusercss') )
-                       || preg_match('/^'.preg_quote($wgUser->getName(), '/').'\//', $this->mTextform) );
+               return ( ( $wgUser->isAllowed( 'editusercssjs' ) && $wgUser->isAllowed( 'editusercss' ) )
+                       || preg_match( '/^' . preg_quote( $wgUser->getName(), '/' ) . '\//', $this->mTextform ) );
        }
+
        /**
         * Protect js subpages of user pages: can $wgUser edit
         * this page?
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool
         * @todo XXX: this might be better using restrictions
         */
        public function userCanEditJsSubpage() {
                global $wgUser;
-               return ( ( $wgUser->isAllowed('editusercssjs') && $wgUser->isAllowed('edituserjs') )
-                      || preg_match('/^'.preg_quote($wgUser->getName(), '/').'\//', $this->mTextform) );
+               return ( ( $wgUser->isAllowed( 'editusercssjs' ) && $wgUser->isAllowed( 'edituserjs' ) )
+                      || preg_match( '/^' . preg_quote( $wgUser->getName(), '/' ) . '\//', $this->mTextform ) );
        }
 
        /**
         * Cascading protection: Return true if cascading restrictions apply to this page, false if not.
         *
-        * @return \type{\bool} If the page is subject to cascading restrictions.
+        * @return Bool If the page is subject to cascading restrictions.
         */
        public function isCascadeProtected() {
                list( $sources, /* $restrictions */ ) = $this->getCascadeProtectionSources( false );
@@ -1770,17 +2025,20 @@ class Title {
        /**
         * Cascading protection: Get the source of any cascading restrictions on this page.
         *
-        * @param $get_pages \type{\bool} Whether or not to retrieve the actual pages that the restrictions have come from.
-        * @return \type{\arrayof{mixed title array, restriction array}} Array of the Title objects of the pages from
-        *         which cascading restrictions have come, false for none, or true if such restrictions exist, but $get_pages was not set.
-        *         The restriction array is an array of each type, each of which contains an array of unique groups.
-        */
-       public function getCascadeProtectionSources( $get_pages = true ) {
+        * @param $getPages Bool Whether or not to retrieve the actual pages
+        *        that the restrictions have come from.
+        * @return Mixed Array of the Title
+        *         objects of the pages from which cascading restrictions have come,
+        *         false for none, or true if such restrictions exist, but $getPages was not set.
+        *         The restriction array is an array of each type, each of which contains a
+        *         array of unique groups.
+        */
+       public function getCascadeProtectionSources( $getPages = true ) {
                $pagerestrictions = array();
 
-               if ( isset( $this->mCascadeSources ) && $get_pages ) {
+               if ( isset( $this->mCascadeSources ) && $getPages ) {
                        return array( $this->mCascadeSources, $this->mCascadingRestrictions );
-               } else if ( isset( $this->mHasCascadingRestrictions ) && !$get_pages ) {
+               } else if ( isset( $this->mHasCascadingRestrictions ) && !$getPages ) {
                        return array( $this->mHasCascadingRestrictions, $pagerestrictions );
                }
 
@@ -1789,22 +2047,25 @@ class Title {
                $dbr = wfGetDB( DB_SLAVE );
 
                if ( $this->getNamespace() == NS_FILE ) {
-                       $tables = array ('imagelinks', 'page_restrictions');
+                       $tables = array( 'imagelinks', 'page_restrictions' );
                        $where_clauses = array(
                                'il_to' => $this->getDBkey(),
                                'il_from=pr_page',
-                               'pr_cascade' => 1 );
+                               'pr_cascade' => 1
+                       );
                } else {
-                       $tables = array ('templatelinks', 'page_restrictions');
+                       $tables = array( 'templatelinks', 'page_restrictions' );
                        $where_clauses = array(
                                'tl_namespace' => $this->getNamespace(),
                                'tl_title' => $this->getDBkey(),
                                'tl_from=pr_page',
-                               'pr_cascade' => 1 );
+                               'pr_cascade' => 1
+                       );
                }
 
-               if ( $get_pages ) {
-                       $cols = array('pr_page', 'page_namespace', 'page_title', 'pr_expiry', 'pr_type', 'pr_level' );
+               if ( $getPages ) {
+                       $cols = array( 'pr_page', 'page_namespace', 'page_title',
+                                                  'pr_expiry', 'pr_type', 'pr_level' );
                        $where_clauses[] = 'page_id=pr_page';
                        $tables[] = 'page';
                } else {
@@ -1813,18 +2074,18 @@ class Title {
 
                $res = $dbr->select( $tables, $cols, $where_clauses, __METHOD__ );
 
-               $sources = $get_pages ? array() : false;
+               $sources = $getPages ? array() : false;
                $now = wfTimestampNow();
                $purgeExpired = false;
 
-               foreach( $res as $row ) {
+               foreach ( $res as $row ) {
                        $expiry = Block::decodeExpiry( $row->pr_expiry );
-                       if( $expiry > $now ) {
-                               if ($get_pages) {
+                       if ( $expiry > $now ) {
+                               if ( $getPages ) {
                                        $page_id = $row->pr_page;
                                        $page_ns = $row->page_namespace;
                                        $page_title = $row->page_title;
-                                       $sources[$page_id] = Title::makeTitle($page_ns, $page_title);
+                                       $sources[$page_id] = Title::makeTitle( $page_ns, $page_title );
                                        # Add groups needed for each restriction type if its not already there
                                        # Make sure this restriction type still exists
 
@@ -1832,9 +2093,9 @@ class Title {
                                                $pagerestrictions[$row->pr_type] = array();
                                        }
 
-                                       if ( isset($pagerestrictions[$row->pr_type]) &&
-                                                       !in_array($row->pr_level, $pagerestrictions[$row->pr_type]) ) {
-                                               $pagerestrictions[$row->pr_type][]=$row->pr_level;
+                                       if ( isset( $pagerestrictions[$row->pr_type] ) &&
+                                                !in_array( $row->pr_level, $pagerestrictions[$row->pr_type] ) ) {
+                                               $pagerestrictions[$row->pr_type][] = $row->pr_level;
                                        }
                                } else {
                                        $sources = true;
@@ -1844,23 +2105,29 @@ class Title {
                                $purgeExpired = true;
                        }
                }
-               if( $purgeExpired ) {
+               if ( $purgeExpired ) {
                        Title::purgeExpiredRestrictions();
                }
 
                wfProfileOut( __METHOD__ );
 
-               if ( $get_pages ) {
+               if ( $getPages ) {
                        $this->mCascadeSources = $sources;
                        $this->mCascadingRestrictions = $pagerestrictions;
                } else {
                        $this->mHasCascadingRestrictions = $sources;
                }
+
                return array( $sources, $pagerestrictions );
        }
 
+       /**
+        * Returns cascading restrictions for the current article
+        *
+        * @return Boolean
+        */
        function areRestrictionsCascading() {
-               if (!$this->mRestrictionsLoaded) {
+               if ( !$this->mRestrictionsLoaded ) {
                        $this->loadRestrictions();
                }
 
@@ -1869,27 +2136,38 @@ class Title {
 
        /**
         * Loads a string into mRestrictions array
-        * @param $res \type{Resource} restrictions as an SQL result.
+        *
+        * @param $res Resource restrictions as an SQL result.
+        * @param $oldFashionedRestrictions string comma-separated list of page
+        *        restrictions from page table (pre 1.10)
         */
        private function loadRestrictionsFromResultWrapper( $res, $oldFashionedRestrictions = null ) {
                $rows = array();
-               $dbr = wfGetDB( DB_SLAVE );
 
-               while( $row = $dbr->fetchObject( $res ) ) {
+               foreach ( $res as $row ) {
                        $rows[] = $row;
                }
 
                $this->loadRestrictionsFromRows( $rows, $oldFashionedRestrictions );
        }
 
+       /**
+        * Compiles list of active page restrictions from both page table (pre 1.10)
+        * and page_restrictions table for this existing page.
+        * Public for usage by LiquidThreads.
+        *
+        * @param $rows array of db result objects
+        * @param $oldFashionedRestrictions string comma-separated list of page
+        *        restrictions from page table (pre 1.10)
+        */
        public function loadRestrictionsFromRows( $rows, $oldFashionedRestrictions = null ) {
                $dbr = wfGetDB( DB_SLAVE );
 
                $restrictionTypes = $this->getRestrictionTypes();
 
-               foreach( $restrictionTypes as $type ){
+               foreach ( $restrictionTypes as $type ) {
                        $this->mRestrictions[$type] = array();
-                       $this->mRestrictionsExpiry[$type] = Block::decodeExpiry('');
+                       $this->mRestrictionsExpiry[$type] = Block::decodeExpiry( '' );
                }
 
                $this->mCascadeRestriction = false;
@@ -1901,11 +2179,11 @@ class Title {
                                array( 'page_id' => $this->getArticleId() ), __METHOD__ );
                }
 
-               if ($oldFashionedRestrictions != '') {
+               if ( $oldFashionedRestrictions != '' ) {
 
-                       foreach( explode( ':', trim( $oldFashionedRestrictions ) ) as $restrict ) {
+                       foreach ( explode( ':', trim( $oldFashionedRestrictions ) ) as $restrict ) {
                                $temp = explode( '=', trim( $restrict ) );
-                               if(count($temp) == 1) {
+                               if ( count( $temp ) == 1 ) {
                                        // old old format should be treated as edit/move restriction
                                        $this->mRestrictions['edit'] = explode( ',', trim( $temp[0] ) );
                                        $this->mRestrictions['move'] = explode( ',', trim( $temp[0] ) );
@@ -1918,17 +2196,17 @@ class Title {
 
                }
 
-               if( count($rows) ) {
+               if ( count( $rows ) ) {
                        # Current system - load second to make them override.
                        $now = wfTimestampNow();
                        $purgeExpired = false;
 
-                       foreach( $rows as $row ) {
+                       foreach ( $rows as $row ) {
                                # Cycle through all the restrictions.
 
                                // Don't take care of restrictions types that aren't allowed
 
-                               if( !in_array( $row->pr_type, $restrictionTypes ) )
+                               if ( !in_array( $row->pr_type, $restrictionTypes ) )
                                        continue;
 
                                // This code should be refactored, now that it's being used more generally,
@@ -1947,7 +2225,7 @@ class Title {
                                }
                        }
 
-                       if( $purgeExpired ) {
+                       if ( $purgeExpired ) {
                                Title::purgeExpiredRestrictions();
                        }
                }
@@ -1957,34 +2235,36 @@ class Title {
 
        /**
         * Load restrictions from the page_restrictions table
+        *
+        * @param $oldFashionedRestrictions string comma-separated list of page
+        *        restrictions from page table (pre 1.10)
         */
        public function loadRestrictions( $oldFashionedRestrictions = null ) {
-               if( !$this->mRestrictionsLoaded ) {
-                       if ($this->exists()) {
+               if ( !$this->mRestrictionsLoaded ) {
+                       if ( $this->exists() ) {
                                $dbr = wfGetDB( DB_SLAVE );
 
                                $res = $dbr->select( 'page_restrictions', '*',
-                                       array ( 'pr_page' => $this->getArticleId() ), __METHOD__ );
+                                       array( 'pr_page' => $this->getArticleId() ), __METHOD__ );
 
                                $this->loadRestrictionsFromResultWrapper( $res, $oldFashionedRestrictions );
                        } else {
                                $title_protection = $this->getTitleProtection();
 
-                               if (is_array($title_protection)) {
-                                       extract($title_protection);
-
+                               if ( $title_protection ) {
                                        $now = wfTimestampNow();
-                                       $expiry = Block::decodeExpiry($pt_expiry);
+                                       $expiry = Block::decodeExpiry( $title_protection['pt_expiry'] );
 
-                                       if (!$expiry || $expiry > $now) {
+                                       if ( !$expiry || $expiry > $now ) {
                                                // Apply the restrictions
                                                $this->mRestrictionsExpiry['create'] = $expiry;
-                                               $this->mRestrictions['create'] = explode(',', trim($pt_create_perm) );
+                                               $this->mRestrictions['create'] = explode( ',', trim( $title_protection['pt_create_perm'] ) );
                                        } else { // Get rid of the old restrictions
                                                Title::purgeExpiredRestrictions();
+                                               $this->mTitleProtection = false;
                                        }
                                } else {
-                                       $this->mRestrictionsExpiry['create'] = Block::decodeExpiry('');
+                                       $this->mRestrictionsExpiry['create'] = Block::decodeExpiry( '' );
                                }
                                $this->mRestrictionsLoaded = true;
                        }
@@ -1996,23 +2276,27 @@ class Title {
         */
        static function purgeExpiredRestrictions() {
                $dbw = wfGetDB( DB_MASTER );
-               $dbw->delete( 'page_restrictions',
+               $dbw->delete(
+                       'page_restrictions',
                        array( 'pr_expiry < ' . $dbw->addQuotes( $dbw->timestamp() ) ),
-                       __METHOD__ );
+                       __METHOD__
+               );
 
-               $dbw->delete( 'protected_titles',
+               $dbw->delete(
+                       'protected_titles',
                        array( 'pt_expiry < ' . $dbw->addQuotes( $dbw->timestamp() ) ),
-                       __METHOD__ );
+                       __METHOD__
+               );
        }
 
        /**
         * Accessor/initialisation for mRestrictions
         *
-        * @param $action \type{\string} action that permission needs to be checked for
-        * @return \type{\arrayof{\string}} the array of groups allowed to edit this article
+        * @param $action String action that permission needs to be checked for
+        * @return Array of Strings the array of groups allowed to edit this article
         */
        public function getRestrictions( $action ) {
-               if( !$this->mRestrictionsLoaded ) {
+               if ( !$this->mRestrictionsLoaded ) {
                        $this->loadRestrictions();
                }
                return isset( $this->mRestrictions[$action] )
@@ -2022,11 +2306,12 @@ class Title {
 
        /**
         * Get the expiry time for the restriction against a given action
+        *
         * @return 14-char timestamp, or 'infinity' if the page is protected forever
-        * or not protected at all, or false if the action is not recognised.
+        *      or not protected at all, or false if the action is not recognised.
         */
        public function getRestrictionExpiry( $action ) {
-               if( !$this->mRestrictionsLoaded ) {
+               if ( !$this->mRestrictionsLoaded ) {
                        $this->loadRestrictions();
                }
                return isset( $this->mRestrictionsExpiry[$action] ) ? $this->mRestrictionsExpiry[$action] : false;
@@ -2034,10 +2319,11 @@ class Title {
 
        /**
         * Is there a version of this page in the deletion archive?
-        * @return \type{\int} the number of archived revisions
+        *
+        * @return Int the number of archived revisions
         */
        public function isDeleted() {
-               if( $this->getNamespace() < 0 ) {
+               if ( $this->getNamespace() < 0 ) {
                        $n = 0;
                } else {
                        $dbr = wfGetDB( DB_SLAVE );
@@ -2045,7 +2331,7 @@ class Title {
                                array( 'ar_namespace' => $this->getNamespace(), 'ar_title' => $this->getDBkey() ),
                                __METHOD__
                        );
-                       if( $this->getNamespace() == NS_FILE ) {
+                       if ( $this->getNamespace() == NS_FILE ) {
                                $n += $dbr->selectField( 'filearchive', 'COUNT(*)',
                                        array( 'fa_name' => $this->getDBkey() ),
                                        __METHOD__
@@ -2057,10 +2343,11 @@ class Title {
 
        /**
         * Is there a version of this page in the deletion archive?
-        * @return bool
+        *
+        * @return Boolean
         */
        public function isDeletedQuick() {
-               if( $this->getNamespace() < 0 ) {
+               if ( $this->getNamespace() < 0 ) {
                        return false;
                }
                $dbr = wfGetDB( DB_SLAVE );
@@ -2068,7 +2355,7 @@ class Title {
                        array( 'ar_namespace' => $this->getNamespace(), 'ar_title' => $this->getDBkey() ),
                        __METHOD__
                );
-               if( !$deleted && $this->getNamespace() == NS_FILE ) {
+               if ( !$deleted && $this->getNamespace() == NS_FILE ) {
                        $deleted = (bool)$dbr->selectField( 'filearchive', '1',
                                array( 'fa_name' => $this->getDBkey() ),
                                __METHOD__
@@ -2080,22 +2367,23 @@ class Title {
        /**
         * Get the article ID for this Title from the link cache,
         * adding it if necessary
-        * @param $flags \type{\int} a bit field; may be GAID_FOR_UPDATE to select
+        *
+        * @param $flags Int a bit field; may be Title::GAID_FOR_UPDATE to select
         *  for update
-        * @return \type{\int} the ID
+        * @return Int the ID
         */
        public function getArticleID( $flags = 0 ) {
-               if( $this->getNamespace() < 0 ) {
+               if ( $this->getNamespace() < 0 ) {
                        return $this->mArticleID = 0;
                }
                $linkCache = LinkCache::singleton();
-               if( $flags & GAID_FOR_UPDATE ) {
+               if ( $flags & self::GAID_FOR_UPDATE ) {
                        $oldUpdate = $linkCache->forUpdate( true );
                        $linkCache->clearLink( $this );
                        $this->mArticleID = $linkCache->addLinkObj( $this );
                        $linkCache->forUpdate( $oldUpdate );
                } else {
-                       if( -1 == $this->mArticleID ) {
+                       if ( -1 == $this->mArticleID ) {
                                $this->mArticleID = $linkCache->addLinkObj( $this );
                        }
                }
@@ -2105,14 +2393,16 @@ class Title {
        /**
         * Is this an article that is a redirect page?
         * Uses link cache, adding it if necessary
-        * @param $flags \type{\int} a bit field; may be GAID_FOR_UPDATE to select for update
-        * @return \type{\bool}
+        *
+        * @param $flags Int a bit field; may be Title::GAID_FOR_UPDATE to select for update
+        * @return Bool
         */
        public function isRedirect( $flags = 0 ) {
-               if( !is_null($this->mRedirect) )
+               if ( !is_null( $this->mRedirect ) ) {
                        return $this->mRedirect;
+               }
                # Calling getArticleID() loads the field from cache as needed
-               if( !$this->getArticleID($flags) ) {
+               if ( !$this->getArticleID( $flags ) ) {
                        return $this->mRedirect = false;
                }
                $linkCache = LinkCache::singleton();
@@ -2124,14 +2414,16 @@ class Title {
        /**
         * What is the length of this page?
         * Uses link cache, adding it if necessary
-        * @param $flags \type{\int} a bit field; may be GAID_FOR_UPDATE to select for update
-        * @return \type{\bool}
+        *
+        * @param $flags Int a bit field; may be Title::GAID_FOR_UPDATE to select for update
+        * @return Int
         */
        public function getLength( $flags = 0 ) {
-               if( $this->mLength != -1 )
+               if ( $this->mLength != -1 ) {
                        return $this->mLength;
+               }
                # Calling getArticleID() loads the field from cache as needed
-               if( !$this->getArticleID($flags) ) {
+               if ( !$this->getArticleID( $flags ) ) {
                        return $this->mLength = 0;
                }
                $linkCache = LinkCache::singleton();
@@ -2142,15 +2434,21 @@ class Title {
 
        /**
         * What is the page_latest field for this page?
-        * @param $flags \type{\int} a bit field; may be GAID_FOR_UPDATE to select for update
-        * @return \type{\int} or false if the page doesn't exist
+        *
+        * @param $flags Int a bit field; may be Title::GAID_FOR_UPDATE to select for update
+        * @return Int or 0 if the page doesn't exist
         */
        public function getLatestRevID( $flags = 0 ) {
-               if( $this->mLatestID !== false )
-                       return $this->mLatestID;
+               if ( $this->mLatestID !== false ) {
+                       return intval( $this->mLatestID );
+               }
+               # Calling getArticleID() loads the field from cache as needed
+               if ( !$this->getArticleID( $flags ) ) {
+                       return $this->mLatestID = 0;
+               }
+               $linkCache = LinkCache::singleton();
+               $this->mLatestID = intval( $linkCache->getGoodLinkFieldObj( $this, 'revision' ) );
 
-               $db = ($flags & GAID_FOR_UPDATE) ? wfGetDB(DB_MASTER) : wfGetDB(DB_SLAVE);
-               $this->mLatestID = $db->selectField( 'page', 'page_latest', $this->pageCond(), __METHOD__ );
                return $this->mLatestID;
        }
 
@@ -2158,32 +2456,40 @@ class Title {
         * This clears some fields in this object, and clears any associated
         * keys in the "bad links" section of the link cache.
         *
-        * - This is called from Article::insertNewArticle() to allow
+        * - This is called from Article::doEdit() and Article::insertOn() to allow
         * loading of the new page_id. It's also called from
         * Article::doDeleteArticle()
         *
-        * @param $newid \type{\int} the new Article ID
+        * @param $newid Int the new Article ID
         */
        public function resetArticleID( $newid ) {
                $linkCache = LinkCache::singleton();
                $linkCache->clearBadLink( $this->getPrefixedDBkey() );
 
-               if ( $newid === false ) { $this->mArticleID = -1; }
-               else { $this->mArticleID = intval( $newid ); }
+               if ( $newid === false ) {
+                       $this->mArticleID = -1;
+               } else {
+                       $this->mArticleID = intval( $newid );
+               }
                $this->mRestrictionsLoaded = false;
                $this->mRestrictions = array();
+               $this->mRedirect = null;
+               $this->mLength = -1;
+               $this->mLatestID = false;
        }
 
        /**
         * Updates page_touched for this page; called from LinksUpdate.php
-        * @return \type{\bool} true if the update succeded
+        *
+        * @return Bool true if the update succeded
         */
        public function invalidateCache() {
-               if( wfReadOnly() ) {
+               if ( wfReadOnly() ) {
                        return;
                }
                $dbw = wfGetDB( DB_MASTER );
-               $success = $dbw->update( 'page',
+               $success = $dbw->update(
+                       'page',
                        array( 'page_touched' => $dbw->timestamp() ),
                        $this->pageCond(),
                        __METHOD__
@@ -2196,13 +2502,13 @@ class Title {
         * Prefix some arbitrary text with the namespace or interwiki prefix
         * of this object
         *
-        * @param $name \type{\string} the text
-        * @return \type{\string} the prefixed text
+        * @param $name String the text
+        * @return String the prefixed text
         * @private
         */
        /* private */ function prefix( $name ) {
                $p = '';
-               if ( '' != $this->mInterwiki ) {
+               if ( $this->mInterwiki != '' ) {
                        $p = $this->mInterwiki . ':';
                }
                if ( 0 != $this->mNamespace ) {
@@ -2211,12 +2517,16 @@ class Title {
                return $p . $name;
        }
 
-       // Returns a simple regex that will match on characters and sequences invalid in titles.
-       //  Note that this doesn't pick up many things that could be wrong with titles, but that
-       //  replacing this regex with something valid will make many titles valid.
+       /**
+        * Returns a simple regex that will match on characters and sequences invalid in titles.
+        * Note that this doesn't pick up many things that could be wrong with titles, but that
+        * replacing this regex with something valid will make many titles valid.
+        *
+        * @return string regex string
+        */
        static function getTitleInvalidRegex() {
                static $rxTc = false;
-               if( !$rxTc ) {
+               if ( !$rxTc ) {
                        # Matching titles will be held as illegal.
                        $rxTc = '/' .
                                # Any character not allowed is forbidden...
@@ -2235,15 +2545,20 @@ class Title {
        }
 
        /**
-        * Capitalize a text if it belongs to a namespace that capitalizes
+        * Capitalize a text string for a title if it belongs to a namespace that capitalizes
+        *
+        * @param $text string containing title to capitalize
+        * @param $ns int namespace index, defaults to NS_MAIN
+        * @return String containing capitalized title
         */
        public static function capitalize( $text, $ns = NS_MAIN ) {
                global $wgContLang;
 
-               if ( MWNamespace::isCapitalized( $ns ) )
+               if ( MWNamespace::isCapitalized( $ns ) ) {
                        return $wgContLang->ucfirst( $text );
-               else
+               } else {
                        return $text;
+               }
        }
 
        /**
@@ -2254,7 +2569,8 @@ class Title {
         * removes illegal characters, splits off the interwiki and
         * namespace prefixes, sets the other forms, and canonicalizes
         * everything.
-        * @return \type{\bool} true on success
+        *
+        * @return Bool true on success
         */
        private function secureAndSplit() {
                global $wgContLang, $wgLocalInterwiki;
@@ -2280,11 +2596,11 @@ class Title {
                $dbkey = preg_replace( '/[ _\xA0\x{1680}\x{180E}\x{2000}-\x{200A}\x{2028}\x{2029}\x{202F}\x{205F}\x{3000}]+/u', '_', $dbkey );
                $dbkey = trim( $dbkey, '_' );
 
-               if ( '' == $dbkey ) {
+               if ( $dbkey == '' ) {
                        return false;
                }
 
-               if( false !== strpos( $dbkey, UTF8_REPLACEMENT ) ) {
+               if ( false !== strpos( $dbkey, UTF8_REPLACEMENT ) ) {
                        # Contained illegal UTF-8 sequences or forbidden Unicode chars.
                        return false;
                }
@@ -2293,7 +2609,7 @@ class Title {
 
                # Initial colon indicates main namespace rather than specified default
                # but should not create invalid {ns,title} pairs such as {0,Project:Foo}
-               if ( ':' == $dbkey{0} ) {
+               if ( ':' == $dbkey { 0 } ) {
                        $this->mNamespace = NS_MAIN;
                        $dbkey = substr( $dbkey, 1 ); # remove the colon but continue processing
                        $dbkey = trim( $dbkey, '_' ); # remove any subsequent whitespace
@@ -2306,19 +2622,20 @@ class Title {
                        $m = array();
                        if ( preg_match( $prefixRegexp, $dbkey, $m ) ) {
                                $p = $m[1];
-                               if ( $ns = $wgContLang->getNsIndex( $p ) ) {
+                               if ( ( $ns = $wgContLang->getNsIndex( $p ) ) !== false ) {
                                        # Ordinary namespace
                                        $dbkey = $m[2];
                                        $this->mNamespace = $ns;
                                        # For Talk:X pages, check if X has a "namespace" prefix
-                                       if( $ns == NS_TALK && preg_match( $prefixRegexp, $dbkey, $x ) ) {
-                                               if( $wgContLang->getNsIndex( $x[1] ) )
+                                       if ( $ns == NS_TALK && preg_match( $prefixRegexp, $dbkey, $x ) ) {
+                                               if ( $wgContLang->getNsIndex( $x[1] ) ) {
                                                        return false; # Disallow Talk:File:x type titles...
-                                               else if( Interwiki::isValidInterwiki( $x[1] ) )
+                                               } else if ( Interwiki::isValidInterwiki( $x[1] ) ) {
                                                        return false; # Disallow Talk:Interwiki:x type titles...
+                                               }
                                        }
-                               } elseif( Interwiki::isValidInterwiki( $p ) ) {
-                                       if( !$firstPass ) {
+                               } elseif ( Interwiki::isValidInterwiki( $p ) ) {
+                                       if ( !$firstPass ) {
                                                # Can't make a local interwiki link to an interwiki link.
                                                # That's just crazy!
                                                return false;
@@ -2329,8 +2646,10 @@ class Title {
                                        $this->mInterwiki = $wgContLang->lc( $p );
 
                                        # Redundant interwiki prefix to the local wiki
-                                       if ( 0 == strcasecmp( $this->mInterwiki, $wgLocalInterwiki ) ) {
-                                               if( $dbkey == '' ) {
+                                       if ( $wgLocalInterwiki !== false
+                                               && 0 == strcasecmp( $this->mInterwiki, $wgLocalInterwiki ) ) 
+                                       {
+                                               if ( $dbkey == '' ) {
                                                        # Can't have an empty self-link
                                                        return false;
                                                }
@@ -2351,16 +2670,16 @@ class Title {
                                # then let the colon expression be part of the title.
                        }
                        break;
-               } while( true );
+               } while ( true );
 
                # We already know that some pages won't be in the database!
                #
-               if ( '' != $this->mInterwiki || NS_SPECIAL == $this->mNamespace ) {
+               if ( $this->mInterwiki != '' || NS_SPECIAL == $this->mNamespace ) {
                        $this->mArticleID = 0;
                }
                $fragment = strstr( $dbkey, '#' );
                if ( false !== $fragment ) {
-                       $this->setFragment( $fragment );
+                       $this->setFragment( preg_replace( '/^#_*/', '#', $fragment ) );
                        $dbkey = substr( $dbkey, 0, strlen( $dbkey ) - strlen( $fragment ) );
                        # remove whitespace again: prevents "Foo_bar_#"
                        # becoming "Foo_bar_"
@@ -2369,7 +2688,7 @@ class Title {
 
                # Reject illegal characters.
                #
-               if( preg_match( $rxTc, $dbkey ) ) {
+               if ( preg_match( $rxTc, $dbkey ) ) {
                        return false;
                }
 
@@ -2393,7 +2712,7 @@ class Title {
                /**
                 * Magic tilde sequences? Nu-uh!
                 */
-               if( strpos( $dbkey, '~~~' ) !== false ) {
+               if ( strpos( $dbkey, '~~~' ) !== false ) {
                        return false;
                }
 
@@ -2419,7 +2738,7 @@ class Title {
                 * site might be case-sensitive.
                 */
                $this->mUserCaseDBKey = $dbkey;
-               if(  $this->mInterwiki == '') {
+               if ( $this->mInterwiki == '' ) {
                        $dbkey = self::capitalize( $dbkey, $this->mNamespace );
                }
 
@@ -2428,7 +2747,7 @@ class Title {
                 * "empty" local links can only be self-links
                 * with a fragment identifier.
                 */
-               if( $dbkey == '' &&
+               if ( $dbkey == '' &&
                        $this->mInterwiki == '' &&
                        $this->mNamespace != NS_MAIN ) {
                        return false;
@@ -2439,10 +2758,10 @@ class Title {
                // there are numerous ways to present the same IP. Having sp:contribs scan
                // them all is silly and having some show the edits and others not is
                // inconsistent. Same for talk/userpages. Keep them normalized instead.
-               $dbkey = ($this->mNamespace == NS_USER || $this->mNamespace == NS_USER_TALK) ?
+               $dbkey = ( $this->mNamespace == NS_USER || $this->mNamespace == NS_USER_TALK ) ?
                        IP::sanitizeIP( $dbkey ) : $dbkey;
                // Any remaining initial :s are illegal.
-               if ( $dbkey !== '' && ':' == $dbkey{0} ) {
+               if ( $dbkey !== '' && ':' == $dbkey { 0 } ) {
                        return false;
                }
 
@@ -2463,7 +2782,7 @@ class Title {
         * Deprecated for public use, use Title::makeTitle() with fragment parameter.
         * Still in active use privately.
         *
-        * @param $fragment \type{\string} text
+        * @param $fragment String text
         */
        public function setFragment( $fragment ) {
                $this->mFragment = str_replace( '_', ' ', substr( $fragment, 1 ) );
@@ -2471,7 +2790,8 @@ class Title {
 
        /**
         * Get a Title object associated with the talk page of this article
-        * @return \type{Title} the object for the talk page
+        *
+        * @return Title the object for the talk page
         */
        public function getTalkPage() {
                return Title::makeTitle( MWNamespace::getTalk( $this->getNamespace() ), $this->getDBkey() );
@@ -2481,12 +2801,12 @@ class Title {
         * Get a title object associated with the subject page of this
         * talk page
         *
-        * @return \type{Title} the object for the subject page
+        * @return Title the object for the subject page
         */
        public function getSubjectPage() {
                // Is this the same title?
                $subjectNS = MWNamespace::getSubject( $this->getNamespace() );
-               if( $this->getNamespace() == $subjectNS ) {
+               if ( $this->getNamespace() == $subjectNS ) {
                        return $this;
                }
                return Title::makeTitle( $subjectNS, $this->getDBkey() );
@@ -2499,8 +2819,10 @@ class Title {
         * WARNING: do not use this function on arbitrary user-supplied titles!
         * On heavily-used templates it will max out the memory.
         *
-        * @param array $options may be FOR UPDATE
-        * @return \type{\arrayof{Title}} the Title objects linking here
+        * @param $options Array: may be FOR UPDATE
+        * @param $table String: table name
+        * @param $prefix String: fields prefix
+        * @return Array of Title objects linking here
         */
        public function getLinksTo( $options = array(), $table = 'pagelinks', $prefix = 'pl' ) {
                $linkCache = LinkCache::singleton();
@@ -2511,25 +2833,27 @@ class Title {
                        $db = wfGetDB( DB_SLAVE );
                }
 
-               $res = $db->select( array( 'page', $table ),
-                       array( 'page_namespace', 'page_title', 'page_id', 'page_len', 'page_is_redirect' ),
+               $res = $db->select(
+                       array( 'page', $table ),
+                       array( 'page_namespace', 'page_title', 'page_id', 'page_len', 'page_is_redirect', 'page_latest' ),
                        array(
                                "{$prefix}_from=page_id",
                                "{$prefix}_namespace" => $this->getNamespace(),
                                "{$prefix}_title"     => $this->getDBkey() ),
                        __METHOD__,
-                       $options );
+                       $options
+               );
 
                $retVal = array();
                if ( $db->numRows( $res ) ) {
-                       foreach( $res as $row ) {
-                               if ( $titleObj = Title::makeTitle( $row->page_namespace, $row->page_title ) ) {
-                                       $linkCache->addGoodLinkObj( $row->page_id, $titleObj, $row->page_len, $row->page_is_redirect );
+                       foreach ( $res as $row ) {
+                               $titleObj = Title::makeTitle( $row->page_namespace, $row->page_title );
+                               if ( $titleObj ) {
+                                       $linkCache->addGoodLinkObj( $row->page_id, $titleObj, $row->page_len, $row->page_is_redirect, $row->page_latest );
                                        $retVal[] = $titleObj;
                                }
                        }
                }
-               $db->freeResult( $res );
                return $retVal;
        }
 
@@ -2540,8 +2864,8 @@ class Title {
         * WARNING: do not use this function on arbitrary user-supplied titles!
         * On heavily-used templates it will max out the memory.
         *
-        * @param array $options may be FOR UPDATE
-        * @return \type{\arrayof{Title}} the Title objects linking here
+        * @param $options Array: may be FOR UPDATE
+        * @return Array of Title the Title objects linking here
         */
        public function getTemplateLinksTo( $options = array() ) {
                return $this->getLinksTo( $options, 'templatelinks', 'tl' );
@@ -2551,7 +2875,7 @@ class Title {
         * Get an array of Title objects referring to non-existent articles linked from this page
         *
         * @todo check if needed (used only in SpecialBrokenRedirects.php, and should use redirect table in this case)
-        * @return \type{\arrayof{Title}} the Title objects
+        * @return Array of Title the Title objects
         */
        public function getBrokenLinksFrom() {
                if ( $this->getArticleId() == 0 ) {
@@ -2577,7 +2901,7 @@ class Title {
                );
 
                $retVal = array();
-               foreach( $res as $row ) {
+               foreach ( $res as $row ) {
                        $retVal[] = Title::makeTitle( $row->pl_namespace, $row->pl_title );
                }
                return $retVal;
@@ -2588,7 +2912,7 @@ class Title {
         * Get a list of URLs to purge from the Squid cache when this
         * page changes
         *
-        * @return \type{\arrayof{\string}} the URLs
+        * @return Array of String the URLs
         */
        public function getSquidURLs() {
                global $wgContLang;
@@ -2599,11 +2923,10 @@ class Title {
                );
 
                // purge variant urls as well
-               if($wgContLang->hasVariants()){
+               if ( $wgContLang->hasVariants() ) {
                        $variants = $wgContLang->getVariants();
-                       foreach($variants as $vCode){
-                               if($vCode==$wgContLang->getCode()) continue; // we don't want default variant
-                               $urls[] = $this->getInternalURL('',$vCode);
+                       foreach ( $variants as $vCode ) {
+                               $urls[] = $this->getInternalURL( '', $vCode );
                        }
                }
 
@@ -2624,7 +2947,9 @@ class Title {
 
        /**
         * Move this page without authentication
-        * @param &$nt \type{Title} the new page Title
+        *
+        * @param $nt Title the new page Title
+        * @return Mixed true on success, getUserPermissionsErrors()-like array on failure
         */
        public function moveNoAuth( &$nt ) {
                return $this->moveTo( $nt, false );
@@ -2633,84 +2958,88 @@ class Title {
        /**
         * Check whether a given move operation would be valid.
         * Returns true if ok, or a getUserPermissionsErrors()-like array otherwise
-        * @param &$nt \type{Title} the new title
-        * @param $auth \type{\bool} indicates whether $wgUser's permissions
+        *
+        * @param $nt Title the new title
+        * @param $auth Bool indicates whether $wgUser's permissions
         *  should be checked
-        * @param $reason \type{\string} is the log summary of the move, used for spam checking
-        * @return \type{\mixed} True on success, getUserPermissionsErrors()-like array on failure
+        * @param $reason String is the log summary of the move, used for spam checking
+        * @return Mixed True on success, getUserPermissionsErrors()-like array on failure
         */
        public function isValidMoveOperation( &$nt, $auth = true, $reason = '' ) {
                global $wgUser;
 
                $errors = array();
-               if( !$nt ) {
+               if ( !$nt ) {
                        // Normally we'd add this to $errors, but we'll get
                        // lots of syntax errors if $nt is not an object
-                       return array(array('badtitletext'));
+                       return array( array( 'badtitletext' ) );
                }
-               if( $this->equals( $nt ) ) {
-                       $errors[] = array('selfmove');
+               if ( $this->equals( $nt ) ) {
+                       $errors[] = array( 'selfmove' );
                }
-               if( !$this->isMovable() ) {
+               if ( !$this->isMovable() ) {
                        $errors[] = array( 'immobile-source-namespace', $this->getNsText() );
                }
                if ( $nt->getInterwiki() != '' ) {
                        $errors[] = array( 'immobile-target-namespace-iw' );
                }
                if ( !$nt->isMovable() ) {
-                       $errors[] = array('immobile-target-namespace', $nt->getNsText() );
+                       $errors[] = array( 'immobile-target-namespace', $nt->getNsText() );
                }
 
                $oldid = $this->getArticleID();
                $newid = $nt->getArticleID();
 
                if ( strlen( $nt->getDBkey() ) < 1 ) {
-                       $errors[] = array('articleexists');
+                       $errors[] = array( 'articleexists' );
                }
-               if ( ( '' == $this->getDBkey() ) ||
+               if ( ( $this->getDBkey() == '' ) ||
                         ( !$oldid ) ||
-                    ( '' == $nt->getDBkey() ) ) {
-                       $errors[] = array('badarticleerror');
+                    ( $nt->getDBkey() == '' ) ) {
+                       $errors[] = array( 'badarticleerror' );
                }
 
                // Image-specific checks
-               if( $this->getNamespace() == NS_FILE ) {
+               if ( $this->getNamespace() == NS_FILE ) {
+                       if ( $nt->getNamespace() != NS_FILE ) {
+                               $errors[] = array( 'imagenocrossnamespace' );
+                       }
                        $file = wfLocalFile( $this );
-                       if( $file->exists() ) {
-                               if( $nt->getNamespace() != NS_FILE ) {
-                                       $errors[] = array('imagenocrossnamespace');
+                       if ( $file->exists() ) {
+                               if ( $nt->getText() != wfStripIllegalFilenameChars( $nt->getText() ) ) {
+                                       $errors[] = array( 'imageinvalidfilename' );
                                }
-                               if( $nt->getText() != wfStripIllegalFilenameChars( $nt->getText() ) ) {
-                                       $errors[] = array('imageinvalidfilename');
+                               if ( !File::checkExtensionCompatibility( $file, $nt->getDBkey() ) ) {
+                                       $errors[] = array( 'imagetypemismatch' );
                                }
-                               if( !File::checkExtensionCompatibility( $file, $nt->getDBkey() ) ) {
-                                       $errors[] = array('imagetypemismatch');
-                               }
-                       } 
+                       }
                        $destfile = wfLocalFile( $nt );
-                       if( !$wgUser->isAllowed( 'reupload-shared' ) && !$destfile->exists() && wfFindFile( $nt ) ) {
+                       if ( !$wgUser->isAllowed( 'reupload-shared' ) && !$destfile->exists() && wfFindFile( $nt ) ) {
                                $errors[] = array( 'file-exists-sharedrepo' );
                        }
-                       
+               }
+
+               if ( $nt->getNamespace() == NS_FILE && $this->getNamespace() != NS_FILE ) {
+                       $errors[] = array( 'nonfile-cannot-move-to-file' );
                }
 
                if ( $auth ) {
                        $errors = wfMergeErrorArrays( $errors,
-                               $this->getUserPermissionsErrors('move', $wgUser),
-                               $this->getUserPermissionsErrors('edit', $wgUser),
-                               $nt->getUserPermissionsErrors('move-target', $wgUser),
-                               $nt->getUserPermissionsErrors('edit', $wgUser) );
+                               $this->getUserPermissionsErrors( 'move', $wgUser ),
+                               $this->getUserPermissionsErrors( 'edit', $wgUser ),
+                               $nt->getUserPermissionsErrors( 'move-target', $wgUser ),
+                               $nt->getUserPermissionsErrors( 'edit', $wgUser ) );
                }
 
                $match = EditPage::matchSummarySpamRegex( $reason );
-               if( $match !== false ) {
+               if ( $match !== false ) {
                        // This is kind of lame, won't display nice
-                       $errors[] = array('spamprotectiontext');
+                       $errors[] = array( 'spamprotectiontext' );
                }
 
                $err = null;
-               if( !wfRunHooks( 'AbortMove', array( $this, $nt, $wgUser, &$err, $reason ) ) ) {
-                       $errors[] = array('hookaborted', $err);
+               if ( !wfRunHooks( 'AbortMove', array( $this, $nt, $wgUser, &$err, $reason ) ) ) {
+                       $errors[] = array( 'hookaborted', $err );
                }
 
                # The move is allowed only if (1) the target doesn't exist, or
@@ -2718,44 +3047,48 @@ class Title {
                # (so we can undo bad moves right after they're done).
 
                if ( 0 != $newid ) { # Target exists; check for validity
-                       if ( ! $this->isValidMoveTarget( $nt ) ) {
-                               $errors[] = array('articleexists');
+                       if ( !$this->isValidMoveTarget( $nt ) ) {
+                               $errors[] = array( 'articleexists' );
                        }
                } else {
                        $tp = $nt->getTitleProtection();
                        $right = ( $tp['pt_create_perm'] == 'sysop' ) ? 'protect' : $tp['pt_create_perm'];
                        if ( $tp and !$wgUser->isAllowed( $right ) ) {
-                               $errors[] = array('cantmove-titleprotected');
+                               $errors[] = array( 'cantmove-titleprotected' );
                        }
                }
-               if(empty($errors))
+               if ( empty( $errors ) ) {
                        return true;
+               }
                return $errors;
        }
 
        /**
         * Move a title to a new location
-        * @param &$nt \type{Title} the new title
-        * @param $auth \type{\bool} indicates whether $wgUser's permissions
+        *
+        * @param $nt Title the new title
+        * @param $auth Bool indicates whether $wgUser's permissions
         *  should be checked
-        * @param $reason \type{\string} The reason for the move
-        * @param $createRedirect \type{\bool} Whether to create a redirect from the old title to the new title.
+        * @param $reason String The reason for the move
+        * @param $createRedirect Bool Whether to create a redirect from the old title to the new title.
         *  Ignored if the user doesn't have the suppressredirect right.
-        * @return \type{\mixed} true on success, getUserPermissionsErrors()-like array on failure
+        * @return Mixed true on success, getUserPermissionsErrors()-like array on failure
         */
        public function moveTo( &$nt, $auth = true, $reason = '', $createRedirect = true ) {
+               global $wgContLang;
+
                $err = $this->isValidMoveOperation( $nt, $auth, $reason );
-               if( is_array( $err ) ) {
+               if ( is_array( $err ) ) {
                        return $err;
                }
 
                // If it is a file, move it first. It is done before all other moving stuff is done because it's hard to revert
                $dbw = wfGetDB( DB_MASTER );
-               if( $this->getNamespace() == NS_FILE ) {
+               if ( $this->getNamespace() == NS_FILE ) {
                        $file = wfLocalFile( $this );
-                       if( $file->exists() ) {
+                       if ( $file->exists() ) {
                                $status = $file->move( $nt );
-                               if( !$status->isOk() ) {
+                               if ( !$status->isOk() ) {
                                        return $status->getErrorsArray();
                                }
                        }
@@ -2763,40 +3096,32 @@ class Title {
 
                $pageid = $this->getArticleID();
                $protected = $this->isProtected();
-               if( $nt->exists() ) {
-                       $err = $this->moveOverExistingRedirect( $nt, $reason, $createRedirect );
-                       $pageCountChange = ($createRedirect ? 0 : -1);
-               } else { # Target didn't exist, do normal move.
-                       $err = $this->moveToNewTitle( $nt, $reason, $createRedirect );
-                       $pageCountChange = ($createRedirect ? 1 : 0);
-               }
+               $pageCountChange = ( $createRedirect ? 1 : 0 ) - ( $nt->exists() ? 1 : 0 );
 
-               if( is_array( $err ) ) {
+               // Do the actual move
+               $err = $this->moveToInternal( $nt, $reason, $createRedirect );
+               if ( is_array( $err ) ) {
                        return $err;
                }
+
                $redirid = $this->getArticleID();
 
-               // Category memberships include a sort key which may be customized.
-               // If it's left as the default (the page title), we need to update
-               // the sort key to match the new title.
-               //
-               // Be careful to avoid resetting cl_timestamp, which may disturb
-               // time-based lists on some sites.
-               //
-               // Warning -- if the sort key is *explicitly* set to the old title,
-               // we can't actually distinguish it from a default here, and it'll
-               // be set to the new title even though it really shouldn't.
-               // It'll get corrected on the next edit, but resetting cl_timestamp.
+               // Refresh the sortkey for this row.  Be careful to avoid resetting
+               // cl_timestamp, which may disturb time-based lists on some sites.
+               $prefix = $dbw->selectField(
+                       'categorylinks',
+                       'cl_sortkey_prefix',
+                       array( 'cl_from' => $pageid ),
+                       __METHOD__
+               );
                $dbw->update( 'categorylinks',
                        array(
-                               'cl_sortkey' => $nt->getPrefixedText(),
+                               'cl_sortkey' => $wgContLang->convertToSortkey( $nt->getCategorySortkey( $prefix ) ),
                                'cl_timestamp=cl_timestamp' ),
-                       array(
-                               'cl_from' => $pageid,
-                               'cl_sortkey' => $this->getPrefixedText() ),
+                       array( 'cl_from' => $pageid ),
                        __METHOD__ );
 
-               if( $protected ) {
+               if ( $protected ) {
                        # Protect the redirect title as the title used to be...
                        $dbw->insertSelect( 'page_restrictions', 'page_restrictions',
                                array(
@@ -2814,8 +3139,10 @@ class Title {
                        # Update the protection log
                        $log = new LogPage( 'protect' );
                        $comment = wfMsgForContent( 'prot_1movedto2', $this->getPrefixedText(), $nt->getPrefixedText() );
-                       if( $reason ) $comment .= wfMsgForContent( 'colon-separator' ) . $reason;
-                       $log->addEntry( 'move_prot', $nt, $comment, array($this->getPrefixedText()) ); // FIXME: $params?
+                       if ( $reason ) {
+                               $comment .= wfMsgForContent( 'colon-separator' ) . $reason;
+                       }
+                       $log->addEntry( 'move_prot', $nt, $comment, array( $this->getPrefixedText() ) ); // FIXME: $params?
                }
 
                # Update watchlists
@@ -2824,7 +3151,7 @@ class Title {
                $oldtitle = $this->getDBkey();
                $newtitle = $nt->getDBkey();
 
-               if( $oldnamespace != $newnamespace || $oldtitle != $newtitle ) {
+               if ( $oldnamespace != $newnamespace || $oldtitle != $newtitle ) {
                        WatchedItem::duplicateEntries( $this, $nt );
                }
 
@@ -2835,27 +3162,27 @@ class Title {
                $u->doUpdate();
 
                # Update site_stats
-               if( $this->isContentPage() && !$nt->isContentPage() ) {
+               if ( $this->isContentPage() && !$nt->isContentPage() ) {
                        # No longer a content page
                        # Not viewed, edited, removing
                        $u = new SiteStatsUpdate( 0, 1, -1, $pageCountChange );
-               } elseif( !$this->isContentPage() && $nt->isContentPage() ) {
+               } elseif ( !$this->isContentPage() && $nt->isContentPage() ) {
                        # Now a content page
                        # Not viewed, edited, adding
-                       $u = new SiteStatsUpdate( 0, 1, +1, $pageCountChange );
-               } elseif( $pageCountChange ) {
+                       $u = new SiteStatsUpdate( 0, 1, + 1, $pageCountChange );
+               } elseif ( $pageCountChange ) {
                        # Redirect added
                        $u = new SiteStatsUpdate( 0, 0, 0, 1 );
                } else {
                        # Nothing special
                        $u = false;
                }
-               if( $u )
+               if ( $u ) {
                        $u->doUpdate();
+               }
                # Update message cache for interface messages
-               if( $nt->getNamespace() == NS_MEDIAWIKI ) {
-                       global $wgMessageCache;
-
+               global $wgMessageCache;
+               if ( $this->getNamespace() == NS_MEDIAWIKI ) {
                        # @bug 17860: old article can be deleted, if this the case,
                        # delete it from message cache
                        if ( $this->getArticleID() === 0 ) {
@@ -2864,7 +3191,8 @@ class Title {
                                $oldarticle = new Article( $this );
                                $wgMessageCache->replace( $this->getDBkey(), $oldarticle->getContent() );
                        }
-
+               }
+               if ( $nt->getNamespace() == NS_MEDIAWIKI ) {
                        $newarticle = new Article( $nt );
                        $wgMessageCache->replace( $nt->getDBkey(), $newarticle->getContent() );
                }
@@ -2875,70 +3203,82 @@ class Title {
        }
 
        /**
-        * Move page to a title which is at present a redirect to the
-        * source page
+        * Move page to a title which is either a redirect to the
+        * source page or nonexistent
         *
-        * @param &$nt \type{Title} the page to move to, which should currently
-        *  be a redirect
-        * @param $reason \type{\string} The reason for the move
-        * @param $createRedirect \type{\bool} Whether to leave a redirect at the old title.
-        *  Ignored if the user doesn't have the suppressredirect right
+        * @param $nt Title the page to move to, which should
+        * be a redirect or nonexistent
+        * @param $reason String The reason for the move
+        * @param $createRedirect Bool Whether to leave a
+        * redirect at the old title.  Ignored if the user doesn't
+        * have the suppressredirect right
         */
-       private function moveOverExistingRedirect( &$nt, $reason = '', $createRedirect = true ) {
-               global $wgUseSquid, $wgUser;
+       private function moveToInternal( &$nt, $reason = '', $createRedirect = true ) {
+               global $wgUser, $wgContLang;
+
+               $moveOverRedirect = $nt->exists();
 
-               $comment = wfMsgForContent( '1movedto2_redir', $this->getPrefixedText(), $nt->getPrefixedText() );
+               $commentMsg = ( $moveOverRedirect ? '1movedto2_redir' : '1movedto2' );
+               $comment = wfMsgForContent( $commentMsg, $this->getPrefixedText(), $nt->getPrefixedText() );
 
                if ( $reason ) {
                        $comment .= wfMsgForContent( 'colon-separator' ) . $reason;
                }
+               # Truncate for whole multibyte characters. +5 bytes for ellipsis
+               $comment = $wgContLang->truncate( $comment, 250 );
 
-               $now = wfTimestampNow();
-               $newid = $nt->getArticleID();
                $oldid = $this->getArticleID();
                $latest = $this->getLatestRevID();
 
                $dbw = wfGetDB( DB_MASTER );
 
-               $rcts = $dbw->timestamp( $nt->getEarliestRevTime() );
-               $newns = $nt->getNamespace();
-               $newdbk = $nt->getDBkey();
-
-               # Delete the old redirect. We don't save it to history since
-               # by definition if we've got here it's rather uninteresting.
-               # We have to remove it so that the next step doesn't trigger
-               # a conflict on the unique namespace+title index...
-               $dbw->delete( 'page', array( 'page_id' => $newid ), __METHOD__ );
-               if ( !$dbw->cascadingDeletes() ) {
-                       $dbw->delete( 'revision', array( 'rev_page' => $newid ), __METHOD__ );
-                       global $wgUseTrackbacks;
-                       if ($wgUseTrackbacks)
-                               $dbw->delete( 'trackbacks', array( 'tb_page' => $newid ), __METHOD__ );
-                       $dbw->delete( 'pagelinks', array( 'pl_from' => $newid ), __METHOD__ );
-                       $dbw->delete( 'imagelinks', array( 'il_from' => $newid ), __METHOD__ );
-                       $dbw->delete( 'categorylinks', array( 'cl_from' => $newid ), __METHOD__ );
-                       $dbw->delete( 'templatelinks', array( 'tl_from' => $newid ), __METHOD__ );
-                       $dbw->delete( 'externallinks', array( 'el_from' => $newid ), __METHOD__ );
-                       $dbw->delete( 'langlinks', array( 'll_from' => $newid ), __METHOD__ );
-                       $dbw->delete( 'redirect', array( 'rd_from' => $newid ), __METHOD__ );
-               }
-               // If the redirect was recently created, it may have an entry in recentchanges still
-               $dbw->delete( 'recentchanges',
-                       array( 'rc_timestamp' => $rcts, 'rc_namespace' => $newns, 'rc_title' => $newdbk, 'rc_new' => 1 ),
-                       __METHOD__
-               );
+               if ( $moveOverRedirect ) {
+                       $rcts = $dbw->timestamp( $nt->getEarliestRevTime() );
+
+                       $newid = $nt->getArticleID();
+                       $newns = $nt->getNamespace();
+                       $newdbk = $nt->getDBkey();
+
+                       # Delete the old redirect. We don't save it to history since
+                       # by definition if we've got here it's rather uninteresting.
+                       # We have to remove it so that the next step doesn't trigger
+                       # a conflict on the unique namespace+title index...
+                       $dbw->delete( 'page', array( 'page_id' => $newid ), __METHOD__ );
+                       if ( !$dbw->cascadingDeletes() ) {
+                               $dbw->delete( 'revision', array( 'rev_page' => $newid ), __METHOD__ );
+                               global $wgUseTrackbacks;
+                               if ( $wgUseTrackbacks ) {
+                                       $dbw->delete( 'trackbacks', array( 'tb_page' => $newid ), __METHOD__ );
+                               }
+                               $dbw->delete( 'pagelinks', array( 'pl_from' => $newid ), __METHOD__ );
+                               $dbw->delete( 'imagelinks', array( 'il_from' => $newid ), __METHOD__ );
+                               $dbw->delete( 'categorylinks', array( 'cl_from' => $newid ), __METHOD__ );
+                               $dbw->delete( 'templatelinks', array( 'tl_from' => $newid ), __METHOD__ );
+                               $dbw->delete( 'externallinks', array( 'el_from' => $newid ), __METHOD__ );
+                               $dbw->delete( 'langlinks', array( 'll_from' => $newid ), __METHOD__ );
+                               $dbw->delete( 'redirect', array( 'rd_from' => $newid ), __METHOD__ );
+                       }
+                       // If the target page was recently created, it may have an entry in recentchanges still
+                       $dbw->delete( 'recentchanges',
+                               array( 'rc_timestamp' => $rcts, 'rc_namespace' => $newns, 'rc_title' => $newdbk, 'rc_new' => 1 ),
+                               __METHOD__
+                       );
+               }
 
                # Save a null revision in the page's history notifying of the move
                $nullRevision = Revision::newNullRevision( $dbw, $oldid, $comment, true );
+               if ( !is_object( $nullRevision ) ) {
+                       throw new MWException( 'No valid null revision produced in ' . __METHOD__ );
+               }
                $nullRevId = $nullRevision->insertOn( $dbw );
 
                $article = new Article( $this );
-               wfRunHooks( 'NewRevisionFromEditComplete', array($article, $nullRevision, $latest, $wgUser) );
+               wfRunHooks( 'NewRevisionFromEditComplete', array( $article, $nullRevision, $latest, $wgUser ) );
 
                # Change the name of the target page:
                $dbw->update( 'page',
                        /* SET */ array(
-                               'page_touched'   => $dbw->timestamp($now),
+                               'page_touched'   => $dbw->timestamp(),
                                'page_namespace' => $nt->getNamespace(),
                                'page_title'     => $nt->getDBkey(),
                                'page_latest'    => $nullRevId,
@@ -2949,7 +3289,7 @@ class Title {
                $nt->resetArticleID( $oldid );
 
                # Recreate the redirect, this time in the other direction.
-               if( $createRedirect || !$wgUser->isAllowed('suppressredirect') ) {
+               if ( $createRedirect || !$wgUser->isAllowed( 'suppressredirect' ) ) {
                        $mwRedir = MagicWord::get( 'redirect' );
                        $redirectText = $mwRedir->getSynonym( 0 ) . ' [[' . $nt->getPrefixedText() . "]]\n";
                        $redirectArticle = new Article( $this );
@@ -2961,7 +3301,7 @@ class Title {
                        $redirectRevision->insertOn( $dbw );
                        $redirectArticle->updateRevisionOn( $dbw, $redirectRevision, 0 );
 
-                       wfRunHooks( 'NewRevisionFromEditComplete', array($redirectArticle, $redirectRevision, false, $wgUser) );
+                       wfRunHooks( 'NewRevisionFromEditComplete', array( $redirectArticle, $redirectRevision, false, $wgUser ) );
 
                        # Now, we record the link from the redirect to the new title.
                        # It should have no other outgoing links...
@@ -2980,107 +3320,23 @@ class Title {
 
                # Log the move
                $log = new LogPage( 'move' );
-               $log->addEntry( 'move_redir', $this, $reason, array( 1 => $nt->getPrefixedText(), 2 => $redirectSuppressed ) );
-
-               # Purge squid
-               if ( $wgUseSquid ) {
-                       $urls = array_merge( $nt->getSquidURLs(), $this->getSquidURLs() );
-                       $u = new SquidUpdate( $urls );
-                       $u->doUpdate();
-               }
-
-       }
-
-       /**
-        * Move page to non-existing title.
-        * @param &$nt \type{Title} the new Title
-        * @param $reason \type{\string} The reason for the move
-        * @param $createRedirect \type{\bool} Whether to create a redirect from the old title to the new title
-        *  Ignored if the user doesn't have the suppressredirect right
-        */
-       private function moveToNewTitle( &$nt, $reason = '', $createRedirect = true ) {
-               global $wgUseSquid, $wgUser;
-
-               $comment = wfMsgForContent( '1movedto2', $this->getPrefixedText(), $nt->getPrefixedText() );
-               if ( $reason ) {
-                       $comment .= wfMsgExt( 'colon-separator',
-                               array( 'escapenoentities', 'content' ) );
-                       $comment .= $reason;
-               }
+               $logType = ( $moveOverRedirect ? 'move_redir' : 'move' );
+               $log->addEntry( $logType, $this, $reason, array( 1 => $nt->getPrefixedText(), 2 => $redirectSuppressed ) );
 
-               $newid = $nt->getArticleID();
-               $oldid = $this->getArticleID();
-               $latest = $this->getLatestRevId();
-
-               $dbw = wfGetDB( DB_MASTER );
-               $now = $dbw->timestamp();
-
-               # Save a null revision in the page's history notifying of the move
-               $nullRevision = Revision::newNullRevision( $dbw, $oldid, $comment, true );
-               if ( !is_object( $nullRevision ) ) {
-                       throw new MWException( 'No valid null revision produced in ' . __METHOD__ );
-               }
-               $nullRevId = $nullRevision->insertOn( $dbw );
-
-               $article = new Article( $this );
-               wfRunHooks( 'NewRevisionFromEditComplete', array($article, $nullRevision, $latest, $wgUser) );
-
-               # Rename page entry
-               $dbw->update( 'page',
-                       /* SET */ array(
-                               'page_touched'   => $now,
-                               'page_namespace' => $nt->getNamespace(),
-                               'page_title'     => $nt->getDBkey(),
-                               'page_latest'    => $nullRevId,
-                       ),
-                       /* WHERE */ array( 'page_id' => $oldid ),
-                       __METHOD__
-               );
-               $nt->resetArticleID( $oldid );
-
-               if( $createRedirect || !$wgUser->isAllowed('suppressredirect') ) {
-                       # Insert redirect
-                       $mwRedir = MagicWord::get( 'redirect' );
-                       $redirectText = $mwRedir->getSynonym( 0 ) . ' [[' . $nt->getPrefixedText() . "]]\n";
-                       $redirectArticle = new Article( $this );
-                       $newid = $redirectArticle->insertOn( $dbw );
-                       $redirectRevision = new Revision( array(
-                               'page'    => $newid,
-                               'comment' => $comment,
-                               'text'    => $redirectText ) );
-                       $redirectRevision->insertOn( $dbw );
-                       $redirectArticle->updateRevisionOn( $dbw, $redirectRevision, 0 );
-
-                       wfRunHooks( 'NewRevisionFromEditComplete', array($redirectArticle, $redirectRevision, false, $wgUser) );
-
-                       # Record the just-created redirect's linking to the page
-                       $dbw->insert( 'pagelinks',
-                               array(
-                                       'pl_from'      => $newid,
-                                       'pl_namespace' => $nt->getNamespace(),
-                                       'pl_title'     => $nt->getDBkey() ),
-                               __METHOD__ );
-                       $redirectSuppressed = false;
+               # Purge caches for old and new titles
+               if ( $moveOverRedirect ) {
+                       # A simple purge is enough when moving over a redirect
+                       $nt->purgeSquid();
                } else {
-                       $this->resetArticleID( 0 );
-                       $redirectSuppressed = true;
+                       # Purge caches as per article creation, including any pages that link to this title
+                       Article::onArticleCreate( $nt );
                }
-
-               # Log the move
-               $log = new LogPage( 'move' );
-               $log->addEntry( 'move', $this, $reason, array( 1 => $nt->getPrefixedText(), 2 => $redirectSuppressed ) );
-
-               # Purge caches as per article creation
-               Article::onArticleCreate( $nt );
-
-               # Purge old title from squid
-               # The new title, and links to the new title, are purged in Article::onArticleCreate()
                $this->purgeSquid();
-
        }
 
        /**
         * Move this page's subpages to be subpages of $nt
+        *
         * @param $nt Title Move target
         * @param $auth bool Whether $wgUser's permissions should be checked
         * @param $reason string The reason for the move
@@ -3092,22 +3348,25 @@ class Title {
        public function moveSubpages( $nt, $auth = true, $reason = '', $createRedirect = true ) {
                global $wgMaximumMovedPages;
                // Check permissions
-               if( !$this->userCan( 'move-subpages' ) )
+               if ( !$this->userCan( 'move-subpages' ) ) {
                        return array( 'cant-move-subpages' );
+               }
                // Do the source and target namespaces support subpages?
-               if( !MWNamespace::hasSubpages( $this->getNamespace() ) )
+               if ( !MWNamespace::hasSubpages( $this->getNamespace() ) ) {
                        return array( 'namespace-nosubpages',
                                MWNamespace::getCanonicalName( $this->getNamespace() ) );
-               if( !MWNamespace::hasSubpages( $nt->getNamespace() ) )
+               }
+               if ( !MWNamespace::hasSubpages( $nt->getNamespace() ) ) {
                        return array( 'namespace-nosubpages',
                                MWNamespace::getCanonicalName( $nt->getNamespace() ) );
+               }
 
-               $subpages = $this->getSubpages($wgMaximumMovedPages + 1);
+               $subpages = $this->getSubpages( $wgMaximumMovedPages + 1 );
                $retval = array();
                $count = 0;
-               foreach( $subpages as $oldSubpage ) {
+               foreach ( $subpages as $oldSubpage ) {
                        $count++;
-                       if( $count > $wgMaximumMovedPages ) {
+                       if ( $count > $wgMaximumMovedPages ) {
                                $retval[$oldSubpage->getPrefixedTitle()] =
                                                array( 'movepage-max-pages',
                                                        $wgMaximumMovedPages );
@@ -3117,16 +3376,18 @@ class Title {
                        // We don't know whether this function was called before
                        // or after moving the root page, so check both
                        // $this and $nt
-                       if( $oldSubpage->getArticleId() == $this->getArticleId() ||
+                       if ( $oldSubpage->getArticleId() == $this->getArticleId() ||
                                        $oldSubpage->getArticleID() == $nt->getArticleId() )
+                       {
                                // When moving a page to a subpage of itself,
                                // don't move it twice
                                continue;
+                       }
                        $newPageName = preg_replace(
-                                       '#^'.preg_quote( $this->getDBkey(), '#' ).'#',
+                                       '#^' . preg_quote( $this->getDBkey(), '#' ) . '#',
                                        StringUtils::escapeRegexReplacement( $nt->getDBkey() ), # bug 21234
                                        $oldSubpage->getDBkey() );
-                       if( $oldSubpage->isTalkPage() ) {
+                       if ( $oldSubpage->isTalkPage() ) {
                                $newNs = $nt->getTalkPage()->getNamespace();
                        } else {
                                $newNs = $nt->getSubjectPage()->getNamespace();
@@ -3136,7 +3397,7 @@ class Title {
                        $newSubpage = Title::makeTitleSafe( $newNs, $newPageName );
 
                        $success = $oldSubpage->moveTo( $newSubpage, $auth, $reason, $createRedirect );
-                       if( $success === true ) {
+                       if ( $success === true ) {
                                $retval[$oldSubpage->getPrefixedText()] = $newSubpage->getPrefixedText();
                        } else {
                                $retval[$oldSubpage->getPrefixedText()] = $success;
@@ -3149,7 +3410,7 @@ class Title {
         * Checks if this page is just a one-rev redirect.
         * Adds lock, so don't use just for light purposes.
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool
         */
        public function isSingleRevRedirect() {
                $dbw = wfGetDB( DB_MASTER );
@@ -3161,14 +3422,14 @@ class Title {
                        array( 'FOR UPDATE' )
                );
                # Cache some fields we may want
-               $this->mArticleID = $row ? intval($row->page_id) : 0;
+               $this->mArticleID = $row ? intval( $row->page_id ) : 0;
                $this->mRedirect = $row ? (bool)$row->page_is_redirect : false;
-               $this->mLatestID = $row ? intval($row->page_latest) : false;
-               if( !$this->mRedirect ) {
+               $this->mLatestID = $row ? intval( $row->page_latest ) : false;
+               if ( !$this->mRedirect ) {
                        return false;
                }
                # Does the article have a history?
-               $row = $dbw->selectField( array( 'page', 'revision'),
+               $row = $dbw->selectField( array( 'page', 'revision' ),
                        'rev_id',
                        array( 'page_namespace' => $this->getNamespace(),
                                'page_title' => $this->getDBkey(),
@@ -3179,28 +3440,27 @@ class Title {
                        array( 'FOR UPDATE' )
                );
                # Return true if there was no history
-               return ($row === false);
+               return ( $row === false );
        }
 
        /**
         * Checks if $this can be moved to a given Title
         * - Selects for update, so don't call it unless you mean business
         *
-        * @param &$nt \type{Title} the new title to check
-        * @return \type{\bool} TRUE or FALSE
+        * @param $nt Title the new title to check
+        * @return Bool TRUE or FALSE
         */
        public function isValidMoveTarget( $nt ) {
-               $dbw = wfGetDB( DB_MASTER );
-               # Is it an existsing file?
-               if( $nt->getNamespace() == NS_FILE ) {
+               # Is it an existing file?
+               if ( $nt->getNamespace() == NS_FILE ) {
                        $file = wfLocalFile( $nt );
-                       if( $file->exists() ) {
+                       if ( $file->exists() ) {
                                wfDebug( __METHOD__ . ": file exists\n" );
                                return false;
                        }
                }
                # Is it a redirect with no history?
-               if( !$nt->isSingleRevRedirect() ) {
+               if ( !$nt->isSingleRevRedirect() ) {
                        wfDebug( __METHOD__ . ": not a one-rev redirect\n" );
                        return false;
                }
@@ -3212,7 +3472,7 @@ class Title {
                $m = array();
                if ( preg_match( "/\\[\\[\\s*([^\\]\\|]*)]]/", $text, $m ) ) {
                        $redirTitle = Title::newFromText( $m[1] );
-                       if( !is_object( $redirTitle ) ||
+                       if ( !is_object( $redirTitle ) ||
                                ( $redirTitle->getPrefixedDBkey() != $this->getPrefixedDBkey() &&
                                $redirTitle->getPrefixedDBkey() != $nt->getPrefixedDBkey() ) ) {
                                wfDebug( __METHOD__ . ": redirect points to other page\n" );
@@ -3229,7 +3489,7 @@ class Title {
        /**
         * Can this title be added to a user's watchlist?
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool TRUE or FALSE
         */
        public function isWatchable() {
                return !$this->isExternal() && MWNamespace::isWatchable( $this->getNamespace() );
@@ -3239,7 +3499,7 @@ class Title {
         * Get categories to which this Title belongs and return an array of
         * categories' names.
         *
-        * @return \type{\array} array an array of parents in the form:
+        * @return Array of parents in the form:
         *      $parent => $currentarticle
         */
        public function getParentCategories() {
@@ -3251,48 +3511,47 @@ class Title {
 
                # NEW SQL
                $sql = "SELECT * FROM $categorylinks"
-                    ." WHERE cl_from='$titlekey'"
-                        ." AND cl_from <> '0'"
-                        ." ORDER BY cl_sortkey";
+                    . " WHERE cl_from='$titlekey'"
+                        . " AND cl_from <> '0'"
+                        . " ORDER BY cl_sortkey";
 
                $res = $dbr->query( $sql );
+               $data = array();
 
-               if( $dbr->numRows( $res ) > 0 ) {
-                       foreach( $res as $row )
-                               //$data[] = Title::newFromText($wgContLang->getNSText ( NS_CATEGORY ).':'.$row->cl_to);
-                               $data[$wgContLang->getNSText( NS_CATEGORY ).':'.$row->cl_to] = $this->getFullText();
-                       $dbr->freeResult( $res );
-               } else {
-                       $data = array();
+               if ( $dbr->numRows( $res ) > 0 ) {
+                       foreach ( $res as $row ) {
+                               // $data[] = Title::newFromText($wgContLang->getNSText ( NS_CATEGORY ).':'.$row->cl_to);
+                               $data[$wgContLang->getNSText( NS_CATEGORY ) . ':' . $row->cl_to] = $this->getFullText();
+                       }
                }
                return $data;
        }
 
        /**
         * Get a tree of parent categories
-        * @param $children \type{\array} an array with the children in the keys, to check for circular refs
-        * @return \type{\array} Tree of parent categories
+        *
+        * @param $children Array with the children in the keys, to check for circular refs
+        * @return Array Tree of parent categories
         */
        public function getParentCategoryTree( $children = array() ) {
                $stack = array();
                $parents = $this->getParentCategories();
 
-               if( $parents ) {
-                       foreach( $parents as $parent => $current ) {
+               if ( $parents ) {
+                       foreach ( $parents as $parent => $current ) {
                                if ( array_key_exists( $parent, $children ) ) {
                                        # Circular reference
                                        $stack[$parent] = array();
                                } else {
-                                       $nt = Title::newFromText($parent);
+                                       $nt = Title::newFromText( $parent );
                                        if ( $nt ) {
-                                               $stack[$parent] = $nt->getParentCategoryTree( $children + array($parent => 1) );
+                                               $stack[$parent] = $nt->getParentCategoryTree( $children + array( $parent => 1 ) );
                                        }
                                }
                        }
-                       return $stack;
-               } else {
-                       return array();
                }
+
+               return $stack;
        }
 
 
@@ -3300,10 +3559,10 @@ class Title {
         * Get an associative array for selecting this title from
         * the "page" table
         *
-        * @return \type{\array} Selection array
+        * @return Array Selection array
         */
        public function pageCond() {
-               if( $this->mArticleID > 0 ) {
+               if ( $this->mArticleID > 0 ) {
                        // PK avoids secondary lookups in InnoDB, shouldn't hurt other DBs
                        return array( 'page_id' => $this->mArticleID );
                } else {
@@ -3314,15 +3573,15 @@ class Title {
        /**
         * Get the revision ID of the previous revision
         *
-        * @param $revId \type{\int} Revision ID. Get the revision that was before this one.
-        * @param $flags \type{\int} GAID_FOR_UPDATE
+        * @param $revId Int Revision ID. Get the revision that was before this one.
+        * @param $flags Int Title::GAID_FOR_UPDATE
         * @return \twotypes{\int,\bool} Old revision ID, or FALSE if none exists
         */
-       public function getPreviousRevisionID( $revId, $flags=0 ) {
-               $db = ($flags & GAID_FOR_UPDATE) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
+       public function getPreviousRevisionID( $revId, $flags = 0 ) {
+               $db = ( $flags & self::GAID_FOR_UPDATE ) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
                return $db->selectField( 'revision', 'rev_id',
                        array(
-                               'rev_page' => $this->getArticleId($flags),
+                               'rev_page' => $this->getArticleId( $flags ),
                                'rev_id < ' . intval( $revId )
                        ),
                        __METHOD__,
@@ -3333,15 +3592,15 @@ class Title {
        /**
         * Get the revision ID of the next revision
         *
-        * @param $revId \type{\int} Revision ID. Get the revision that was after this one.
-        * @param $flags \type{\int} GAID_FOR_UPDATE
+        * @param $revId Int Revision ID. Get the revision that was after this one.
+        * @param $flags Int Title::GAID_FOR_UPDATE
         * @return \twotypes{\int,\bool} Next revision ID, or FALSE if none exists
         */
-       public function getNextRevisionID( $revId, $flags=0 ) {
-               $db = ($flags & GAID_FOR_UPDATE) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
+       public function getNextRevisionID( $revId, $flags = 0 ) {
+               $db = ( $flags & self::GAID_FOR_UPDATE ) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
                return $db->selectField( 'revision', 'rev_id',
                        array(
-                               'rev_page' => $this->getArticleId($flags),
+                               'rev_page' => $this->getArticleId( $flags ),
                                'rev_id > ' . intval( $revId )
                        ),
                        __METHOD__,
@@ -3352,19 +3611,21 @@ class Title {
        /**
         * Get the first revision of the page
         *
-        * @param $flags \type{\int} GAID_FOR_UPDATE
+        * @param $flags Int Title::GAID_FOR_UPDATE
         * @return Revision (or NULL if page doesn't exist)
         */
-       public function getFirstRevision( $flags=0 ) {
-               $db = ($flags & GAID_FOR_UPDATE) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
-               $pageId = $this->getArticleId($flags);
-               if( !$pageId ) return null;
+       public function getFirstRevision( $flags = 0 ) {
+               $db = ( $flags & self::GAID_FOR_UPDATE ) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
+               $pageId = $this->getArticleId( $flags );
+               if ( !$pageId ) {
+                       return null;
+               }
                $row = $db->selectRow( 'revision', '*',
                        array( 'rev_page' => $pageId ),
                        __METHOD__,
                        array( 'ORDER BY' => 'rev_timestamp ASC', 'LIMIT' => 1 )
                );
-               if( !$row ) {
+               if ( !$row ) {
                        return null;
                } else {
                        return new Revision( $row );
@@ -3384,11 +3645,11 @@ class Title {
        /**
         * Get the oldest revision timestamp of this page
         *
-        * @return string, MW timestamp
+        * @return String: MW timestamp
         */
        public function getEarliestRevTime() {
                $dbr = wfGetDB( DB_SLAVE );
-               if( $this->exists() ) {
+               if ( $this->exists() ) {
                        $min = $dbr->selectField( 'revision',
                                'MIN(rev_timestamp)',
                                array( 'rev_page' => $this->getArticleId() ),
@@ -3402,25 +3663,48 @@ class Title {
         * Get the number of revisions between the given revision IDs.
         * Used for diffs and other things that really need it.
         *
-        * @param $old \type{\int} Revision ID.
-        * @param $new \type{\int} Revision ID.
-        * @return \type{\int} Number of revisions between these IDs.
+        * @param $old Int Revision ID.
+        * @param $new Int Revision ID.
+        * @return Int Number of revisions between these IDs.
         */
        public function countRevisionsBetween( $old, $new ) {
                $dbr = wfGetDB( DB_SLAVE );
-               return (int)$dbr->selectField( 'revision', 'count(*)',
-                       'rev_page = ' . intval( $this->getArticleId() ) .
-                       ' AND rev_id > ' . intval( $old ) .
-                       ' AND rev_id < ' . intval( $new ),
-                       __METHOD__
+               return (int)$dbr->selectField( 'revision', 'count(*)', array(
+                               'rev_page' => intval( $this->getArticleId() ),
+                               'rev_id > ' . intval( $old ),
+                               'rev_id < ' . intval( $new )
+                       ), __METHOD__
+               );
+       }
+
+       /**
+        * Get the number of authors between the given revision IDs.
+        * Used for diffs and other things that really need it.
+        *
+        * @param $fromRevId Int Revision ID (first before range)
+        * @param $toRevId Int Revision ID (first after range)
+        * @param $limit Int Maximum number of authors
+        * @param $flags Int Title::GAID_FOR_UPDATE
+        * @return Int
+        */
+       public function countAuthorsBetween( $fromRevId, $toRevId, $limit, $flags = 0 ) {
+               $db = ( $flags & self::GAID_FOR_UPDATE ) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
+               $res = $db->select( 'revision', 'DISTINCT rev_user_text',
+                       array(
+                               'rev_page' => $this->getArticleID(),
+                               'rev_id > ' . (int)$fromRevId,
+                               'rev_id < ' . (int)$toRevId
+                       ), __METHOD__,
+                       array( 'LIMIT' => $limit )
                );
+               return (int)$db->numRows( $res );
        }
 
        /**
         * Compare with another title.
         *
-        * @param \type{Title} $title
-        * @return \type{\bool} TRUE or FALSE
+        * @param $title Title
+        * @return Bool TRUE or FALSE
         */
        public function equals( Title $title ) {
                // Note: === is necessary for proper matching of number-like titles.
@@ -3431,9 +3715,11 @@ class Title {
 
        /**
         * Callback for usort() to do title sorts by (namespace, title)
+        * 
+        * @return Integer: result of string comparison, or namespace comparison
         */
        public static function compare( $a, $b ) {
-               if( $a->getNamespace() == $b->getNamespace() ) {
+               if ( $a->getNamespace() == $b->getNamespace() ) {
                        return strcmp( $a->getText(), $b->getText() );
                } else {
                        return $a->getNamespace() - $b->getNamespace();
@@ -3443,7 +3729,7 @@ class Title {
        /**
         * Return a string representation of this title
         *
-        * @return \type{\string} String representation of this title
+        * @return String String representation of this title
         */
        public function __toString() {
                return $this->getPrefixedText();
@@ -3456,7 +3742,7 @@ class Title {
         * If you want to know if a title can be meaningfully viewed, you should
         * probably call the isKnown() method instead.
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool
         */
        public function exists() {
                return $this->getArticleId() != 0;
@@ -3476,30 +3762,28 @@ class Title {
         * existing code, but we might want to add an optional parameter to skip
         * it and any other expensive checks.)
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool
         */
        public function isAlwaysKnown() {
-               if( $this->mInterwiki != '' ) {
+               if ( $this->mInterwiki != '' ) {
                        return true;  // any interwiki link might be viewable, for all we know
                }
                switch( $this->mNamespace ) {
-               case NS_MEDIA:
-               case NS_FILE:
-                       return wfFindFile( $this );  // file exists, possibly in a foreign repo
-               case NS_SPECIAL:
-                       return SpecialPage::exists( $this->getDBkey() );  // valid special page
-               case NS_MAIN:
-                       return $this->mDbkeyform == '';  // selflink, possibly with fragment
-               case NS_MEDIAWIKI:
-                       // If the page is form Mediawiki:message/lang, calling wfMsgWeirdKey causes
-                       // the full l10n of that language to be loaded. That takes much memory and
-                       // isn't needed. So we strip the language part away.
-                       // Also, extension messages which are not loaded, are shown as red, because
-                       // we don't call MessageCache::loadAllMessages.
-                       list( $basename, /* rest */ ) = explode( '/', $this->mDbkeyform, 2 );
-                       return wfMsgWeirdKey( $basename );  // known system message
-               default:
-                       return false;
+                       case NS_MEDIA:
+                       case NS_FILE:
+                               return (bool)wfFindFile( $this );  // file exists, possibly in a foreign repo
+                       case NS_SPECIAL:
+                               return SpecialPage::exists( $this->getDBkey() );  // valid special page
+                       case NS_MAIN:
+                               return $this->mDbkeyform == '';  // selflink, possibly with fragment
+                       case NS_MEDIAWIKI:
+                               // If the page is form Mediawiki:message/lang, calling wfMsgWeirdKey causes
+                               // the full l10n of that language to be loaded. That takes much memory and
+                               // isn't needed. So we strip the language part away.
+                               list( $basename, /* rest */ ) = explode( '/', $this->mDbkeyform, 2 );
+                               return (bool)wfMsgWeirdKey( $basename );  // known system message
+                       default:
+                               return false;
                }
        }
 
@@ -3509,17 +3793,41 @@ class Title {
         * links to the title should be rendered as "bluelinks" (as opposed to
         * "redlinks" to non-existent pages).
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool
         */
        public function isKnown() {
-               return $this->exists() || $this->isAlwaysKnown();
+               return $this->isAlwaysKnown() || $this->exists();
+       }
+
+       /**
+        * Does this page have source text?
+        *
+        * @return Boolean
+        */
+       public function hasSourceText() {
+               if ( $this->exists() ) {
+                       return true;
+               }
+
+               if ( $this->mNamespace == NS_MEDIAWIKI ) {
+                       // If the page doesn't exist but is a known system message, default
+                       // message content will be displayed, same for language subpages
+                       // Also, if the page is form Mediawiki:message/lang, calling wfMsgWeirdKey
+                       // causes the full l10n of that language to be loaded. That takes much
+                       // memory and isn't needed. So we strip the language part away.
+                       list( $basename, /* rest */ ) = explode( '/', $this->mDbkeyform, 2 );
+                       return (bool)wfMsgWeirdKey( $basename );
+               }
+
+               return false;
        }
 
        /**
-       * Is this in a namespace that allows actual pages?
-       *
-       * @return \type{\bool} TRUE or FALSE
-       */
+        * Is this in a namespace that allows actual pages?
+        *
+        * @return Bool
+        * @internal note -- uses hardcoded namespace index instead of constants
+        */
        public function canExist() {
                return $this->mNamespace >= 0 && $this->mNamespace != NS_MEDIA;
        }
@@ -3541,34 +3849,38 @@ class Title {
 
        /**
         * Get the last touched timestamp
-        * @param Database $db, optional db
-        * @return \type{\string} Last touched timestamp
+        *
+        * @param $db DatabaseBase: optional db
+        * @return String Last touched timestamp
         */
        public function getTouched( $db = null ) {
-               $db = isset($db) ? $db : wfGetDB( DB_SLAVE );
+               $db = isset( $db ) ? $db : wfGetDB( DB_SLAVE );
                $touched = $db->selectField( 'page', 'page_touched', $this->pageCond(), __METHOD__ );
                return $touched;
        }
 
        /**
         * Get the timestamp when this page was updated since the user last saw it.
-        * @param User $user
-        * @return mixed string/NULL
+        *
+        * @param $user User
+        * @return Mixed: string/null
         */
        public function getNotificationTimestamp( $user = null ) {
                global $wgUser, $wgShowUpdatedMarker;
                // Assume current user if none given
-               if( !$user ) $user = $wgUser;
+               if ( !$user ) {
+                       $user = $wgUser;
+               }
                // Check cache first
                $uid = $user->getId();
-               if( isset($this->mNotificationTimestamp[$uid]) ) {
+               if ( isset( $this->mNotificationTimestamp[$uid] ) ) {
                        return $this->mNotificationTimestamp[$uid];
                }
-               if( !$uid || !$wgShowUpdatedMarker ) {
+               if ( !$uid || !$wgShowUpdatedMarker ) {
                        return $this->mNotificationTimestamp[$uid] = false;
                }
                // Don't cache too much!
-               if( count($this->mNotificationTimestamp) >= self::CACHE_MAX ) {
+               if ( count( $this->mNotificationTimestamp ) >= self::CACHE_MAX ) {
                        $this->mNotificationTimestamp = array();
                }
                $dbr = wfGetDB( DB_SLAVE );
@@ -3585,22 +3897,24 @@ class Title {
 
        /**
         * Get the trackback URL for this page
-        * @return \type{\string} Trackback URL
+        *
+        * @return String Trackback URL
         */
        public function trackbackURL() {
                global $wgScriptPath, $wgServer, $wgScriptExtension;
 
                return "$wgServer$wgScriptPath/trackback$wgScriptExtension?article="
-                       . htmlspecialchars(urlencode($this->getPrefixedDBkey()));
+                       . htmlspecialchars( urlencode( $this->getPrefixedDBkey() ) );
        }
 
        /**
         * Get the trackback RDF for this page
-        * @return \type{\string} Trackback RDF
+        *
+        * @return String Trackback RDF
         */
        public function trackbackRDF() {
-               $url = htmlspecialchars($this->getFullURL());
-               $title = htmlspecialchars($this->getText());
+               $url = htmlspecialchars( $this->getFullURL() );
+               $title = htmlspecialchars( $this->getText() );
                $tburl = $this->trackbackURL();
 
                // Autodiscovery RDF is placed in comments so HTML validator
@@ -3623,7 +3937,9 @@ class Title {
 
        /**
         * Generate strings used for xml 'id' names in monobook tabs
-        * @return \type{\string} XML 'id' name
+        *
+        * @param $prepend string defaults to 'nstab-'
+        * @return String XML 'id' name
         */
        public function getNamespaceKey( $prepend = 'nstab-' ) {
                global $wgContLang;
@@ -3650,9 +3966,20 @@ class Title {
                return $prepend . $namespaceKey;
        }
 
+       /**
+        * Returns true if this is a special page.
+        *
+        * @return boolean
+        */
+       public function isSpecialPage() {
+               return $this->getNamespace() == NS_SPECIAL;
+       }
+
        /**
         * Returns true if this title resolves to the named special page
-        * @param $name \type{\string} The special page name
+        *
+        * @param $name String The special page name
+        * @return boolean
         */
        public function isSpecial( $name ) {
                if ( $this->getNamespace() == NS_SPECIAL ) {
@@ -3666,7 +3993,9 @@ class Title {
 
        /**
         * If the Title refers to a special page alias which is not the local default,
-        * @return \type{Title} A new Title which points to the local default. Otherwise, returns $this.
+        *
+        * @return Title A new Title which points to the local default.
+        *         Otherwise, returns $this.
         */
        public function fixSpecialName() {
                if ( $this->getNamespace() == NS_SPECIAL ) {
@@ -3686,7 +4015,7 @@ class Title {
         * In other words, is this a content page, for the purposes of calculating
         * statistics, etc?
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Boolean
         */
        public function isContentPage() {
                return MWNamespace::isContent( $this->getNamespace() );
@@ -3695,9 +4024,9 @@ class Title {
        /**
         * Get all extant redirects to this Title
         *
-        * @param $ns \twotypes{\int,\null} Single namespace to consider;
+        * @param $ns Int|Null Single namespace to consider;
         *            NULL to consider all namespaces
-        * @return \type{\arrayof{Title}} Redirects to this title
+        * @return Array of Title redirects to this title
         */
        public function getRedirectsHere( $ns = null ) {
                $redirs = array();
@@ -3708,7 +4037,9 @@ class Title {
                        'rd_title' => $this->getDBkey(),
                        'rd_from = page_id'
                );
-               if ( !is_null($ns) ) $where['page_namespace'] = $ns;
+               if ( !is_null( $ns ) ) {
+                       $where['page_namespace'] = $ns;
+               }
 
                $res = $dbr->select(
                        array( 'redirect', 'page' ),
@@ -3717,8 +4048,7 @@ class Title {
                        __METHOD__
                );
 
-
-               foreach( $res as $row ) {
+               foreach ( $res as $row ) {
                        $redirs[] = self::newFromRow( $row );
                }
                return $redirs;
@@ -3727,18 +4057,18 @@ class Title {
        /**
         * Check if this Title is a valid redirect target
         *
-        * @return \type{\bool} TRUE or FALSE
+        * @return Bool
         */
        public function isValidRedirectTarget() {
                global $wgInvalidRedirectTargets;
 
                // invalid redirect targets are stored in a global array, but explicity disallow Userlogout here
-               if( $this->isSpecial( 'Userlogout' ) ) {
+               if ( $this->isSpecial( 'Userlogout' ) ) {
                        return false;
                }
 
-               foreach( $wgInvalidRedirectTargets as $target ) {
-                       if( $this->isSpecial( $target ) ) {
+               foreach ( $wgInvalidRedirectTargets as $target ) {
+                       if ( $this->isSpecial( $target ) ) {
                                return false;
                        }
                }
@@ -3748,6 +4078,8 @@ class Title {
 
        /**
         * Get a backlink cache object
+        *
+        * @return object BacklinkCache
         */
        function getBacklinkCache() {
                if ( is_null( $this->mBacklinkCache ) ) {
@@ -3759,11 +4091,11 @@ class Title {
        /**
         * Whether the magic words __INDEX__ and __NOINDEX__ function for
         * this page.
-        * @return Bool
+        *
+        * @return Boolean
         */
-       public function canUseNoindex(){
-               global $wgArticleRobotPolicies, $wgContentNamespaces,
-                      $wgExemptFromUserRobotsControl;
+       public function canUseNoindex() {
+               global $wgContentNamespaces, $wgExemptFromUserRobotsControl;
 
                $bannedNamespaces = is_null( $wgExemptFromUserRobotsControl )
                        ? $wgContentNamespaces
@@ -3773,16 +4105,43 @@ class Title {
 
        }
 
+       /**
+        * Returns restriction types for the current Title
+        *
+        * @return array applicable restriction types
+        */
        public function getRestrictionTypes() {
                global $wgRestrictionTypes;
-               $types = $this->exists() ? $wgRestrictionTypes : array('create');
+               
+               $types = $this->exists() ? $wgRestrictionTypes : array( 'create' );
 
-               if ( $this->getNamespace() == NS_FILE ) {
-                       $types[] = 'upload';
+               if ( $this->getNamespace() != NS_FILE ) {
+                       $types = array_diff( $types, array( 'upload' ) );
                }
-
+               
                wfRunHooks( 'TitleGetRestrictionTypes', array( $this, &$types ) );
-
+               
                return $types;
        }
+
+       /**
+        * Returns the raw sort key to be used for categories, with the specified
+        * prefix.  This will be fed to Language::convertToSortkey() to get a
+        * binary sortkey that can be used for actual sorting.
+        *
+        * @param $prefix string The prefix to be used, specified using
+        *   {{defaultsort:}} or like [[Category:Foo|prefix]].  Empty for no
+        *   prefix.
+        * @return string
+        */
+       public function getCategorySortkey( $prefix = '' ) {
+               $unprefixed = $this->getText();
+               if ( $prefix !== '' ) {
+                       # Separate with a null byte, so the unprefixed part is only used as
+                       # a tiebreaker when two pages have the exact same prefix -- null
+                       # sorts before everything else (hopefully).
+                       return "$prefix\0$unprefixed";
+               }
+               return $unprefixed;
+       }
 }