*/
private $params = array();
+ /**
+ * TODO: fold $output, etc, into this
+ * @var RequestContext
+ */
+ private $context;
+
/**
* Stores key/value pairs to circumvent global variables
* Note that keys are case-insensitive!
return $default;
}
+ public function request( WebRequest &$x = null ){
+ return wfSetVar( $this->context->request, $x );
+ }
+
+ public function output( OutputPage &$x = null ){
+ return wfSetVar( $this->context->output, $x );
+ }
+
+ public function __construct( RequestContext $context ){
+ $this->context = $context;
+ $this->context->setTitle( $this->parseTitle() );
+ }
+
/**
* Initialization of ... everything
* Performs the request too
*
- * @param $title Title ($wgTitle)
* @param $article Article
- * @param $output OutputPage
- * @param $user User
- * @param $request WebRequest
*/
- public function performRequestForTitle( &$title, &$article, &$output, &$user, $request ) {
+ public function performRequestForTitle( &$article ) {
wfProfileIn( __METHOD__ );
- $output->setTitle( $title );
- if ( $request->getVal( 'printable' ) === 'yes' ) {
- $output->setPrintable();
+ if ( $this->context->request->getVal( 'printable' ) === 'yes' ) {
+ $this->context->output->setPrintable();
}
- wfRunHooks( 'BeforeInitialize', array( &$title, &$article, &$output, &$user, $request, $this ) );
+ wfRunHooks( 'BeforeInitialize', array(
+ &$this->context->title,
+ &$article,
+ &$this->context->output,
+ &$this->context->user,
+ $this->context->request,
+ $this
+ ) );
// If the user is not logged in, the Namespace:title of the article must be in
// the Read array in order for the user to see it. (We have to check here to
// catch special pages etc. We check again in Article::view())
- if ( !is_null( $title ) && !$title->userCanRead() ) {
- $output->loginToUse();
- $this->finalCleanup( $output );
- $output->disable();
+ if ( !is_null( $this->context->title ) && !$this->context->title->userCanRead() ) {
+ $this->context->output->loginToUse();
+ $this->finalCleanup();
+ $this->context->output->disable();
wfProfileOut( __METHOD__ );
return false;
}
// Call handleSpecialCases() to deal with all special requests...
- if ( !$this->handleSpecialCases( $title, $output, $request ) ) {
+ if ( !$this->handleSpecialCases() ) {
// ...otherwise treat it as an article view. The article
// may be a redirect to another article or URL.
- $new_article = $this->initializeArticle( $title, $output, $request );
+ $new_article = $this->initializeArticle();
if ( is_object( $new_article ) ) {
$article = $new_article;
- $this->performAction( $output, $article, $title, $user, $request );
+ $this->performAction( $article );
} elseif ( is_string( $new_article ) ) {
- $output->redirect( $new_article );
+ $this->context->output->redirect( $new_article );
} else {
wfProfileOut( __METHOD__ );
throw new MWException( "Shouldn't happen: MediaWiki::initializeArticle() returned neither an object nor a URL" );
}
/**
- * Checks some initial queries
- * FIXME: rename to parseTitle() ?
+ * Parse $request to get the Title object
*
- * @param $request WebRequest
* @return Title object to be $wgTitle
*/
- /* private */ function checkInitialQueries( WebRequest $request ) {
+ private function parseTitle() {
global $wgContLang;
- $curid = $request->getInt( 'curid' );
- $title = $request->getVal( 'title' );
+ $curid = $this->context->request->getInt( 'curid' );
+ $title = $this->context->request->getVal( 'title' );
- if ( $request->getCheck( 'search' ) ) {
+ if ( $this->context->request->getCheck( 'search' ) ) {
// Compatibility with old search URLs which didn't use Special:Search
// Just check for presence here, so blank requests still
// show the search page when using ugly URLs (bug 8054).
} elseif ( $curid ) {
// URLs like this are generated by RC, because rc_title isn't always accurate
$ret = Title::newFromID( $curid );
- } elseif ( $title == '' && $this->getAction( $request ) != 'delete' ) {
+ } elseif ( $title == '' && $this->getAction() != 'delete' ) {
$ret = Title::newMainPage();
} else {
$ret = Title::newFromURL( $title );
// check variant links so that interwiki links don't have to worry
// about the possible different language variants
- if ( count( $wgContLang->getVariants() ) > 1 && !is_null( $ret ) && $ret->getArticleID() == 0 )
+ if ( count( $wgContLang->getVariants() ) > 1 && !is_null( $ret ) && $ret->getArticleID() == 0 ){
$wgContLang->findVariantLink( $title, $ret );
+ }
}
// For non-special titles, check for implicit titles
if ( is_null( $ret ) || $ret->getNamespace() != NS_SPECIAL ) {
// We can have urls with just ?diff=,?oldid= or even just ?diff=
- $oldid = $request->getInt( 'oldid' );
- $oldid = $oldid ? $oldid : $request->getInt( 'diff' );
+ $oldid = $this->context->request->getInt( 'oldid' );
+ $oldid = $oldid ? $oldid : $this->context->request->getInt( 'diff' );
// Allow oldid to override a changed or missing title
if ( $oldid ) {
$rev = Revision::newFromId( $oldid );
$ret = $rev ? $rev->getTitle() : $ret;
}
}
+
+ if( $ret === null || ( $ret->getDBkey() == '' && $ret->getInterwiki() == '' ) ){
+ $ret = new BadTitle;
+ }
return $ret;
}
+ /**
+ * Get the Title object that we'll be acting on, as specified in the WebRequest
+ * @return Title
+ */
+ public function getTitle(){
+ if( $this->context->title === null ){
+ $this->context->title = $this->parseTitle();
+ }
+ return $this->context->title;
+ }
+
/**
* Initialize some special cases:
* - bad titles
* - redirect loop
* - special pages
*
- * @param $title Title
- * @param $output OutputPage
- * @param $request WebRequest
* @return bool true if the request is already executed
*/
- private function handleSpecialCases( &$title, &$output, $request ) {
+ private function handleSpecialCases() {
wfProfileIn( __METHOD__ );
// Invalid titles. Bug 21776: The interwikis must redirect even if the page name is empty.
- if ( is_null( $title ) || ( ( $title->getDBkey() == '' ) && ( $title->getInterwiki() == '' ) ) ) {
- $title = SpecialPage::getTitleFor( 'Badtitle' );
- $output->setTitle( $title ); // bug 21456
+ if ( $this->context->title instanceof BadTitle ) {
// Die now before we mess up $wgArticle and the skin stops working
throw new ErrorPageError( 'badtitle', 'badtitletext' );
// Interwiki redirects
- } else if ( $title->getInterwiki() != '' ) {
- $rdfrom = $request->getVal( 'rdfrom' );
+ } else if ( $this->context->title->getInterwiki() != '' ) {
+ $rdfrom = $this->context->request->getVal( 'rdfrom' );
if ( $rdfrom ) {
- $url = $title->getFullURL( 'rdfrom=' . urlencode( $rdfrom ) );
+ $url = $this->context->title->getFullURL( 'rdfrom=' . urlencode( $rdfrom ) );
} else {
- $query = $request->getValues();
+ $query = $this->context->request->getValues();
unset( $query['title'] );
- $url = $title->getFullURL( $query );
+ $url = $this->context->title->getFullURL( $query );
}
- /* Check for a redirect loop */
- if ( !preg_match( '/^' . preg_quote( $this->getVal( 'Server' ), '/' ) . '/', $url ) && $title->isLocal() ) {
+ // Check for a redirect loop
+ if ( !preg_match( '/^' . preg_quote( $this->getVal( 'Server' ), '/' ) . '/', $url ) && $this->context->title->isLocal() ) {
// 301 so google et al report the target as the actual url.
- $output->redirect( $url, 301 );
+ $this->context->output->redirect( $url, 301 );
} else {
- $title = SpecialPage::getTitleFor( 'Badtitle' );
- $output->setTitle( $title ); // bug 21456
+ $this->context->title = new BadTitle;
wfProfileOut( __METHOD__ );
throw new ErrorPageError( 'badtitle', 'badtitletext' );
}
// Redirect loops, no title in URL, $wgUsePathInfo URLs, and URLs with a variant
- } else if ( $request->getVal( 'action', 'view' ) == 'view' && !$request->wasPosted()
- && ( $request->getVal( 'title' ) === null || $title->getPrefixedDBKey() != $request->getVal( 'title' ) )
- && !count( array_diff( array_keys( $request->getValues() ), array( 'action', 'title' ) ) ) )
+ } else if ( $this->context->request->getVal( 'action', 'view' ) == 'view' && !$this->context->request->wasPosted()
+ && ( $this->context->request->getVal( 'title' ) === null || $this->context->title->getPrefixedDBKey() != $this->context->request->getVal( 'title' ) )
+ && !count( array_diff( array_keys( $this->context->request->getValues() ), array( 'action', 'title' ) ) ) )
{
- if ( $title->getNamespace() == NS_SPECIAL ) {
- list( $name, $subpage ) = SpecialPage::resolveAliasWithSubpage( $title->getDBkey() );
+ if ( $this->context->title->getNamespace() == NS_SPECIAL ) {
+ list( $name, $subpage ) = SpecialPage::resolveAliasWithSubpage( $this->context->title->getDBkey() );
if ( $name ) {
- $title = SpecialPage::getTitleFor( $name, $subpage );
+ $this->context->title = SpecialPage::getTitleFor( $name, $subpage );
}
}
- $targetUrl = $title->getFullURL();
+ $targetUrl = $this->context->title->getFullURL();
// Redirect to canonical url, make it a 301 to allow caching
- if ( $targetUrl == $request->getFullRequestURL() ) {
+ if ( $targetUrl == $this->context->request->getFullRequestURL() ) {
$message = "Redirect loop detected!\n\n" .
"This means the wiki got confused about what page was " .
"requested; this sometimes happens when moving a wiki " .
wfProfileOut( __METHOD__ );
return false;
} else {
- $output->setSquidMaxage( 1200 );
- $output->redirect( $targetUrl, '301' );
+ $this->context->output->setSquidMaxage( 1200 );
+ $this->context->output->redirect( $targetUrl, '301' );
}
// Special pages
- } else if ( NS_SPECIAL == $title->getNamespace() ) {
- /* actions that need to be made when we have a special pages */
- SpecialPage::executePath( $title );
+ } else if ( NS_SPECIAL == $this->context->title->getNamespace() ) {
+ // actions that need to be made when we have a special pages
+ SpecialPage::executePath( $this->context->title, $this->context );
} else {
- /* No match to special cases */
+ // No match to special cases
wfProfileOut( __METHOD__ );
return false;
}
- /* Did match a special case */
+ // Did match a special case
wfProfileOut( __METHOD__ );
return true;
}
* @param $title Title
* @return Article object
*/
- public static function articleFromTitle( &$title ) {
+ public static function articleFromTitle( &$title, RequestContext &$context ) {
if ( NS_MEDIA == $title->getNamespace() ) {
// FIXME: where should this go?
$title = Title::makeTitle( NS_FILE, $title->getDBkey() );
switch( $title->getNamespace() ) {
case NS_FILE:
- return new ImagePage( $title );
+ $page = new ImagePage( $title );
+ break;
case NS_CATEGORY:
- return new CategoryPage( $title );
+ $page = new CategoryPage( $title );
+ break;
default:
- return new Article( $title );
+ $page = new Article( $title );
}
+ $page->setContext( $context );
+ return $page;
}
/**
* passed through the "action" parameter. Actions disabled in
* $wgDisabledActions will be replaced by "nosuchaction"
*
- * @param $request WebRequest
* @return String: action
*/
- public function getAction( WebRequest $request ) {
+ public function getAction() {
global $wgDisabledActions;
- $action = $request->getVal( 'action', 'view' );
+ $action = $this->context->request->getVal( 'action', 'view' );
// Check for disabled actions
if ( in_array( $action, $wgDisabledActions ) ) {
// Workaround for bug #20966: inability of IE to provide an action dependent
// on which submit button is clicked.
if ( $action === 'historysubmit' ) {
- if ( $request->getBool( 'revisiondelete' ) ) {
+ if ( $this->context->request->getBool( 'revisiondelete' ) ) {
return 'revisiondelete';
- } elseif ( $request->getBool( 'revisionmove' ) ) {
+ } elseif ( $this->context->request->getBool( 'revisionmove' ) ) {
return 'revisionmove';
} else {
return 'view';
* Initialize the object to be known as $wgArticle for "standard" actions
* Create an Article object for the page, following redirects if needed.
*
- * @param $title Title ($wgTitle)
- * @param $output OutputPage ($wgOut)
- * @param $request WebRequest ($wgRequest)
* @return mixed an Article, or a string to redirect to another URL
*/
- private function initializeArticle( &$title, &$output, $request ) {
+ private function initializeArticle() {
wfProfileIn( __METHOD__ );
- $action = $request->getVal( 'action', 'view' );
- $article = self::articleFromTitle( $title );
+ $action = $this->context->request->getVal( 'action', 'view' );
+ $article = self::articleFromTitle( $this->context->title, $this->context );
// NS_MEDIAWIKI has no redirects.
// It is also used for CSS/JS, so performance matters here...
- if ( $title->getNamespace() == NS_MEDIAWIKI ) {
+ if ( $this->context->title->getNamespace() == NS_MEDIAWIKI ) {
wfProfileOut( __METHOD__ );
return $article;
}
// Namespace might change when using redirects
// Check for redirects ...
- $file = ( $title->getNamespace() == NS_FILE ) ? $article->getFile() : null;
+ $file = ( $this->context->title->getNamespace() == NS_FILE ) ? $article->getFile() : null;
if ( ( $action == 'view' || $action == 'render' ) // ... for actions that show content
- && !$request->getVal( 'oldid' ) && // ... and are not old revisions
- !$request->getVal( 'diff' ) && // ... and not when showing diff
- $request->getVal( 'redirect' ) != 'no' && // ... unless explicitly told not to
+ && !$this->context->request->getVal( 'oldid' ) && // ... and are not old revisions
+ !$this->context->request->getVal( 'diff' ) && // ... and not when showing diff
+ $this->context->request->getVal( 'redirect' ) != 'no' && // ... unless explicitly told not to
// ... and the article is not a non-redirect image page with associated file
!( is_object( $file ) && $file->exists() && !$file->getRedirected() ) )
{
$ignoreRedirect = $target = false;
wfRunHooks( 'InitializeArticleMaybeRedirect',
- array( &$title, &$request, &$ignoreRedirect, &$target, &$article ) );
+ array( &$this->context->title, &$this->context->request, &$ignoreRedirect, &$target, &$article ) );
// Follow redirects only for... redirects.
// If $target is set, then a hook wanted to redirect.
}
if ( is_object( $target ) ) {
// Rewrite environment to redirected article
- $rarticle = self::articleFromTitle( $target );
+ $rarticle = self::articleFromTitle( $target, $this->context );
$rarticle->loadPageData();
if ( $rarticle->exists() || ( is_object( $file ) && !$file->isLocal() ) ) {
- $rarticle->setRedirectedFrom( $title );
+ $rarticle->setRedirectedFrom( $this->context->title );
$article = $rarticle;
- $title = $target;
- $output->setTitle( $title );
+ $this->context->title = $target;
}
}
} else {
- $title = $article->getTitle();
+ $this->context->title = $article->getTitle();
}
}
wfProfileOut( __METHOD__ );
/**
* Cleaning up request by doing deferred updates, DB transaction, and the output
- *
- * @param $output OutputPage
*/
- public function finalCleanup( &$output ) {
+ public function finalCleanup() {
wfProfileIn( __METHOD__ );
// Now commit any transactions, so that unreported errors after
// output() don't roll back the whole DB transaction
$factory = wfGetLBFactory();
$factory->commitMasterChanges();
// Output everything!
- $output->output();
+ $this->context->output->output();
// Do any deferred jobs
wfDoUpdates( 'commit' );
// Close the session so that jobs don't access the current session
/**
* Perform one of the "standard" actions
*
- * @param $output OutputPage
* @param $article Article
- * @param $title Title
- * @param $user User
- * @param $request WebRequest
*/
- private function performAction( &$output, &$article, &$title, &$user, &$request ) {
+ private function performAction( &$article ) {
wfProfileIn( __METHOD__ );
- if ( !wfRunHooks( 'MediaWikiPerformAction', array( $output, $article, $title, $user, $request, $this ) ) ) {
+ if ( !wfRunHooks( 'MediaWikiPerformAction', array(
+ $this->context->output, $article, $this->context->title,
+ $this->context->user, $this->context->request, $this ) ) )
+ {
wfProfileOut( __METHOD__ );
return;
}
- $action = $this->getAction( $request );
+ $act = $this->getAction();
- switch( $action ) {
+ $action = Action::factory( $this->getAction(), $article );
+ if( $action instanceof Action ){
+ $action->show();
+ wfProfileOut( __METHOD__ );
+ return;
+ }
+
+ switch( $act ) {
case 'view':
- $output->setSquidMaxage( $this->getVal( 'SquidMaxage' ) );
+ $this->context->output->setSquidMaxage( $this->getVal( 'SquidMaxage' ) );
$article->view();
break;
case 'raw': // includes JS/CSS
$raw->view();
wfProfileOut( __METHOD__ . '-raw' );
break;
- case 'watch':
- case 'unwatch':
- case 'delete':
case 'revert':
case 'rollback':
case 'protect':
case 'render':
case 'deletetrackback':
case 'purge':
- $article->$action();
+ $article->$act();
break;
case 'print':
$article->view();
$rdf->show();
}
break;
- case 'credits':
- Credits::showPage( $article );
- break;
case 'submit':
if ( session_id() == '' ) {
- /* Send a cookie so anons get talk message notifications */
+ // Send a cookie so anons get talk message notifications
wfSetupSession();
}
- /* Continue... */
+ // Continue...
case 'edit':
- if ( wfRunHooks( 'CustomEditor', array( $article, $user ) ) ) {
- $internal = $request->getVal( 'internaledit' );
- $external = $request->getVal( 'externaledit' );
- $section = $request->getVal( 'section' );
- $oldid = $request->getVal( 'oldid' );
- if ( !$this->getVal( 'UseExternalEditor' ) || $action == 'submit' || $internal ||
- $section || $oldid || ( !$user->getOption( 'externaleditor' ) && !$external ) ) {
+ if ( wfRunHooks( 'CustomEditor', array( $article, $this->context->user ) ) ) {
+ $internal = $this->context->request->getVal( 'internaledit' );
+ $external = $this->context->request->getVal( 'externaledit' );
+ $section = $this->context->request->getVal( 'section' );
+ $oldid = $this->context->request->getVal( 'oldid' );
+ if ( !$this->getVal( 'UseExternalEditor' ) || $act == 'submit' || $internal ||
+ $section || $oldid || ( !$this->context->user->getOption( 'externaleditor' ) && !$external ) ) {
$editor = new EditPage( $article );
$editor->submit();
- } elseif ( $this->getVal( 'UseExternalEditor' ) && ( $external || $user->getOption( 'externaleditor' ) ) ) {
- $mode = $request->getVal( 'mode' );
+ } elseif ( $this->getVal( 'UseExternalEditor' ) && ( $external || $this->context->user->getOption( 'externaleditor' ) ) ) {
+ $mode = $this->context->request->getVal( 'mode' );
$extedit = new ExternalEdit( $article, $mode );
$extedit->edit();
}
}
break;
case 'history':
- if ( $request->getFullRequestURL() == $title->getInternalURL( 'action=history' ) ) {
- $output->setSquidMaxage( $this->getVal( 'SquidMaxage' ) );
+ if ( $this->context->request->getFullRequestURL() == $this->context->title->getInternalURL( 'action=history' ) ) {
+ $this->context->output->setSquidMaxage( $this->getVal( 'SquidMaxage' ) );
}
$history = new HistoryPage( $article );
$history->history();
$special->execute( '' );
break;
default:
- if ( wfRunHooks( 'UnknownAction', array( $action, $article ) ) ) {
- $output->showErrorPage( 'nosuchaction', 'nosuchactiontext' );
+ if ( wfRunHooks( 'UnknownAction', array( $act, $article ) ) ) {
+ $this->context->output->showErrorPage( 'nosuchaction', 'nosuchactiontext' );
}
}
wfProfileOut( __METHOD__ );