summaryrefslogtreecommitdiff
path: root/includes/Title.php
diff options
context:
space:
mode:
Diffstat (limited to 'includes/Title.php')
-rw-r--r--includes/Title.php1904
1 files changed, 1175 insertions, 729 deletions
diff --git a/includes/Title.php b/includes/Title.php
index be41a85a..5ae2f1a0 100644
--- a/includes/Title.php
+++ b/includes/Title.php
@@ -4,24 +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 );
+/**
+ * @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( '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
@@ -30,51 +40,60 @@ 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
* instead of spaces, possibly including namespace and
* interwiki prefixes
@@ -83,26 +102,27 @@ class Title {
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' );
}
@@ -114,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;
@@ -156,6 +176,7 @@ 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
*/
@@ -171,7 +192,7 @@ class Title {
}
$t->mDbkeyform = str_replace( ' ', '_', $url );
- if( $t->secureAndSplit() ) {
+ if ( $t->secureAndSplit() ) {
return $t;
} else {
return null;
@@ -182,13 +203,13 @@ 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
+ * @param $flags \type{\int} use Title::GAID_FOR_UPDATE to use master
* @return \type{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;
@@ -198,6 +219,7 @@ 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
*/
@@ -206,28 +228,37 @@ class Title {
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
*/
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;
}
@@ -242,11 +273,12 @@ class Title {
* @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 "#")
+ * @param $interwiki \type{\string} The interwiki prefix
* @return \type{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 );
@@ -264,12 +296,13 @@ class Title {
* @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 "#")
+ * @param $interwiki \type{\string} The interwiki prefix
* @return \type{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;
@@ -278,6 +311,7 @@ class Title {
/**
* Create a new Title for the Main Page
+ *
* @return \type{Title} the new object
*/
public static function newMainPage() {
@@ -295,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 );
@@ -328,23 +362,25 @@ class Title {
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;
@@ -364,16 +400,16 @@ class 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.
@@ -381,7 +417,7 @@ class Title {
}
$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;
@@ -390,12 +426,13 @@ 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
* if no such article was found
@@ -403,11 +440,15 @@ class Title {
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;
@@ -415,6 +456,7 @@ class Title {
/**
* Get a regex character class describing the legal characters in a link
+ *
* @return \type{\string} the list of characters, not delimited
*/
public static function legalChars() {
@@ -453,16 +495,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 "#")
+ * @param $interwiki \type{\string} The interwiki prefix
* @return \type{\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;
}
@@ -491,14 +538,32 @@ class Title {
* @return \type{\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 \type{\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 ) {
# Note that we don't urlencode the fragment. urlencoded Unicode
@@ -508,34 +573,43 @@ class Title {
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
*/
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
*/
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;
@@ -547,52 +621,65 @@ class Title {
//
// 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
*/
function getUserCaseDBKey() {
return $this->mUserCaseDBKey;
}
+
/**
* Get the namespace text of the subject (rather than talk) page
+ *
* @return \type{\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
*/
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
*/
public function canTalk() {
return( MWNamespace::canTalk( $this->mNamespace ) );
}
+
/**
* Get the interwiki prefix (or null string)
+ *
* @return \type{\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
*/
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
@@ -604,14 +691,17 @@ class Title {
return '#' . Title::escapeFragmentForURL( $this->mFragment );
}
}
+
/**
* Get the default namespace index, for when there is no namespace
+ *
* @return \type{\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
* search index
*/
@@ -621,6 +711,7 @@ class Title {
/**
* Get the prefixed database key form
+ *
* @return \type{\string} the prefixed title, with underscores and
* any interwiki and namespace prefixes
*/
@@ -633,6 +724,7 @@ class Title {
/**
* Get the prefixed title with spaces.
* This is the form usually used for display
+ *
* @return \type{\string} the prefixed title, with spaces
*/
public function getPrefixedText() {
@@ -647,12 +739,13 @@ class Title {
/**
* Get the prefixed title with spaces, plus any fragment
* (part beginning with '#')
+ *
* @return \type{\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;
@@ -660,34 +753,38 @@ class Title {
/**
* Get the base name, i.e. the leftmost parts before the /
+ *
* @return \type{\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
*/
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
*/
public function getSubpageUrlForm() {
@@ -698,6 +795,7 @@ class Title {
/**
* Get a URL-encoded title (not an actual URL) including interwiki
+ *
* @return \type{\string} the URL-encoded form
*/
public function getPrefixedURL() {
@@ -717,9 +815,9 @@ class Title {
* @return \type{\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 );
}
@@ -729,11 +827,11 @@ 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 != '' ) {
@@ -755,7 +853,8 @@ 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).
@@ -764,19 +863,12 @@ class Title {
*/
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 ) {
@@ -789,8 +881,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;
@@ -804,15 +896,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 );
}
}
@@ -827,7 +921,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;
}
}
@@ -854,9 +948,9 @@ class Title {
*/
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();
@@ -868,6 +962,7 @@ 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
*/
@@ -904,11 +999,14 @@ class Title {
/**
* Get the edit URL for this Title
+ *
* @return \type{\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;
@@ -917,6 +1015,7 @@ 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
*/
public function getEscapedText() {
@@ -925,23 +1024,27 @@ class Title {
/**
* Is this Title interwiki?
+ *
* @return \type{\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)
+ * @param $action \type{\string} Action to check (default: edit)
* @return \type{\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
@@ -956,7 +1059,8 @@ class Title {
/**
* Does the title correspond to a protected article?
- * @param $what \type{\string} the action the page is protected from,
+ *
+ * @param $action \type{\string} the action the page is protected from,
* by default checks all actions.
* @return \type{\bool}
*/
@@ -966,15 +1070,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;
}
}
@@ -986,11 +1091,15 @@ class Title {
/**
* Is this a conversion table for the LanguageConverter?
+ *
* @return \type{\bool}
*/
public function isConversionTable() {
- if($this->getNamespace() == NS_MEDIAWIKI
- && strpos( $this->getText(), 'Conversiontable' ) !== false ) {
+ if(
+ $this->getNamespace() == NS_MEDIAWIKI &&
+ strpos( $this->getText(), 'Conversiontable' ) !== false
+ )
+ {
return true;
}
@@ -999,13 +1108,14 @@ class Title {
/**
* Is $wgUser watching this page?
+ *
* @return \type{\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 );
@@ -1039,10 +1149,11 @@ class Title {
*/
public function isNamespaceProtected() {
global $wgNamespaceProtection, $wgUser;
- if( isset( $wgNamespaceProtection[ $this->mNamespace ] ) ) {
- foreach( (array)$wgNamespaceProtection[ $this->mNamespace ] as $right ) {
- if( $right != '' && !$wgUser->isAllowed( $right ) )
+ if ( isset( $wgNamespaceProtection[$this->mNamespace] ) ) {
+ foreach ( (array)$wgNamespaceProtection[$this->mNamespace] as $right ) {
+ if ( $right != '' && !$wgUser->isAllowed( $right ) ) {
return true;
+ }
}
}
return false;
@@ -1050,13 +1161,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}
*/
public function userCan( $action, $doExpensiveQueries = true ) {
global $wgUser;
- return ($this->getUserPermissionsErrorsInternal( $action, $wgUser, $doExpensiveQueries, true ) === array());
+ return ( $this->getUserPermissionsErrorsInternal( $action, $wgUser, $doExpensiveQueries, true ) === array() );
}
/**
@@ -1071,79 +1183,14 @@ class Title {
* @return \type{\array} 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] );
}
}
@@ -1151,36 +1198,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;
@@ -1193,135 +1239,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() ) {
+ $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 {
@@ -1329,74 +1423,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 \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.
+ */
+ 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;
@@ -1404,6 +1624,8 @@ class Title {
/**
* Is this title subject to title protection?
+ * Title protection is the one applied against creation of such title.
+ *
* @return \type{\mixed} An associative array representing any existent title
* protection, or false if there's none.
*/
@@ -1414,75 +1636,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
+ * @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 );
@@ -1498,9 +1723,12 @@ 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;
}
/**
@@ -1515,7 +1743,8 @@ class Title {
/**
* Can $wgUser read this page?
- * @return \type{\bool} TRUE or FALSE
+ *
+ * @return \type{\bool}
* @todo fold these checks into userCan()
*/
public function userCanRead() {
@@ -1524,21 +1753,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;
@@ -1554,10 +1783,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;
@@ -1566,14 +1796,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;
}
@@ -1583,33 +1813,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;
+ }
}
}
@@ -1618,7 +1850,8 @@ class Title {
/**
* Is this a talk page of some sort?
- * @return \type{\bool} TRUE or FALSE
+ *
+ * @return \type{\bool}
*/
public function isTalkPage() {
return MWNamespace::isTalk( $this->getNamespace() );
@@ -1626,7 +1859,8 @@ class Title {
/**
* Is this a subpage?
- * @return \type{\bool} TRUE or FALSE
+ *
+ * @return \type{\bool}
*/
public function isSubpage() {
return MWNamespace::hasSubpages( $this->mNamespace )
@@ -1636,10 +1870,11 @@ class Title {
/**
* Does this have subpages? (Warning, usually requires an extra DB query.)
- * @return \type{\bool} TRUE or FALSE
+ *
+ * @return \type{\bool}
*/
public function hasSubpages() {
- if( !MWNamespace::hasSubpages( $this->mNamespace ) ) {
+ if ( !MWNamespace::hasSubpages( $this->mNamespace ) ) {
# Duh
return false;
}
@@ -1648,32 +1883,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
* @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' ),
@@ -1688,7 +1927,7 @@ class Title {
* Could this page contain custom CSS or JavaScript, based
* on the title?
*
- * @return \type{\bool} TRUE or FALSE
+ * @return \type{\bool}
*/
public function isCssOrJsPage() {
return $this->mNamespace == NS_MEDIAWIKI
@@ -1697,69 +1936,75 @@ class Title {
/**
* Is this a .css or .js subpage of a user page?
- * @return \type{\bool} TRUE or FALSE
+ * @return \type{\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 \type{\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 \type{\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 \type{\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 \type{\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 \type{\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 ) );
}
/**
@@ -1775,17 +2020,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 \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 $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 );
}
@@ -1794,22 +2042,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 {
@@ -1818,18 +2069,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
@@ -1837,9 +2088,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;
@@ -1849,23 +2100,28 @@ 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;
}
+
+ wfProfileOut( __METHOD__ );
return array( $sources, $pagerestrictions );
}
+ /**
+ * Returns cascading restrictions for the current article
+ *
+ * @return Boolean
+ */
function areRestrictionsCascading() {
- if (!$this->mRestrictionsLoaded) {
+ if ( !$this->mRestrictionsLoaded ) {
$this->loadRestrictions();
}
@@ -1874,27 +2130,38 @@ class Title {
/**
* Loads a string into mRestrictions array
+ *
* @param $res \type{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;
@@ -1906,11 +2173,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] ) );
@@ -1923,17 +2190,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,
@@ -1952,7 +2219,7 @@ class Title {
}
}
- if( $purgeExpired ) {
+ if ( $purgeExpired ) {
Title::purgeExpiredRestrictions();
}
}
@@ -1962,34 +2229,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;
}
@@ -2001,13 +2270,17 @@ 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__
+ );
}
/**
@@ -2017,7 +2290,7 @@ class Title {
* @return \type{\arrayof{\string}} 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] )
@@ -2027,11 +2300,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;
@@ -2039,10 +2313,11 @@ class Title {
/**
* Is there a version of this page in the deletion archive?
+ *
* @return \type{\int} the number of archived revisions
*/
public function isDeleted() {
- if( $this->getNamespace() < 0 ) {
+ if ( $this->getNamespace() < 0 ) {
$n = 0;
} else {
$dbr = wfGetDB( DB_SLAVE );
@@ -2050,7 +2325,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__
@@ -2062,10 +2337,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 );
@@ -2073,7 +2349,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__
@@ -2085,22 +2361,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 \type{\int} a bit field; may be Title::GAID_FOR_UPDATE to select
* for update
* @return \type{\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 );
}
}
@@ -2110,14 +2387,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
+ *
+ * @param $flags \type{\int} a bit field; may be Title::GAID_FOR_UPDATE to select for update
* @return \type{\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();
@@ -2129,14 +2408,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
+ *
+ * @param $flags \type{\int} a bit field; may be Title::GAID_FOR_UPDATE to select for update
* @return \type{\bool}
*/
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();
@@ -2147,15 +2428,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 \type{\int} a bit field; may be Title::GAID_FOR_UPDATE to select for update
+ * @return \type{\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;
}
@@ -2173,22 +2460,30 @@ class Title {
$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
*/
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__
@@ -2216,12 +2511,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...
@@ -2240,15 +2539,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;
+ }
}
/**
@@ -2259,6 +2563,7 @@ 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
*/
private function secureAndSplit() {
@@ -2289,7 +2594,7 @@ class Title {
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;
}
@@ -2298,7 +2603,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
@@ -2311,19 +2616,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;
@@ -2334,8 +2640,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;
}
@@ -2356,7 +2664,7 @@ 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!
#
@@ -2365,7 +2673,7 @@ class Title {
}
$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_"
@@ -2374,7 +2682,7 @@ class Title {
# Reject illegal characters.
#
- if( preg_match( $rxTc, $dbkey ) ) {
+ if ( preg_match( $rxTc, $dbkey ) ) {
return false;
}
@@ -2398,7 +2706,7 @@ class Title {
/**
* Magic tilde sequences? Nu-uh!
*/
- if( strpos( $dbkey, '~~~' ) !== false ) {
+ if ( strpos( $dbkey, '~~~' ) !== false ) {
return false;
}
@@ -2424,7 +2732,7 @@ class Title {
* site might be case-sensitive.
*/
$this->mUserCaseDBKey = $dbkey;
- if( $this->mInterwiki == '') {
+ if ( $this->mInterwiki == '' ) {
$dbkey = self::capitalize( $dbkey, $this->mNamespace );
}
@@ -2433,7 +2741,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;
@@ -2444,10 +2752,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;
}
@@ -2476,7 +2784,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() );
@@ -2486,12 +2795,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() );
@@ -2504,7 +2813,9 @@ 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
+ * @param $options Array: may be FOR UPDATE
+ * @param $table String: table name
+ * @param $prefix String: fields prefix
* @return \type{\arrayof{Title}} the Title objects linking here
*/
public function getLinksTo( $options = array(), $table = 'pagelinks', $prefix = 'pl' ) {
@@ -2516,25 +2827,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;
}
@@ -2545,7 +2858,7 @@ 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
+ * @param $options Array: may be FOR UPDATE
* @return \type{\arrayof{Title}} the Title objects linking here
*/
public function getTemplateLinksTo( $options = array() ) {
@@ -2582,7 +2895,7 @@ class Title {
);
$retVal = array();
- foreach( $res as $row ) {
+ foreach ( $res as $row ) {
$retVal[] = Title::makeTitle( $row->pl_namespace, $row->pl_title );
}
return $retVal;
@@ -2604,7 +2917,7 @@ class Title {
);
// purge variant urls as well
- if($wgContLang->hasVariants()){
+ if ( $wgContLang->hasVariants() ) {
$variants = $wgContLang->getVariants();
foreach ( $variants as $vCode ) {
$urls[] = $this->getInternalURL( '', $vCode );
@@ -2628,7 +2941,9 @@ class Title {
/**
* Move this page without authentication
- * @param &$nt \type{Title} the new page Title
+ *
+ * @param $nt \type{Title} the new page Title
+ * @return \type{\mixed} true on success, getUserPermissionsErrors()-like array on failure
*/
public function moveNoAuth( &$nt ) {
return $this->moveTo( $nt, false );
@@ -2637,7 +2952,8 @@ 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 $nt \type{Title} the new title
* @param $auth \type{\bool} indicates whether $wgUser's permissions
* should be checked
* @param $reason \type{\string} is the log summary of the move, used for spam checking
@@ -2647,74 +2963,77 @@ class Title {
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() == '' ) ||
( !$oldid ) ||
( $nt->getDBkey() == '' ) ) {
- $errors[] = array('badarticleerror');
+ $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
@@ -2722,24 +3041,26 @@ 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 $nt \type{Title} the new title
* @param $auth \type{\bool} indicates whether $wgUser's permissions
* should be checked
* @param $reason \type{\string} The reason for the move
@@ -2749,58 +3070,57 @@ class Title {
*/
public function moveTo( &$nt, $auth = true, $reason = '', $createRedirect = true ) {
$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();
}
}
}
- $pageid = $this->getArticleID();
+ $dbw->begin(); # If $file was a LocalFile, its transaction would have closed our own.
+ $pageid = $this->getArticleID( GAID_FOR_UPDATE );
$protected = $this->isProtected();
- if( $nt->exists() ) {
+ if ( $nt->exists() ) {
$err = $this->moveOverExistingRedirect( $nt, $reason, $createRedirect );
- $pageCountChange = ($createRedirect ? 0 : -1);
+ $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 );
}
- if( is_array( $err ) ) {
+ if ( is_array( $err ) ) {
+ # FIXME: What about the File we have already moved?
+ $dbw->rollback();
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' => Collation::singleton()->getSortKey(
+ $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(
@@ -2818,8 +3138,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
@@ -2828,7 +3150,7 @@ class Title {
$oldtitle = $this->getDBkey();
$newtitle = $nt->getDBkey();
- if( $oldnamespace != $newnamespace || $oldtitle != $newtitle ) {
+ if ( $oldnamespace != $newnamespace || $oldtitle != $newtitle ) {
WatchedItem::duplicateEntries( $this, $nt );
}
@@ -2838,28 +3160,30 @@ class Title {
$u = new SearchUpdate( $redirid, $this->getPrefixedDBkey(), '' );
$u->doUpdate();
+ $dbw->commit();
+
# 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 ) {
@@ -2868,7 +3192,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() );
}
@@ -2882,7 +3207,7 @@ class Title {
* Move page to a title which is at present a redirect to the
* source page
*
- * @param &$nt \type{Title} the page to move to, which should currently
+ * @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.
@@ -2918,8 +3243,9 @@ class Title {
if ( !$dbw->cascadingDeletes() ) {
$dbw->delete( 'revision', array( 'rev_page' => $newid ), __METHOD__ );
global $wgUseTrackbacks;
- if ($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__ );
@@ -2939,12 +3265,12 @@ class Title {
$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( $now ),
'page_namespace' => $nt->getNamespace(),
'page_title' => $nt->getDBkey(),
'page_latest' => $nullRevId,
@@ -2955,7 +3281,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 );
@@ -2967,7 +3293,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...
@@ -2999,13 +3325,14 @@ class Title {
/**
* Move page to non-existing title.
- * @param &$nt \type{Title} the new 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, $wgContLang;
+ global $wgUser, $wgContLang;
$comment = wfMsgForContent( '1movedto2', $this->getPrefixedText(), $nt->getPrefixedText() );
if ( $reason ) {
@@ -3016,7 +3343,6 @@ class Title {
# Truncate for whole multibyte characters. +5 bytes for ellipsis
$comment = $wgContLang->truncate( $comment, 250 );
- $newid = $nt->getArticleID();
$oldid = $this->getArticleID();
$latest = $this->getLatestRevId();
@@ -3031,7 +3357,7 @@ class Title {
$nullRevId = $nullRevision->insertOn( $dbw );
$article = new Article( $this );
- wfRunHooks( 'NewRevisionFromEditComplete', array($article, $nullRevision, $latest, $wgUser) );
+ wfRunHooks( 'NewRevisionFromEditComplete', array( $article, $nullRevision, $latest, $wgUser ) );
# Rename page entry
$dbw->update( 'page',
@@ -3046,7 +3372,7 @@ class Title {
);
$nt->resetArticleID( $oldid );
- if( $createRedirect || !$wgUser->isAllowed('suppressredirect') ) {
+ if ( $createRedirect || !$wgUser->isAllowed( 'suppressredirect' ) ) {
# Insert redirect
$mwRedir = MagicWord::get( 'redirect' );
$redirectText = $mwRedir->getSynonym( 0 ) . ' [[' . $nt->getPrefixedText() . "]]\n";
@@ -3059,7 +3385,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 ) );
# Record the just-created redirect's linking to the page
$dbw->insert( 'pagelinks',
@@ -3084,11 +3410,11 @@ class Title {
# 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
@@ -3100,22 +3426,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 );
@@ -3125,16 +3454,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();
@@ -3144,7 +3475,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;
@@ -3157,7 +3488,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 \type{\bool}
*/
public function isSingleRevRedirect() {
$dbw = wfGetDB( DB_MASTER );
@@ -3169,14 +3500,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(),
@@ -3187,28 +3518,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
+ * @param $nt \type{Title} the new title to check
* @return \type{\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;
}
@@ -3220,7 +3550,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" );
@@ -3259,25 +3589,25 @@ 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
*/
@@ -3285,22 +3615,21 @@ class Title {
$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;
}
@@ -3311,7 +3640,7 @@ class Title {
* @return \type{\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 {
@@ -3323,14 +3652,14 @@ 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 $flags \type{\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__,
@@ -3342,14 +3671,14 @@ 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 $flags \type{\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__,
@@ -3360,19 +3689,21 @@ class Title {
/**
* Get the first revision of the page
*
- * @param $flags \type{\int} GAID_FOR_UPDATE
+ * @param $flags \type{\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 );
@@ -3392,11 +3723,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() ),
@@ -3425,9 +3756,32 @@ class Title {
}
/**
+ * Get the number of authors between the given revision IDs.
+ * Used for diffs and other things that really need it.
+ *
+ * @param $fromRevId \type{\int} Revision ID (first before range)
+ * @param $toRevId \type{\int} Revision ID (first after range)
+ * @param $limit \type{\int} Maximum number of authors
+ * @param $flags \type{\int} Title::GAID_FOR_UPDATE
+ * @return \type{\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
+ * @param $title \type{Title}
* @return \type{\bool} TRUE or FALSE
*/
public function equals( Title $title ) {
@@ -3439,9 +3793,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();
@@ -3464,7 +3820,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 \type{\bool}
*/
public function exists() {
return $this->getArticleId() != 0;
@@ -3484,30 +3840,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 \type{\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;
}
}
@@ -3517,17 +3871,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 \type{\bool}
*/
public function isKnown() {
- return $this->exists() || $this->isAlwaysKnown();
+ return $this->isAlwaysKnown() || $this->exists();
}
/**
- * Is this in a namespace that allows actual pages?
- *
- * @return \type{\bool} TRUE or FALSE
- */
+ * 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}
+ * @internal note -- uses hardcoded namespace index instead of constants
+ */
public function canExist() {
return $this->mNamespace >= 0 && $this->mNamespace != NS_MEDIA;
}
@@ -3549,34 +3927,39 @@ class Title {
/**
* Get the last touched timestamp
- * @param Database $db, optional db
+ *
+ * @param $db DatabaseBase: optional db
* @return \type{\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]) ) {
+ // avoid isset here, as it'll return false for null entries
+ if ( array_key_exists( $uid, $this->mNotificationTimestamp ) ) {
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 );
@@ -3593,22 +3976,24 @@ class Title {
/**
* Get the trackback URL for this page
+ *
* @return \type{\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
*/
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
@@ -3631,6 +4016,8 @@ class Title {
/**
* Generate strings used for xml 'id' names in monobook tabs
+ *
+ * @param $prepend string defaults to 'nstab-'
* @return \type{\string} XML 'id' name
*/
public function getNamespaceKey( $prepend = 'nstab-' ) {
@@ -3660,14 +4047,18 @@ class Title {
/**
* Returns true if this is a special page.
+ *
+ * @return boolean
*/
- public function isSpecialPage( ) {
+ 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
+ * @return boolean
*/
public function isSpecial( $name ) {
if ( $this->getNamespace() == NS_SPECIAL ) {
@@ -3681,7 +4072,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 \type{Title} A new Title which points to the local default.
+ * Otherwise, returns $this.
*/
public function fixSpecialName() {
if ( $this->getNamespace() == NS_SPECIAL ) {
@@ -3701,7 +4094,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() );
@@ -3723,7 +4116,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' ),
@@ -3732,8 +4127,7 @@ class Title {
__METHOD__
);
-
- foreach( $res as $row ) {
+ foreach ( $res as $row ) {
$redirs[] = self::newFromRow( $row );
}
return $redirs;
@@ -3742,18 +4136,18 @@ class Title {
/**
* Check if this Title is a valid redirect target
*
- * @return \type{\bool} TRUE or FALSE
+ * @return \type{\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;
}
}
@@ -3763,6 +4157,8 @@ class Title {
/**
* Get a backlink cache object
+ *
+ * @return object BacklinkCache
*/
function getBacklinkCache() {
if ( is_null( $this->mBacklinkCache ) ) {
@@ -3774,11 +4170,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
@@ -3788,16 +4184,66 @@ 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');
-
- if ( $this->getNamespace() == NS_FILE ) {
- $types[] = 'upload';
+ $types = self::getFilteredRestrictionTypes( $this->exists() );
+
+ if ( $this->getNamespace() != NS_FILE ) {
+ # Remove the upload restriction for non-file titles
+ $types = array_diff( $types, array( 'upload' ) );
}
-
+
wfRunHooks( 'TitleGetRestrictionTypes', array( $this, &$types ) );
+
+ wfDebug( __METHOD__ . ': applicable restriction types for ' .
+ $this->getPrefixedText() . ' are ' . implode( ',', $types ) );
return $types;
}
+ /**
+ * Get a filtered list of all restriction types supported by this wiki.
+ * @param bool $exists True to get all restriction types that apply to
+ * titles that do exist, False for all restriction types that apply to
+ * titles that do not exist
+ * @return array
+ */
+ public static function getFilteredRestrictionTypes( $exists = true ) {
+ global $wgRestrictionTypes;
+ $types = $wgRestrictionTypes;
+ if ( $exists ) {
+ # Remove the create restriction for existing titles
+ $types = array_diff( $types, array( 'create' ) );
+ } else {
+ # Only the create and upload restrictions apply to non-existing titles
+ $types = array_intersect( $types, array( 'create', 'upload' ) );
+ }
+ return $types;
+ }
+
+ /**
+ * Returns the raw sort key to be used for categories, with the specified
+ * prefix. This will be fed to Collation::getSortKey() 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 line feed, so the unprefixed part is only used as
+ # a tiebreaker when two pages have the exact same prefix.
+ # In UCA, tab is the only character that can sort above LF
+ # so we strip both of them from the original prefix.
+ $prefix = strtr( $prefix, "\n\t", ' ' );
+ return "$prefix\n$unprefixed";
+ }
+ return $unprefixed;
+ }
}