Add support for Number grouping(commafy) based on CLDR number grouping patterns like...
[lhc/web/wiklou.git] / includes / ImagePage.php
index 91fd25d..9a017da 100644 (file)
@@ -1,31 +1,46 @@
 <?php
-
-if ( !defined( 'MEDIAWIKI' ) )
-       die( 1 );
-
 /**
- * Special handling for image description pages
+ * Class for viewing MediaWiki file description pages
  *
  * @ingroup Media
  */
 class ImagePage extends Article {
 
-       /* private */ var $img;  // Image object
-       /* private */ var $displayImg;
-       /* private */ var $repo;
-       /* private */ var $fileLoaded;
+       /**
+        * @var File
+        */
+       private $displayImg;
+       /**
+        * @var FileRepo
+        */
+       private $repo;
+       private $fileLoaded;
+
        var $mExtraDescription = false;
-       var $dupes;
 
-       function __construct( $title ) {
-               parent::__construct( $title );
-               $this->dupes = null;
-               $this->repo = null;
+       protected function newPage( Title $title ) {
+               // Overload mPage with a file-specific page
+               return new WikiFilePage( $title );
        }
-       
+
+       /**
+        * Constructor from a page id
+        * @param $id Int article ID to load
+        */
+       public static function newFromID( $id ) {
+               $t = Title::newFromID( $id );
+               # @todo FIXME: Doesn't inherit right
+               return $t == null ? null : new self( $t );
+               # return $t == null ? null : new static( $t ); // PHP 5.3
+       }
+
+       /**
+        * @param $file File:
+        * @return void
+        */
        public function setFile( $file ) {
+               $this->mPage->setFile( $file );
                $this->displayImg = $file;
-               $this->img = $file;
                $this->fileLoaded = true;
        }
 
@@ -35,18 +50,19 @@ class ImagePage extends Article {
                }
                $this->fileLoaded = true;
 
-               $this->displayImg = $this->img = false;
-               wfRunHooks( 'ImagePageFindFile', array( $this, &$this->img, &$this->displayImg ) );
-               if ( !$this->img ) {
-                       $this->img = wfFindFile( $this->mTitle );
-                       if ( !$this->img ) {
-                               $this->img = wfLocalFile( $this->mTitle );
+               $this->displayImg = $img = false;
+               wfRunHooks( 'ImagePageFindFile', array( $this, &$img, &$this->displayImg ) );
+               if ( !$img ) { // not set by hook?
+                       $img = wfFindFile( $this->getTitle() );
+                       if ( !$img ) {
+                               $img = wfLocalFile( $this->getTitle() );
                        }
                }
-               if ( !$this->displayImg ) {
-                       $this->displayImg = $this->img;
+               $this->mPage->setFile( $img );
+               if ( !$this->displayImg ) { // not set by hook?
+                       $this->displayImg = $img;
                }
-               $this->repo = $this->img->getRepo();
+               $this->repo = $img->getRepo();
        }
 
        /**
@@ -65,25 +81,25 @@ class ImagePage extends Article {
                $diff = $wgRequest->getVal( 'diff' );
                $diffOnly = $wgRequest->getBool( 'diffonly', $wgUser->getOption( 'diffonly' ) );
 
-               if ( $this->mTitle->getNamespace() != NS_FILE || ( isset( $diff ) && $diffOnly ) ) {
+               if ( $this->getTitle()->getNamespace() != NS_FILE || ( isset( $diff ) && $diffOnly ) ) {
                        return parent::view();
                }
-                       
+
                $this->loadFile();
 
-               if ( $this->mTitle->getNamespace() == NS_FILE && $this->img->getRedirected() ) {
-                       if ( $this->mTitle->getDBkey() == $this->img->getName() || isset( $diff ) ) {
+               if ( $this->getTitle()->getNamespace() == NS_FILE && $this->mPage->getFile()->getRedirected() ) {
+                       if ( $this->getTitle()->getDBkey() == $this->mPage->getFile()->getName() || isset( $diff ) ) {
                                // mTitle is the same as the redirect target so ask Article
                                // to perform the redirect for us.
                                $wgRequest->setVal( 'diffonly', 'true' );
                                return parent::view();
                        } else {
-                               // mTitle is not the same as the redirect target so it is 
+                               // mTitle is not the same as the redirect target so it is
                                // probably the redirect page itself. Fake the redirect symbol
-                               $wgOut->setPageTitle( $this->mTitle->getPrefixedText() );
-                               $wgOut->addHTML( $this->viewRedirect( Title::makeTitle( NS_FILE, $this->img->getName() ),
+                               $wgOut->setPageTitle( $this->getTitle()->getPrefixedText() );
+                               $wgOut->addHTML( $this->viewRedirect( Title::makeTitle( NS_FILE, $this->mPage->getFile()->getName() ),
                                        /* $appendSubtitle */ true, /* $forceKnown */ true ) );
-                               $this->viewUpdates();
+                               $this->mPage->viewUpdates();
                                return;
                        }
                }
@@ -91,34 +107,43 @@ class ImagePage extends Article {
                $this->showRedirectedFromHeader();
 
                if ( $wgShowEXIF && $this->displayImg->exists() ) {
-                       // FIXME: bad interface, see note on MediaHandler::formatMetadata().
+                       // @todo FIXME: Bad interface, see note on MediaHandler::formatMetadata().
                        $formattedMetadata = $this->displayImg->formatMetadata();
                        $showmeta = $formattedMetadata !== false;
                } else {
                        $showmeta = false;
                }
 
-               if ( !$diff && $this->displayImg->exists() )
+               if ( !$diff && $this->displayImg->exists() ) {
                        $wgOut->addHTML( $this->showTOC( $showmeta ) );
+               }
 
-               if ( !$diff )
+               if ( !$diff ) {
                        $this->openShowImage();
+               }
 
                # No need to display noarticletext, we use our own message, output in openShowImage()
-               if ( $this->getID() ) {
+               if ( $this->mPage->getID() ) {
+                       # NS_FILE is in the user language, but this section (the actual wikitext)
+                       # should be in page content language
+                       $pageLang = $this->getTitle()->getPageLanguage();
+                       $wgOut->addHTML( Xml::openElement( 'div', array( 'id' => 'mw-imagepage-content',
+                               'lang' => $pageLang->getCode(), 'dir' => $pageLang->getDir(),
+                               'class' => 'mw-content-'.$pageLang->getDir() ) ) );
                        parent::view();
+                       $wgOut->addHTML( Xml::closeElement( 'div' ) );
                } else {
                        # Just need to set the right headers
                        $wgOut->setArticleFlag( true );
-                       $wgOut->setPageTitle( $this->mTitle->getPrefixedText() );
-                       $this->viewUpdates();
+                       $wgOut->setPageTitle( $this->getTitle()->getPrefixedText() );
+                       $this->mPage->viewUpdates();
                }
 
                # Show shared description, if needed
                if ( $this->mExtraDescription ) {
-                       $fol = wfMsgNoTrans( 'shareddescriptionfollows' );
-                       if ( $fol != '-' && !wfEmptyMsg( 'shareddescriptionfollows', $fol ) ) {
-                               $wgOut->addWikiText( $fol );
+                       $fol = wfMessage( 'shareddescriptionfollows' );
+                       if ( !$fol->isDisabled() ) {
+                               $wgOut->addWikiText( $fol->plain() );
                        }
                        $wgOut->addHTML( '<div id="shared-image-desc">' . $this->mExtraDescription . "</div>\n" );
                }
@@ -131,104 +156,38 @@ class ImagePage extends Article {
                        array( 'id' => 'filelinks' ),
                        wfMsg( 'imagelinks' ) ) . "\n" );
                $this->imageDupes();
-               # TODO! FIXME! For some freaky reason, we can't redirect to foreign images.
+               # @todo FIXME: For some freaky reason, we can't redirect to foreign images.
                # Yet we return metadata about the target. Definitely an issue in the FileRepo
-               $this->imageRedirects();
                $this->imageLinks();
-               
+
                # Allow extensions to add something after the image links
                $html = '';
                wfRunHooks( 'ImagePageAfterImageLinks', array( $this, &$html ) );
-               if ( $html )
+               if ( $html ) {
                        $wgOut->addHTML( $html );
+               }
 
                if ( $showmeta ) {
-                       $expand = htmlspecialchars( Xml::escapeJsString( wfMsg( 'metadata-expand' ) ) );
-                       $collapse = htmlspecialchars( Xml::escapeJsString( wfMsg( 'metadata-collapse' ) ) );
                        $wgOut->addHTML( Xml::element( 'h2', array( 'id' => 'metadata' ), wfMsg( 'metadata' ) ) . "\n" );
                        $wgOut->addWikiText( $this->makeMetadataTable( $formattedMetadata ) );
-                       $wgOut->addModules( array( 'mediawiki.legacy.metadata' ) );
-                       $wgOut->addHTML(
-                               "<script type=\"text/javascript\">attachMetadataToggle('mw_metadata', '$expand', '$collapse');</script>\n" );
-               }
-               
-               $css = $this->repo->getDescriptionStylesheetUrl();
-               if ( $css ) {
-                       $wgOut->addStyle( $css );
-               }
-       }
-       
-       public function getRedirectTarget() {
-               $this->loadFile();
-               if ( $this->img->isLocal() ) {
-                       return parent::getRedirectTarget();
-               }
-               // Foreign image page
-               $from = $this->img->getRedirected();
-               $to = $this->img->getName();
-               if ( $from == $to ) {
-                       return null;
+                       $wgOut->addModules( array( 'mediawiki.action.view.metadata' ) );
                }
-               return $this->mRedirectTarget = Title::makeTitle( NS_FILE, $to );
-       }
-       public function followRedirect() {
-               $this->loadFile();
-               if ( $this->img->isLocal() ) {
-                       return parent::followRedirect();
-               }
-               $from = $this->img->getRedirected();
-               $to = $this->img->getName();
-               if ( $from == $to ) {
-                       return false;
+
+               // Add remote Filepage.css
+               if( !$this->repo->isLocal() ) {
+                       $css = $this->repo->getDescriptionStylesheetUrl();
+                       if ( $css ) {
+                               $wgOut->addStyle( $css );
+                       }
                }
-               return Title::makeTitle( NS_FILE, $to );
-       }
-       public function isRedirect( $text = false ) {
-               $this->loadFile();
-               if ( $this->img->isLocal() )
-                       return parent::isRedirect( $text );
-                       
-               return (bool)$this->img->getRedirected();
-       }
-       
-       public function isLocal() {
-               $this->loadFile();
-               return $this->img->isLocal();
-       }
-       
-       public function getFile() {
-               $this->loadFile();
-               return $this->img;
+               // always show the local local Filepage.css, bug 29277
+               $wgOut->addModuleStyles( 'filepage' );
        }
-       
+
        public function getDisplayedFile() {
                $this->loadFile();
                return $this->displayImg;
        }
-       
-       public function getDuplicates() {
-               $this->loadFile();
-               if ( !is_null( $this->dupes ) ) {
-                       return $this->dupes;
-               }
-               if ( !( $hash = $this->img->getSha1() ) ) {
-                       return $this->dupes = array();
-               }
-               $dupes = RepoGroup::singleton()->findBySha1( $hash );
-               // Remove duplicates with self and non matching file sizes
-               $self = $this->img->getRepoName() . ':' . $this->img->getName();
-               $size = $this->img->getSize();
-               foreach ( $dupes as $index => $file ) {
-                       $key = $file->getRepoName() . ':' . $file->getName();
-                       if ( $key == $self )
-                               unset( $dupes[$index] );
-                       if ( $file->getSize() != $size )
-                               unset( $dupes[$index] );
-               }
-               return $this->dupes = $dupes;
-               
-       }
-       
 
        /**
         * Create the TOC
@@ -238,26 +197,26 @@ class ImagePage extends Article {
         */
        protected function showTOC( $metadata ) {
                $r = array(
-                               '<li><a href="#file">' . wfMsgHtml( 'file-anchor-link' ) . '</a></li>',
-                               '<li><a href="#filehistory">' . wfMsgHtml( 'filehist' ) . '</a></li>',
-                               '<li><a href="#filelinks">' . wfMsgHtml( 'imagelinks' ) . '</a></li>',
+                       '<li><a href="#file">' . wfMsgHtml( 'file-anchor-link' ) . '</a></li>',
+                       '<li><a href="#filehistory">' . wfMsgHtml( 'filehist' ) . '</a></li>',
+                       '<li><a href="#filelinks">' . wfMsgHtml( 'imagelinks' ) . '</a></li>',
                );
                if ( $metadata ) {
                        $r[] = '<li><a href="#metadata">' . wfMsgHtml( 'metadata' ) . '</a></li>';
                }
-       
+
                wfRunHooks( 'ImagePageShowTOC', array( $this, &$r ) );
-               
+
                return '<ul id="filetoc">' . implode( "\n", $r ) . '</ul>';
        }
 
        /**
         * Make a table with metadata to be shown in the output page.
         *
-        * FIXME: bad interface, see note on MediaHandler::formatMetadata().
+        * @todo FIXME: Bad interface, see note on MediaHandler::formatMetadata().
         *
         * @param $metadata Array: the array containing the EXIF data
-        * @return String
+        * @return String The metadata table. This is treated as Wikitext (!)
         */
        protected function makeMetadataTable( $metadata ) {
                $r = "<div class=\"mw-imagepage-section-metadata\">";
@@ -265,7 +224,7 @@ class ImagePage extends Article {
                $r .= "<table id=\"mw_metadata\" class=\"mw_metadata\">\n";
                foreach ( $metadata as $type => $stuff ) {
                        foreach ( $stuff as $v ) {
-                               # FIXME, why is this using escapeId for a class?!
+                               # @todo FIXME: Why is this using escapeId for a class?!
                                $class = Sanitizer::escapeId( $v['id'] );
                                if ( $type == 'collapsed' ) {
                                        $class .= ' collapsable';
@@ -287,20 +246,18 @@ class ImagePage extends Article {
         */
        public function getContent() {
                $this->loadFile();
-               if ( $this->img && !$this->img->isLocal() && 0 == $this->getID() ) {
+               if ( $this->mPage->getFile() && !$this->mPage->getFile()->isLocal() && 0 == $this->getID() ) {
                        return '';
                }
-               return Article::getContent();
+               return parent::getContent();
        }
 
        protected function openShowImage() {
                global $wgOut, $wgUser, $wgImageLimits, $wgRequest,
-                       $wgLang, $wgContLang, $wgEnableUploads;
+                       $wgLang, $wgEnableUploads;
 
                $this->loadFile();
 
-               $full_url  = $this->displayImg->getURL();
-               $linkAttribs = false;
                $sizeSel = intval( $wgUser->getOption( 'imagesize' ) );
                if ( !isset( $wgImageLimits[$sizeSel] ) ) {
                        $sizeSel = User::getDefaultOption( 'imagesize' );
@@ -315,8 +272,7 @@ class ImagePage extends Article {
                $max = $wgImageLimits[$sizeSel];
                $maxWidth = $max[0];
                $maxHeight = $max[1];
-               $sk = $wgUser->getSkin();
-               $dirmark = $wgContLang->getDirMark();
+               $dirmark = $wgLang->getDirMark();
 
                if ( $this->displayImg->exists() ) {
                        # image
@@ -331,10 +287,8 @@ class ImagePage extends Article {
                        $width = $width_orig;
                        $height_orig = $this->displayImg->getHeight( $page );
                        $height = $height_orig;
-                       $mime = $this->displayImg->getMimeType();
-                       $showLink = false;
-                       $linkAttribs = array( 'href' => $full_url );
-                       $longDesc = $this->displayImg->getLongDesc();
+
+                       $longDesc = wfMsg( 'parentheses', $this->displayImg->getLongDesc() );
 
                        wfRunHooks( 'ImageOpenShowImageInlineBefore', array( &$this, &$wgOut ) );
 
@@ -342,7 +296,7 @@ class ImagePage extends Article {
                                # image
 
                                # "Download high res version" link below the image
-                               # $msgsize = wfMsgHtml('file-info-size', $width_orig, $height_orig, $sk->formatSize( $this->displayImg->getSize() ), $mime );
+                               # $msgsize = wfMsgHtml( 'file-info-size', $width_orig, $height_orig, Linker::formatSize( $this->displayImg->getSize() ), $mime );
                                # We'll show a thumbnail of this image
                                if ( $width > $maxWidth || $height > $maxHeight ) {
                                        # Calculate the thumbnail size.
@@ -359,27 +313,31 @@ class ImagePage extends Article {
                                                # because of rounding.
                                        }
                                        $msgbig  = wfMsgHtml( 'show-big-image' );
-                                       $msgsmall = wfMsgExt( 'show-big-image-thumb', 'parseinline',
-                                               $wgLang->formatNum( $width ),
-                                               $wgLang->formatNum( $height )
-                                       );
+                                       $otherSizes = array();
+                                       foreach ( $wgImageLimits as $size ) {
+                                               if ( $size[0] < $width_orig && $size[1] < $height_orig &&
+                                                               $size[0] != $width && $size[1] != $height ) {
+                                                       $otherSizes[] = $this->makeSizeLink( $params, $size[0], $size[1] );
+                                               }
+                                       }
+                                       $msgsmall = wfMessage( 'show-big-image-preview' )->
+                                               rawParams( $this->makeSizeLink( $params, $width, $height ) )->
+                                               parse() . ' ' .
+                                               Html::rawElement( 'span', array( 'class' => 'mw-filepage-other-resolutions' ),
+                                                       wfMessage( 'show-big-image-other' )->
+                                                       rawParams( $wgLang->pipeList( $otherSizes ), count( $otherSizes ) )->parse()
+                                               );
                                } else {
                                        # Image is small enough to show full size on image page
-                                       $msgbig = htmlspecialchars( $this->displayImg->getName() );
-                                       $msgsmall = wfMsgExt( 'file-nohires', array( 'parseinline' ) );
+                                       $msgsmall = wfMessage( 'file-nohires' )->parse();
                                }
 
                                $params['width'] = $width;
+                               $params['height'] = $height;
                                $thumbnail = $this->displayImg->transform( $params );
 
-                               $anchorclose = "<br />";
-                               if ( $this->displayImg->mustRender() ) {
-                                       $showLink = true;
-                               } else {
-                                       $anchorclose .=
-                                               $msgsmall .
-                                               '<br />' . Xml::tags( 'a', $linkAttribs,  $msgbig ) . "$dirmark " . $longDesc;
-                               }
+                               $showLink = true;
+                               $anchorclose = Html::rawElement( 'div', array( 'class' => 'mw-filepage-resolutioninfo' ), $msgsmall );
 
                                $isMulti = $this->displayImg->isMultipage() && $this->displayImg->pageCount() > 1;
                                if ( $isMulti ) {
@@ -401,14 +359,14 @@ class ImagePage extends Article {
 
                                        if ( $page > 1 ) {
                                                $label = $wgOut->parse( wfMsg( 'imgmultipageprev' ), false );
-                                               $link = $sk->link(
-                                                       $this->mTitle,
+                                               $link = Linker::link(
+                                                       $this->getTitle(),
                                                        $label,
                                                        array(),
                                                        array( 'page' => $page - 1 ),
                                                        array( 'known', 'noclasses' )
                                                );
-                                               $thumb1 = $sk->makeThumbLinkObj( $this->mTitle, $this->displayImg, $link, $label, 'none',
+                                               $thumb1 = Linker::makeThumbLinkObj( $this->getTitle(), $this->displayImg, $link, $label, 'none',
                                                        array( 'page' => $page - 1 ) );
                                        } else {
                                                $thumb1 = '';
@@ -416,14 +374,14 @@ class ImagePage extends Article {
 
                                        if ( $page < $count ) {
                                                $label = wfMsg( 'imgmultipagenext' );
-                                               $link = $sk->link(
-                                                       $this->mTitle,
+                                               $link = Linker::link(
+                                                       $this->getTitle(),
                                                        $label,
                                                        array(),
                                                        array( 'page' => $page + 1 ),
                                                        array( 'known', 'noclasses' )
                                                );
-                                               $thumb2 = $sk->makeThumbLinkObj( $this->mTitle, $this->displayImg, $link, $label, 'none',
+                                               $thumb2 = Linker::makeThumbLinkObj( $this->getTitle(), $this->displayImg, $link, $label, 'none',
                                                        array( 'page' => $page + 1 ) );
                                        } else {
                                                $thumb2 = '';
@@ -447,7 +405,7 @@ class ImagePage extends Article {
                                        $wgOut->addHTML(
                                                '</td><td><div class="multipageimagenavbox">' .
                                                Xml::openElement( 'form', $formParams ) .
-                                               Xml::hidden( 'title', $this->getTitle()->getPrefixedDbKey() ) .
+                                               Html::hidden( 'title', $this->getTitle()->getPrefixedDBkey() ) .
                                                wfMsgExt( 'imgmultigoto', array( 'parseinline', 'replaceafter' ), $select ) .
                                                Xml::submitButton( wfMsg( 'imgmultigo' ) ) .
                                                Xml::closeElement( 'form' ) .
@@ -460,33 +418,31 @@ class ImagePage extends Article {
                                        $icon = $this->displayImg->iconThumb();
 
                                        $wgOut->addHTML( '<div class="fullImageLink" id="file">' .
-                                       $icon->toHtml( array( 'file-link' => true ) ) .
-                                       "</div>\n" );
+                                               $icon->toHtml( array( 'file-link' => true ) ) .
+                                               "</div>\n" );
                                }
 
                                $showLink = true;
                        }
 
-
                        if ( $showLink ) {
                                $filename = wfEscapeWikiText( $this->displayImg->getName() );
-                               $medialink = "[[Media:$filename|$filename]]";
+                               $linktext = $filename;
+                               if ( isset( $msgbig ) ) {
+                                       $linktext = wfEscapeWikiText( $msgbig );
+                               }
+                               $medialink = "[[Media:$filename|$linktext]]";
 
                                if ( !$this->displayImg->isSafeFile() ) {
                                        $warning = wfMsgNoTrans( 'mediawarning' );
                                        $wgOut->addWikiText( <<<EOT
-<div class="fullMedia">
-<span class="dangerousLink">{$medialink}</span>$dirmark
-<span class="fileInfo">$longDesc</span>
-</div>
+<div class="fullMedia"><span class="dangerousLink">{$medialink}</span>$dirmark <span class="fileInfo">$longDesc</span></div>
 <div class="mediaWarning">$warning</div>
 EOT
                                                );
                                } else {
                                        $wgOut->addWikiText( <<<EOT
-<div class="fullMedia">
-{$medialink}{$dirmark}
-<span class="fileInfo">$longDesc</span>
+<div class="fullMedia">{$medialink}{$dirmark} <span class="fileInfo">$longDesc</span>
 </div>
 EOT
                                        );
@@ -503,15 +459,44 @@ EOT
                                $uploadTitle = SpecialPage::getTitleFor( 'Upload' );
                                $nofile = array(
                                        'filepage-nofile-link',
-                                       $uploadTitle->getFullUrl( array( 'wpDestFile' => $this->img->getName() ) )
+                                       $uploadTitle->getFullURL( array( 'wpDestFile' => $this->mPage->getFile()->getName() ) )
                                );
-                       }
-                       else
-                       {
+                       } else {
                                $nofile = 'filepage-nofile';
                        }
+                       // Note, if there is an image description page, but
+                       // no image, then this setRobotPolicy is overriden
+                       // by Article::View().
                        $wgOut->setRobotPolicy( 'noindex,nofollow' );
                        $wgOut->wrapWikiMsg( "<div id='mw-imagepage-nofile' class='plainlinks'>\n$1\n</div>", $nofile );
+                       if ( !$this->getID() ) {
+                               // If there is no image, no shared image, and no description page,
+                               // output a 404, to be consistent with articles.
+                               $wgRequest->response()->header( 'HTTP/1.1 404 Not Found' );
+                       }
+               }
+               $wgOut->setFileVersion( $this->displayImg );
+       }
+
+       /**
+        * Creates an thumbnail of specified size and returns an HTML link to it
+        * @param array $params Scaler parameters
+        * @param int $width
+        * @param int $height
+        */
+       private function makeSizeLink( $params, $width, $height ) {
+               $params['width'] = $width;
+               $params['height'] = $height;
+               $thumbnail = $this->displayImg->transform( $params );
+               if ( $thumbnail && !$thumbnail->isError() ) {
+                       return Html::rawElement( 'a', array(
+                               'href' => $thumbnail->getUrl(),
+                               'class' => 'mw-thumbnail-link'
+                               ), wfMessage( 'show-big-image-size' )->numParams(
+                                       $thumbnail->getWidth(), $thumbnail->getHeight()
+                               )->parse() );
+               } else {
+                       return '';
                }
        }
 
@@ -523,11 +508,16 @@ EOT
 
                $this->loadFile();
 
-               $descUrl = $this->img->getDescriptionUrl();
-               $descText = $this->img->getDescriptionText();
+               $descUrl = $this->mPage->getFile()->getDescriptionUrl();
+               $descText = $this->mPage->getFile()->getDescriptionText();
+
+               /* Add canonical to head if there is no local page for this shared file */
+               if( $descUrl && $this->mPage->getID() == 0 ) {
+                       $wgOut->addLink( array( 'rel' => 'canonical', 'href' => $descUrl ) );
+               }
 
                $wrap = "<div class=\"sharedUploadNotice\">\n$1\n</div>\n";
-               $repo = $this->img->getRepo()->getDisplayName();
+               $repo = $this->mPage->getFile()->getRepo()->getDisplayName();
 
                if ( $descUrl && $descText && wfMsgNoTrans( 'sharedupload-desc-here' ) !== '-'  ) {
                        $wgOut->wrapWikiMsg( $wrap, array( 'sharedupload-desc-here', $repo, $descUrl ) );
@@ -545,8 +535,8 @@ EOT
        public function getUploadUrl() {
                $this->loadFile();
                $uploadTitle = SpecialPage::getTitleFor( 'Upload' );
-               return $uploadTitle->getFullUrl( array(
-                       'wpDestFile' => $this->img->getName(),
+               return $uploadTitle->getFullURL( array(
+                       'wpDestFile' => $this->mPage->getFile()->getName(),
                        'wpForReUpload' => 1
                 ) );
        }
@@ -558,26 +548,27 @@ EOT
        protected function uploadLinksBox() {
                global $wgUser, $wgOut, $wgEnableUploads, $wgUseExternalEditor;
 
-               if ( !$wgEnableUploads ) { return; }
+               if ( !$wgEnableUploads ) {
+                       return;
+               }
 
                $this->loadFile();
-               if ( !$this->img->isLocal() )
+               if ( !$this->mPage->getFile()->isLocal() ) {
                        return;
-
-               $sk = $wgUser->getSkin();
+               }
 
                $wgOut->addHTML( "<br /><ul>\n" );
 
                # "Upload a new version of this file" link
-               if ( UploadBase::userCanReUpload( $wgUser, $this->img->name ) ) {
-                       $ulink = $sk->makeExternalLink( $this->getUploadUrl(), wfMsg( 'uploadnewversion-linktext' ) );
+               if ( UploadBase::userCanReUpload( $wgUser, $this->mPage->getFile()->name ) ) {
+                       $ulink = Linker::makeExternalLink( $this->getUploadUrl(), wfMsg( 'uploadnewversion-linktext' ) );
                        $wgOut->addHTML( "<li id=\"mw-imagepage-reupload-link\"><div class=\"plainlinks\">{$ulink}</div></li>\n" );
                }
 
                # External editing link
                if ( $wgUseExternalEditor ) {
-                       $elink = $sk->link(
-                               $this->mTitle,
+                       $elink = Linker::link(
+                               $this->getTitle(),
                                wfMsgHtml( 'edit-externally' ),
                                array(),
                                array(
@@ -587,7 +578,11 @@ EOT
                                ),
                                array( 'known', 'noclasses' )
                        );
-                       $wgOut->addHTML( '<li id="mw-imagepage-edit-external">' . $elink . ' <small>' . wfMsgExt( 'edit-externally-help', array( 'parseinline' ) ) . "</small></li>\n" );
+                       $wgOut->addHTML(
+                               '<li id="mw-imagepage-edit-external">' . $elink . ' <small>' .
+                               wfMsgExt( 'edit-externally-help', array( 'parseinline' ) ) .
+                               "</small></li>\n"
+                       );
                }
 
                $wgOut->addHTML( "</ul>\n" );
@@ -605,70 +600,120 @@ EOT
                $this->loadFile();
                $pager = new ImageHistoryPseudoPager( $this );
                $wgOut->addHTML( $pager->getBody() );
+               $wgOut->preventClickjacking( $pager->getPreventClickjacking() );
 
-               $this->img->resetHistory(); // free db resources
+               $this->mPage->getFile()->resetHistory(); // free db resources
 
                # Exist check because we don't want to show this on pages where an image
                # doesn't exist along with the noimage message, that would suck. -ævar
-               if ( $this->img->exists() ) {
+               if ( $this->mPage->getFile()->exists() ) {
                        $this->uploadLinksBox();
                }
        }
 
-       protected function imageLinks() {
-               global $wgUser, $wgOut, $wgLang;
-
-               $limit = 100;
-
+       protected function queryImageLinks( $target, $limit ) {
                $dbr = wfGetDB( DB_SLAVE );
 
-               $res = $dbr->select(
+               return $dbr->select(
                        array( 'imagelinks', 'page' ),
-                       array( 'page_namespace', 'page_title' ),
-                       array( 'il_to' => $this->mTitle->getDBkey(), 'il_from = page_id' ),
+                       array( 'page_namespace', 'page_title', 'page_is_redirect', 'il_to' ),
+                       array( 'il_to' => $target, 'il_from = page_id' ),
                        __METHOD__,
-                       array( 'LIMIT' => $limit + 1 )
+                       array( 'LIMIT' => $limit + 1, 'ORDER BY' => 'il_from', )
                );
-               $count = $dbr->numRows( $res );
+       }
+
+       protected function imageLinks() {
+               global $wgOut, $wgLang;
+
+               $limit = 100;
+
+               $res = $this->queryImageLinks( $this->getTitle()->getDbKey(), $limit + 1);
+               $rows = array();
+               $redirects = array();
+               foreach ( $res as $row ) {
+                       if ( $row->page_is_redirect ) {
+                               $redirects[$row->page_title] = array();
+                       }
+                       $rows[] = $row;
+               }
+               $count = count( $rows );
+
+               $hasMore = $count > $limit;
+               if ( !$hasMore && count( $redirects ) ) {
+                       $res = $this->queryImageLinks( array_keys( $redirects ),
+                               $limit - count( $rows ) + 1 );
+                       foreach ( $res as $row ) {
+                               $redirects[$row->il_to][] = $row;
+                               $count++;
+                       }
+                       $hasMore = ( $res->numRows() + count( $rows ) ) > $limit;
+               }
+
                if ( $count == 0 ) {
-                       $wgOut->wrapWikiMsg( Html::rawElement( 'div', array ( 'id' => 'mw-imagepage-nolinkstoimage' ), "\n$1\n" ), 'nolinkstoimage' );
+                       $wgOut->wrapWikiMsg(
+                               Html::rawElement( 'div',
+                                       array( 'id' => 'mw-imagepage-nolinkstoimage' ), "\n$1\n" ),
+                               'nolinkstoimage'
+                       );
                        return;
                }
-               
+
                $wgOut->addHTML( "<div id='mw-imagepage-section-linkstoimage'>\n" );
-               if ( $count <= $limit - 1 ) {
+               if ( !$hasMore ) {
                        $wgOut->addWikiMsg( 'linkstoimage', $count );
                } else {
                        // More links than the limit. Add a link to [[Special:Whatlinkshere]]
                        $wgOut->addWikiMsg( 'linkstoimage-more',
                                $wgLang->formatNum( $limit ),
-                               $this->mTitle->getPrefixedDBkey()
+                               $this->getTitle()->getPrefixedDBkey()
                        );
                }
 
-               $wgOut->addHTML( Html::openElement( 'ul', array( 'class' => 'mw-imagepage-linkstoimage' ) ) . "\n" );
-               $sk = $wgUser->getSkin();
+               $wgOut->addHTML(
+                       Html::openElement( 'ul',
+                               array( 'class' => 'mw-imagepage-linkstoimage' ) ) . "\n"
+               );
                $count = 0;
-               $elements = array();
-               while ( $s = $res->fetchObject() ) {
-                       $count++;
-                       if ( $count <= $limit ) {
-                               // We have not yet reached the extra one that tells us there is more to fetch
-                               $elements[] =  $s;
-                       }
-               }
 
                // Sort the list by namespace:title
-               usort ( $elements, array( $this, 'compare' ) );
+               usort( $rows, array( $this, 'compare' ) );
 
                // Create links for every element
-               foreach( $elements as $element ) {    
-                       $link = $sk->linkKnown( Title::makeTitle( $element->page_namespace, $element->page_title ) );
-                       $wgOut->addHTML( Html::rawElement(
+               $currentCount = 0;
+               foreach( $rows as $element ) {
+                       $currentCount++;
+                       if ( $currentCount > $limit ) {
+                               break;
+                       }
+
+                       $link = Linker::linkKnown( Title::makeTitle( $element->page_namespace, $element->page_title ) );
+                       if ( !isset( $redirects[$element->page_title] ) ) {
+                               $liContents = $link;
+                       } else {
+                               $ul = "<ul class='mw-imagepage-redirectstofile'>\n";
+                               foreach ( $redirects[$element->page_title] as $row ) {
+                                       $currentCount++;
+                                       if ( $currentCount > $limit ) {
+                                               break;
+                                       }
+
+                                       $link2 = Linker::linkKnown( Title::makeTitle( $row->page_namespace, $row->page_title ) );
+                                       $ul .= Html::rawElement(
                                                'li',
                                                array( 'id' => 'mw-imagepage-linkstoimage-ns' . $element->page_namespace ),
-                                               $link
-                                       ) . "\n"
+                                               $link2
+                                               ) . "\n";
+                               }
+                               $ul .= '</ul>';
+                               $liContents = wfMessage( 'linkstoimage-redirect' )->rawParams(
+                                       $link, $ul )->parse();
+                       }
+                       $wgOut->addHTML( Html::rawElement(
+                                       'li',
+                                       array( 'id' => 'mw-imagepage-linkstoimage-ns' . $element->page_namespace ),
+                                       $liContents
+                               ) . "\n"
                        );
 
                };
@@ -677,57 +722,31 @@ EOT
 
                // Add a links to [[Special:Whatlinkshere]]
                if ( $count > $limit ) {
-                       $wgOut->addWikiMsg( 'morelinkstoimage', $this->mTitle->getPrefixedDBkey() );
+                       $wgOut->addWikiMsg( 'morelinkstoimage', $this->getTitle()->getPrefixedDBkey() );
                }
                $wgOut->addHTML( Html::closeElement( 'div' ) . "\n" );
        }
-       
-       protected function imageRedirects() {
-               global $wgUser, $wgOut, $wgLang;
-
-               $redirects = $this->getTitle()->getRedirectsHere( NS_FILE );
-               if ( count( $redirects ) == 0 ) return;
-
-               $wgOut->addHTML( "<div id='mw-imagepage-section-redirectstofile'>\n" );
-               $wgOut->addWikiMsg( 'redirectstofile',
-                       $wgLang->formatNum( count( $redirects ) )
-               );
-               $wgOut->addHTML( "<ul class='mw-imagepage-redirectstofile'>\n" );
-
-               $sk = $wgUser->getSkin();
-               foreach ( $redirects as $title ) {
-                       $link = $sk->link(
-                               $title,
-                               null,
-                               array(),
-                               array( 'redirect' => 'no' ),
-                               array( 'known', 'noclasses' )
-                       );
-                       $wgOut->addHTML( "<li>{$link}</li>\n" );
-               }
-               $wgOut->addHTML( "</ul></div>\n" );
-
-       }
 
        protected function imageDupes() {
-               global $wgOut, $wgUser, $wgLang;
+               global $wgOut, $wgLang;
 
                $this->loadFile();
 
-               $dupes = $this->getDuplicates();
-               if ( count( $dupes ) == 0 ) return;
+               $dupes = $this->mPage->getDuplicates();
+               if ( count( $dupes ) == 0 ) {
+                       return;
+               }
 
                $wgOut->addHTML( "<div id='mw-imagepage-section-duplicates'>\n" );
                $wgOut->addWikiMsg( 'duplicatesoffile',
-                       $wgLang->formatNum( count( $dupes ) ), $this->mTitle->getDBkey()
+                       $wgLang->formatNum( count( $dupes ) ), $this->getTitle()->getDBkey()
                );
                $wgOut->addHTML( "<ul class='mw-imagepage-duplicates'>\n" );
 
-               $sk = $wgUser->getSkin();
                foreach ( $dupes as $file ) {
                        $fromSrc = '';
                        if ( $file->isLocal() ) {
-                               $link = $sk->link(
+                               $link = Linker::link(
                                        $file->getTitle(),
                                        null,
                                        array(),
@@ -735,7 +754,7 @@ EOT
                                        array( 'known', 'noclasses' )
                                );
                        } else {
-                               $link = $sk->makeExternalLink( $file->getDescriptionUrl(),
+                               $link = Linker::makeExternalLink( $file->getDescriptionUrl(),
                                        $file->getTitle()->getPrefixedText() );
                                $fromSrc = wfMsg( 'shared-repo-from', $file->getRepo()->getDisplayName() );
                        }
@@ -749,68 +768,40 @@ EOT
         */
        public function delete() {
                global $wgUploadMaintenance;
-               if ( $wgUploadMaintenance && $this->mTitle && $this->mTitle->getNamespace() == NS_FILE ) {
+               if ( $wgUploadMaintenance && $this->getTitle() && $this->getTitle()->getNamespace() == NS_FILE ) {
                        global $wgOut;
                        $wgOut->wrapWikiMsg( "<div class='error'>\n$1\n</div>\n", array( 'filedelete-maintenance' ) );
                        return;
                }
 
                $this->loadFile();
-               if ( !$this->img->exists() || !$this->img->isLocal() || $this->img->getRedirected() ) {
+               if ( !$this->mPage->getFile()->exists() || !$this->mPage->getFile()->isLocal() || $this->mPage->getFile()->getRedirected() ) {
                        // Standard article deletion
                        parent::delete();
                        return;
                }
-               $deleter = new FileDeleteForm( $this->img );
+               $deleter = new FileDeleteForm( $this->mPage->getFile() );
                $deleter->execute();
        }
 
-       /**
-        * Revert the file to an earlier version
-        */
-       public function revert() {
-               $this->loadFile();
-               $reverter = new FileRevertForm( $this->img );
-               $reverter->execute();
-       }
-
-       /**
-        * Override handling of action=purge
-        */
-       public function doPurge() {
-               $this->loadFile();
-               if ( $this->img->exists() ) {
-                       wfDebug( "ImagePage::doPurge purging " . $this->img->getName() . "\n" );
-                       $update = new HTMLCacheUpdate( $this->mTitle, 'imagelinks' );
-                       $update->doUpdate();
-                       $this->img->upgradeRow();
-                       $this->img->purgeCache();
-               } else {
-                       wfDebug( "ImagePage::doPurge no image for " . $this->img->getName() . "; limiting purge to cache only\n" );
-                       // even if the file supposedly doesn't exist, force any cached information
-                       // to be updated (in case the cached information is wrong)
-                       $this->img->purgeCache();
-               }
-               parent::doPurge();
-       }
-
        /**
         * Display an error with a wikitext description
+        *
+        * @param $description String
         */
        function showError( $description ) {
                global $wgOut;
-               $wgOut->setPageTitle( wfMsg( "internalerror" ) );
-               $wgOut->setRobotPolicy( "noindex,nofollow" );
+               $wgOut->setPageTitle( wfMsg( 'internalerror' ) );
+               $wgOut->setRobotPolicy( 'noindex,nofollow' );
                $wgOut->setArticleRelated( false );
                $wgOut->enableClientCache( false );
                $wgOut->addWikiText( $description );
        }
 
-
        /**
         * Callback for usort() to do link sorts by (namespace, title)
         * Function copied from Title::compare()
-        * 
+        *
         * @param $a object page to compare with
         * @param $b object page to compare with
         * @return Integer: result of string comparison, or namespace comparison
@@ -831,11 +822,29 @@ EOT
  */
 class ImageHistoryList {
 
-       protected $imagePage, $img, $skin, $title, $repo, $showThumb;
+       /**
+        * @var Title
+        */
+       protected $title;
+
+       /**
+        * @var File
+        */
+       protected $img;
+
+       /**
+        * @var ImagePage
+        */
+       protected $imagePage;
+
+       protected $repo, $showThumb;
+       protected $preventClickjacking = false;
 
+       /**
+        * @param ImagePage $imagePage
+        */
        public function __construct( $imagePage ) {
-               global $wgUser, $wgShowArchiveThumbnails;
-               $this->skin = $wgUser->getSkin();
+               global $wgShowArchiveThumbnails;
                $this->current = $imagePage->getFile();
                $this->img = $imagePage->getDisplayedFile();
                $this->title = $imagePage->getTitle();
@@ -847,10 +856,6 @@ class ImageHistoryList {
                return $this->imagePage;
        }
 
-       public function getSkin() {
-               return $this->skin;
-       }
-
        public function getFile() {
                return $this->img;
        }
@@ -863,7 +868,7 @@ class ImageHistoryList {
                        . $navLinks . "\n"
                        . Xml::openElement( 'table', array( 'class' => 'wikitable filehistory' ) ) . "\n"
                        . '<tr><td></td>'
-                       . ( $this->current->isLocal() && ( $wgUser->isAllowed( 'delete' ) || $wgUser->isAllowed( 'deletedhistory' ) ) ? '<td></td>' : '' )
+                       . ( $this->current->isLocal() && ( $wgUser->isAllowedAny( 'delete', 'deletedhistory' ) ) ? '<td></td>' : '' )
                        . '<th>' . wfMsgHtml( 'filehist-datetime' ) . '</th>'
                        . ( $this->showThumb ? '<th>' . wfMsgHtml( 'filehist-thumb' ) . '</th>' : '' )
                        . '<th>' . wfMsgHtml( 'filehist-dimensions' ) . '</th>'
@@ -876,8 +881,13 @@ class ImageHistoryList {
                return "</table>\n$navLinks\n</div>\n";
        }
 
+       /**
+        * @param $iscur
+        * @param $file File
+        * @return string
+        */
        public function imageHistoryLine( $iscur, $file ) {
-               global $wgUser, $wgLang;
+               global $wgUser, $wgLang, $wgContLang;
 
                $timestamp = wfTimestamp( TS_MW, $file->getTimestamp() );
                $img = $iscur ? $file->getName() : $file->getArchiveName();
@@ -889,14 +899,15 @@ class ImageHistoryList {
                $row = $selected = '';
 
                // Deletion link
-               if ( $local && ( $wgUser->isAllowed( 'delete' ) || $wgUser->isAllowed( 'deletedhistory' ) ) ) {
+               if ( $local && ( $wgUser->isAllowedAny( 'delete', 'deletedhistory' ) ) ) {
                        $row .= '<td>';
                        # Link to remove from history
                        if ( $wgUser->isAllowed( 'delete' ) ) {
                                $q = array( 'action' => 'delete' );
-                               if ( !$iscur )
+                               if ( !$iscur ) {
                                        $q['oldimage'] = $img;
-                               $row .= $this->skin->link(
+                               }
+                               $row .= Linker::link(
                                        $this->title,
                                        wfMsgHtml( $iscur ? 'filehist-deleteall' : 'filehist-deleteone' ),
                                        array(), $q, array( 'known' )
@@ -910,7 +921,7 @@ class ImageHistoryList {
                                }
                                // If file is top revision or locked from this user, don't link
                                if ( $iscur || !$file->userCan( File::DELETED_RESTRICTED ) ) {
-                                       $del = $this->skin->revDeleteLinkDisabled( $canHide );
+                                       $del = Linker::revDeleteLinkDisabled( $canHide );
                                } else {
                                        list( $ts, $name ) = explode( '!', $img, 2 );
                                        $query = array(
@@ -918,7 +929,7 @@ class ImageHistoryList {
                                                'target' => $this->title->getPrefixedText(),
                                                'ids'    => $ts,
                                        );
-                                       $del = $this->skin->revDeleteLink( $query,
+                                       $del = Linker::revDeleteLink( $query,
                                                $file->isDeleted( File::DELETED_RESTRICTED ), $canHide );
                                }
                                $row .= $del;
@@ -934,7 +945,7 @@ class ImageHistoryList {
                        if ( $file->isDeleted( File::DELETED_FILE ) ) {
                                $row .= wfMsgHtml( 'filehist-revert' );
                        } else {
-                               $row .= $this->skin->link(
+                               $row .= Linker::link(
                                        $this->title,
                                        wfMsgHtml( 'filehist-revert' ),
                                        array(),
@@ -958,19 +969,24 @@ class ImageHistoryList {
                        # Don't link to unviewable files
                        $row .= '<span class="history-deleted">' . $wgLang->timeAndDate( $timestamp, true ) . '</span>';
                } elseif ( $file->isDeleted( File::DELETED_FILE ) ) {
-                       $revdel = SpecialPage::getTitleFor( 'Revisiondelete' );
-                       # Make a link to review the image
-                       $url = $this->skin->link(
-                               $revdel,
-                               $wgLang->timeAndDate( $timestamp, true ),
-                               array(),
-                               array(
-                                       'target' => $this->title->getPrefixedText(),
-                                       'file' => $img,
-                                       'token' => $wgUser->editToken( $img )
-                               ),
-                               array( 'known', 'noclasses' )
-                       );
+                       if ( $local ) {
+                               $this->preventClickjacking();
+                               $revdel = SpecialPage::getTitleFor( 'Revisiondelete' );
+                               # Make a link to review the image
+                               $url = Linker::link(
+                                       $revdel,
+                                       $wgLang->timeAndDate( $timestamp, true ),
+                                       array(),
+                                       array(
+                                               'target' => $this->title->getPrefixedText(),
+                                               'file' => $img,
+                                               'token' => $wgUser->editToken( $img )
+                                       ),
+                                       array( 'known', 'noclasses' )
+                               );
+                       } else {
+                               $url = $wgLang->timeAndDate( $timestamp, true );
+                       }
                        $row .= '<span class="history-deleted">' . $url . '</span>';
                } else {
                        $url = $iscur ? $this->current->getUrl() : $this->current->getArchiveUrl( $img );
@@ -986,38 +1002,42 @@ class ImageHistoryList {
                // Image dimensions + size
                $row .= '<td>';
                $row .= htmlspecialchars( $file->getDimensionsString() );
-               $row .= " <span style='white-space: nowrap;'>(" . $this->skin->formatSize( $file->getSize() ) . ')</span>';
+               $row .= ' <span style="white-space: nowrap;">(' . Linker::formatSize( $file->getSize() ) . ')</span>';
                $row .= '</td>';
 
                // Uploading user
                $row .= '<td>';
-               if ( $local ) {
-                       // Hide deleted usernames
-                       if ( $file->isDeleted( File::DELETED_USER ) ) {
-                               $row .= '<span class="history-deleted">' . wfMsgHtml( 'rev-deleted-user' ) . '</span>';
+               // Hide deleted usernames
+               if ( $file->isDeleted( File::DELETED_USER ) ) {
+                       $row .= '<span class="history-deleted">' . wfMsgHtml( 'rev-deleted-user' ) . '</span>';
+               } else {
+                       if ( $local ) {
+                               $row .= Linker::userLink( $user, $usertext ) . ' <span style="white-space: nowrap;">' .
+                               Linker::userToolLinks( $user, $usertext ) . '</span>';
                        } else {
-                               $row .= $this->skin->userLink( $user, $usertext ) . " <span style='white-space: nowrap;'>" .
-                                       $this->skin->userToolLinks( $user, $usertext ) . "</span>";
+                               $row .= htmlspecialchars( $usertext );
                        }
-               } else {
-                       $row .= htmlspecialchars( $usertext );
                }
-               $row .= '</td><td>';
+               $row .= '</td>';
 
                // Don't show deleted descriptions
                if ( $file->isDeleted( File::DELETED_COMMENT ) ) {
-                       $row .= '<span class="history-deleted">' . wfMsgHtml( 'rev-deleted-comment' ) . '</span>';
+                       $row .= '<td><span class="history-deleted">' . wfMsgHtml( 'rev-deleted-comment' ) . '</span></td>';
                } else {
-                       $row .= $this->skin->commentBlock( $description, $this->title );
+                       $row .= '<td dir="' . $wgContLang->getDir() . '">' . Linker::commentBlock( $description, $this->title ) . '</td>';
                }
-               $row .= '</td>';
 
+               $rowClass = null;
                wfRunHooks( 'ImagePageFileHistoryLine', array( $this, $file, &$row, &$rowClass ) );
-               $classAttr = $rowClass ? " class='$rowClass'" : "";
+               $classAttr = $rowClass ? " class='$rowClass'" : '';
 
                return "<tr{$classAttr}>{$row}</tr>\n";
        }
 
+       /**
+        * @param $file File
+        * @return string
+        */
        protected function getThumbForLine( $file ) {
                global $wgLang;
 
@@ -1036,17 +1056,42 @@ class ImageHistoryList {
                                        $wgLang->time( $timestamp, true ) ),
                                'file-link' => true,
                        );
-                       
-                       if ( !$thumbnail ) return wfMsgHtml( 'filehist-nothumb' );
+
+                       if ( !$thumbnail ) {
+                               return wfMsgHtml( 'filehist-nothumb' );
+                       }
 
                        return $thumbnail->toHtml( $options );
                } else {
                        return wfMsgHtml( 'filehist-nothumb' );
                }
        }
+
+       protected function preventClickjacking( $enable = true ) {
+               $this->preventClickjacking = $enable;
+       }
+
+       public function getPreventClickjacking() {
+               return $this->preventClickjacking;
+       }
 }
 
 class ImageHistoryPseudoPager extends ReverseChronologicalPager {
+       protected $preventClickjacking = false;
+
+       /**
+        * @var File
+        */
+       protected $mImg;
+
+       /**
+        * @var Title
+        */
+       protected $mTitle;
+
+       /**
+        * @param ImagePage $imagePage
+        */
        function __construct( $imagePage ) {
                parent::__construct();
                $this->mImagePage = $imagePage;
@@ -1056,7 +1101,7 @@ class ImageHistoryPseudoPager extends ReverseChronologicalPager {
                $this->mHist = array();
                $this->mRange = array( 0, 0 ); // display range
        }
-       
+
        function getTitle() {
                return $this->mTitle;
        }
@@ -1072,7 +1117,7 @@ class ImageHistoryPseudoPager extends ReverseChronologicalPager {
        function formatRow( $row ) {
                return '';
        }
-       
+
        function getBody() {
                $s = '';
                $this->doQuery();
@@ -1087,12 +1132,18 @@ class ImageHistoryPseudoPager extends ReverseChronologicalPager {
                                $s .= $list->imageHistoryLine( !$file->isOld(), $file );
                        }
                        $s .= $list->endImageHistoryList( $navLink );
+
+                       if ( $list->getPreventClickjacking() ) {
+                               $this->preventClickjacking();
+                       }
                }
                return $s;
        }
 
        function doQuery() {
-               if ( $this->mQueryDone ) return;
+               if ( $this->mQueryDone ) {
+                       return;
+               }
                $this->mImg = $this->mImagePage->getFile(); // ensure loading
                if ( !$this->mImg->exists() ) {
                        return;
@@ -1169,4 +1220,13 @@ class ImageHistoryPseudoPager extends ReverseChronologicalPager {
                }
                $this->mQueryDone = true;
        }
+
+       protected function preventClickjacking( $enable = true ) {
+               $this->preventClickjacking = $enable;
+       }
+
+       public function getPreventClickjacking() {
+               return $this->preventClickjacking;
+       }
+
 }