/**
* Initialization of ... everything
* Performs the request too
+ * FIXME: why is this crap called "initialize" when it performs everything?
*
* @param $title Title ($wgTitle)
* @param $article Article
*/
function initialize( &$title, &$article, &$output, &$user, $request ) {
wfProfileIn( __METHOD__ );
- $this->preliminaryChecks( $title, $output, $request );
+ if( !$this->preliminaryChecks( $title, $output, $request ) ) {
+ wfProfileOut( __METHOD__ );
+ return;
+ }
if( !$this->initializeSpecialCases( $title, $output, $request ) ) {
$new_article = $this->initializeArticle( $title, $request );
if( is_object( $new_article ) ) {
}
}
-
/**
* Checks some initial queries
* Note that $title here is *not* a Title object, but a string!
// about the possible different language variants
if( count( $wgContLang->getVariants() ) > 1 && !is_null( $ret ) && $ret->getArticleID() == 0 )
$wgContLang->findVariantLink( $title, $ret );
-
}
- if( ( $oldid = $wgRequest->getInt( 'oldid' ) )
- && ( is_null( $ret ) || $ret->getNamespace() != NS_SPECIAL ) ) {
- // Allow oldid to override a changed or missing title.
- $rev = Revision::newFromId( $oldid );
- if( $rev ) {
- $ret = $rev->getTitle();
+ # 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 = $wgRequest->getInt( 'oldid' );
+ $oldid = $oldid ? $oldid : $wgRequest->getInt( 'diff' );
+ // Allow oldid to override a changed or missing title
+ if( $oldid ) {
+ $rev = Revision::newFromId( $oldid );
+ $ret = $rev ? $rev->getTitle() : $ret;
}
}
return $ret;
if( !is_null( $title ) && !$title->userCanRead() ) {
$output->loginToUse();
$output->output();
- exit;
+ $output->disable();
+ return false;
}
+ return true;
}
/**
* - redirect loop
* - special pages
*
+ * FIXME: why is this crap called "initialize" when it performs everything?
+ *
* @param $title Title
* @param $output OutputPage
* @param $request WebRequest
} else if( NS_SPECIAL == $title->getNamespace() ) {
/* actions that need to be made when we have a special pages */
SpecialPage::executePath( $title );
- } else if( NS_MEDIA == $title->getNamespace() ) {
- global $wgOut;
- $fileTitle = Title::makeTitle( NS_FILE, $title->getDBKey() );
- $wgOut->redirect( $fileTitle->getFullUrl() );
} else {
- /* Try low-level file cache hit */
- if( $title->getNamespace() != NS_MEDIAWIKI && HTMLFileCache::useFileCache() ) {
- $cache = new HTMLFileCache( $title );
- if( $cache->isFileCacheGood( /* Assume up to date */ ) ) {
- global $wgOut;
- /* Check incoming headers to see if client has this cached */
- if( !$wgOut->checkLastModified( $cache->fileCacheTime() ) ) {
- wfDebug( "MediaWiki::initializeSpecialCases(): about to load file cache\n" );
- $cache->loadFromFileCache();
- # Tell $wgOut that output is taken care of
- $wgOut->disable();
- # Do any stats increment/watchlist stuff
- $article = self::articleFromTitle( $title );
- $article->viewUpdates();
- }
- wfProfileOut( __METHOD__ );
- $this->restInPeace();
- exit;
- }
- }
/* No match to special cases */
wfProfileOut( __METHOD__ );
return false;
function initializeArticle( &$title, $request ) {
wfProfileIn( __METHOD__ );
- $action = $this->getVal( 'action' );
+ $action = $this->getVal( 'action', 'view' );
$article = self::articleFromTitle( $title );
-
+ # NS_MEDIAWIKI has no redirects.
+ # It is also used for CSS/JS, so performance matters here...
+ if( $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;
return $target;
}
}
-
- if( is_object( $target ) ) {
+ if( is_object($target) ) {
// Rewrite environment to redirected article
$rarticle = self::articleFromTitle( $target );
$rarticle->loadPageData( $rarticle->pageDataFromTitle( $dbr, $target ) );
$article->view();
break;
case 'raw': // includes JS/CSS
+ wfProfileIn( __METHOD__.'-raw' );
$raw = new RawPage( $article );
$raw->view();
+ wfProfileOut( __METHOD__.'-raw' );
break;
case 'watch':
case 'unwatch':