This commit is contained in:
Niklas Laxström 2011-12-16 09:29:18 +00:00
parent b61f0adad9
commit e86fff1335

View file

@ -348,12 +348,12 @@ class SpecialPage {
$this->mListed = $listed; $this->mListed = $listed;
$this->mIncludable = $includable; $this->mIncludable = $includable;
if ( !$function ) { if ( !$function ) {
$this->mFunction = 'wfSpecial'.$name; $this->mFunction = 'wfSpecial' . $name;
} else { } else {
$this->mFunction = $function; $this->mFunction = $function;
} }
if ( $file === 'default' ) { if ( $file === 'default' ) {
$this->mFile = dirname(__FILE__) . "/specials/Special$name.php"; $this->mFile = dirname( __FILE__ ) . "/specials/Special$name.php";
} else { } else {
$this->mFile = $file; $this->mFile = $file;
} }
@ -372,7 +372,7 @@ class SpecialPage {
wfDeprecated( __METHOD__, '1.17' ); wfDeprecated( __METHOD__, '1.17' );
// Sometimes $fName is SpecialPage, sometimes it's specialpage. <3 PHP // Sometimes $fName is SpecialPage, sometimes it's specialpage. <3 PHP
if( strtolower( $fName ) == 'specialpage' ) { if ( strtolower( $fName ) == 'specialpage' ) {
$name = isset( $a[0] ) ? $a[0] : ''; $name = isset( $a[0] ) ? $a[0] : '';
$restriction = isset( $a[1] ) ? $a[1] : ''; $restriction = isset( $a[1] ) ? $a[1] : '';
$listed = isset( $a[2] ) ? $a[2] : true; $listed = isset( $a[2] ) ? $a[2] : true;
@ -438,7 +438,7 @@ class SpecialPage {
* @param $x Bool * @param $x Bool
* @return Bool * @return Bool
*/ */
function listed( $x = null) { function listed( $x = null ) {
return wfSetVar( $this->mListed, $x ); return wfSetVar( $this->mListed, $x );
} }
@ -446,7 +446,7 @@ class SpecialPage {
* Whether it's allowed to transclude the special page via {{Special:Foo/params}} * Whether it's allowed to transclude the special page via {{Special:Foo/params}}
* @return Bool * @return Bool
*/ */
public function isIncludable(){ public function isIncludable() {
return $this->mIncludable; return $this->mIncludable;
} }
@ -466,7 +466,7 @@ class SpecialPage {
* @return Mixed * @return Mixed
* @deprecated since 1.18 * @deprecated since 1.18
*/ */
function restriction( $x = null) { wfDeprecated( __METHOD__, '1.18' ); return wfSetVar( $this->mRestriction, $x ); } function restriction( $x = null ) { wfDeprecated( __METHOD__, '1.18' ); return wfSetVar( $this->mRestriction, $x ); }
/** /**
* These mutators are very evil, as the relevant variables should not mutate. So * These mutators are very evil, as the relevant variables should not mutate. So
@ -475,7 +475,7 @@ class SpecialPage {
* @return Mixed * @return Mixed
* @deprecated since 1.18 * @deprecated since 1.18
*/ */
function func( $x = null) { wfDeprecated( __METHOD__, '1.18' ); return wfSetVar( $this->mFunction, $x ); } function func( $x = null ) { wfDeprecated( __METHOD__, '1.18' ); return wfSetVar( $this->mFunction, $x ); }
/** /**
* These mutators are very evil, as the relevant variables should not mutate. So * These mutators are very evil, as the relevant variables should not mutate. So
@ -484,7 +484,7 @@ class SpecialPage {
* @return Mixed * @return Mixed
* @deprecated since 1.18 * @deprecated since 1.18
*/ */
function file( $x = null) { wfDeprecated( __METHOD__, '1.18' ); return wfSetVar( $this->mFile, $x ); } function file( $x = null ) { wfDeprecated( __METHOD__, '1.18' ); return wfSetVar( $this->mFile, $x ); }
/** /**
* These mutators are very evil, as the relevant variables should not mutate. So * These mutators are very evil, as the relevant variables should not mutate. So
@ -536,7 +536,7 @@ class SpecialPage {
public function isRestricted() { public function isRestricted() {
global $wgGroupPermissions; global $wgGroupPermissions;
// DWIM: If all anons can do something, then it is not restricted // DWIM: If all anons can do something, then it is not restricted
return $this->mRestriction != '' && empty($wgGroupPermissions['*'][$this->mRestriction]); return $this->mRestriction != '' && empty( $wgGroupPermissions['*'][$this->mRestriction] );
} }
/** /**
@ -605,7 +605,7 @@ class SpecialPage {
$func = $this->mFunction; $func = $this->mFunction;
// only load file if the function does not exist // only load file if the function does not exist
if( !is_callable($func) && $this->mFile ) { if ( !is_callable( $func ) && $this->mFile ) {
require_once( $this->mFile ); require_once( $this->mFile );
} }
$this->outputHeader(); $this->outputHeader();
@ -623,7 +623,7 @@ class SpecialPage {
function outputHeader( $summaryMessageKey = '' ) { function outputHeader( $summaryMessageKey = '' ) {
global $wgContLang; global $wgContLang;
if( $summaryMessageKey == '' ) { if ( $summaryMessageKey == '' ) {
$msg = $wgContLang->lc( $this->getName() ) . '-summary'; $msg = $wgContLang->lc( $this->getName() ) . '-summary';
} else { } else {
$msg = $summaryMessageKey; $msg = $summaryMessageKey;
@ -781,7 +781,7 @@ class SpecialPage {
$feedTemplate = wfScript( 'api' ) . '?'; $feedTemplate = wfScript( 'api' ) . '?';
foreach( $wgFeedClasses as $format => $class ) { foreach ( $wgFeedClasses as $format => $class ) {
$theseParams = $params + array( 'feedformat' => $format ); $theseParams = $params + array( 'feedformat' => $format );
$url = $feedTemplate . wfArrayToCGI( $theseParams ); $url = $feedTemplate . wfArrayToCGI( $theseParams );
$this->getOutput()->addFeedLink( $format, $url ); $this->getOutput()->addFeedLink( $format, $url );
@ -878,7 +878,7 @@ abstract class FormSpecialPage extends SpecialPage {
* Maybe do something interesting with the subpage parameter * Maybe do something interesting with the subpage parameter
* @param $par String * @param $par String
*/ */
protected function setParameter( $par ){} protected function setParameter( $par ) {}
/** /**
* Called from execute() to check if the given user can perform this action. * Called from execute() to check if the given user can perform this action.
@ -928,7 +928,7 @@ class UnlistedSpecialPage extends SpecialPage {
parent::__construct( $name, $restriction, false, $function, $file ); parent::__construct( $name, $restriction, false, $function, $file );
} }
public function isListed(){ public function isListed() {
return false; return false;
} }
} }
@ -944,7 +944,7 @@ class IncludableSpecialPage extends SpecialPage {
parent::__construct( $name, $restriction, $listed, $function, $file, true ); parent::__construct( $name, $restriction, $listed, $function, $file, true );
} }
public function isIncludable(){ public function isIncludable() {
return true; return true;
} }
} }
@ -961,7 +961,7 @@ abstract class RedirectSpecialPage extends UnlistedSpecialPage {
// Query parameteres added by redirects // Query parameteres added by redirects
protected $mAddedRedirectParams = array(); protected $mAddedRedirectParams = array();
public function execute( $par ){ public function execute( $par ) {
$redirect = $this->getRedirect( $par ); $redirect = $this->getRedirect( $par );
$query = $this->getRedirectQuery(); $query = $this->getRedirectQuery();
// Redirect to a page title with possible query parameters // Redirect to a page title with possible query parameters
@ -1001,13 +1001,13 @@ abstract class RedirectSpecialPage extends UnlistedSpecialPage {
public function getRedirectQuery() { public function getRedirectQuery() {
$params = array(); $params = array();
foreach( $this->mAllowedRedirectParams as $arg ) { foreach ( $this->mAllowedRedirectParams as $arg ) {
if( $this->getRequest()->getVal( $arg, null ) !== null ){ if ( $this->getRequest()->getVal( $arg, null ) !== null ) {
$params[$arg] = $this->getRequest()->getVal( $arg ); $params[$arg] = $this->getRequest()->getVal( $arg );
} }
} }
foreach( $this->mAddedRedirectParams as $arg => $val ) { foreach ( $this->mAddedRedirectParams as $arg => $val ) {
$params[$arg] = $val; $params[$arg] = $val;
} }
@ -1044,7 +1044,7 @@ abstract class SpecialRedirectToSpecial extends RedirectSpecialPage {
* ListAdmins --> ListUsers/sysop * ListAdmins --> ListUsers/sysop
*/ */
class SpecialListAdmins extends SpecialRedirectToSpecial { class SpecialListAdmins extends SpecialRedirectToSpecial {
function __construct(){ function __construct() {
parent::__construct( 'Listadmins', 'Listusers', 'sysop' ); parent::__construct( 'Listadmins', 'Listusers', 'sysop' );
} }
} }
@ -1053,7 +1053,7 @@ class SpecialListAdmins extends SpecialRedirectToSpecial {
* ListBots --> ListUsers/bot * ListBots --> ListUsers/bot
*/ */
class SpecialListBots extends SpecialRedirectToSpecial { class SpecialListBots extends SpecialRedirectToSpecial {
function __construct(){ function __construct() {
parent::__construct( 'Listbots', 'Listusers', 'bot' ); parent::__construct( 'Listbots', 'Listusers', 'bot' );
} }
} }
@ -1063,7 +1063,7 @@ class SpecialListBots extends SpecialRedirectToSpecial {
* @todo FIXME: This (and the rest of the login frontend) needs to die a horrible painful death * @todo FIXME: This (and the rest of the login frontend) needs to die a horrible painful death
*/ */
class SpecialCreateAccount extends SpecialRedirectToSpecial { class SpecialCreateAccount extends SpecialRedirectToSpecial {
function __construct(){ function __construct() {
parent::__construct( 'CreateAccount', 'Userlogin', 'signup', array( 'uselang' ) ); parent::__construct( 'CreateAccount', 'Userlogin', 'signup', array( 'uselang' ) );
} }
} }
@ -1158,7 +1158,7 @@ class SpecialPermanentLink extends RedirectSpecialPage {
function getRedirect( $subpage ) { function getRedirect( $subpage ) {
$subpage = intval( $subpage ); $subpage = intval( $subpage );
if( $subpage === 0 ) { if ( $subpage === 0 ) {
# throw an error page when no subpage was given # throw an error page when no subpage was given
throw new ErrorPageError( 'nopagetitle', 'nopagetext' ); throw new ErrorPageError( 'nopagetitle', 'nopagetext' );
} }