X-Git-Url: https://git.cyclocoop.org/?a=blobdiff_plain;f=includes%2FImagePage.php;h=91fd25db1368c36044487f59d26931d829a44bf2;hb=b29514b0b3f71e4794e8d9810b20911f3ff420db;hp=55e63baffe8912536f4e7e8bd98d48c1b8a94b41;hpb=61fe8acfc56b7110e002c9d4f5456fffc29d5241;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/ImagePage.php b/includes/ImagePage.php index 55e63baffe..91fd25db13 100644 --- a/includes/ImagePage.php +++ b/includes/ImagePage.php @@ -1,6 +1,6 @@ dupes = null; $this->repo = null; } + + public function setFile( $file ) { + $this->displayImg = $file; + $this->img = $file; + $this->fileLoaded = true; + } protected function loadFile() { if ( $this->fileLoaded ) { @@ -47,37 +53,42 @@ class ImagePage extends Article { * Handler for action=render * Include body text only; none of the image extras */ - function render() { + public function render() { global $wgOut; $wgOut->setArticleBodyOnly( true ); parent::view(); } - function view() { + public function view() { global $wgOut, $wgShowEXIF, $wgRequest, $wgUser; + + $diff = $wgRequest->getVal( 'diff' ); + $diffOnly = $wgRequest->getBool( 'diffonly', $wgUser->getOption( 'diffonly' ) ); + + if ( $this->mTitle->getNamespace() != NS_FILE || ( isset( $diff ) && $diffOnly ) ) { + return parent::view(); + } + $this->loadFile(); - if ( $this->mTitle->getNamespace() == NS_IMAGE && $this->img->getRedirected() ) { - if ( $this->mTitle->getDBkey() == $this->img->getName() ) { + if ( $this->mTitle->getNamespace() == NS_FILE && $this->img->getRedirected() ) { + if ( $this->mTitle->getDBkey() == $this->img->getName() || isset( $diff ) ) { // mTitle is the same as the redirect target so ask Article // to perform the redirect for us. - return Article::view(); + $wgRequest->setVal( 'diffonly', 'true' ); + return parent::view(); } else { // 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() ); - $this->viewRedirect( Title::makeTitle( NS_IMAGE, $this->img->getName() ), - /* $overwriteSubtitle */ true, /* $forceKnown */ true ); + $wgOut->addHTML( $this->viewRedirect( Title::makeTitle( NS_FILE, $this->img->getName() ), + /* $appendSubtitle */ true, /* $forceKnown */ true ) ); $this->viewUpdates(); return; } } - $diff = $wgRequest->getVal( 'diff' ); - $diffOnly = $wgRequest->getBool( 'diffonly', $wgUser->getOption( 'diffonly' ) ); - - if ( $this->mTitle->getNamespace() != NS_IMAGE || ( isset( $diff ) && $diffOnly ) ) - return Article::view(); + $this->showRedirectedFromHeader(); if ( $wgShowEXIF && $this->displayImg->exists() ) { // FIXME: bad interface, see note on MediaHandler::formatMetadata(). @@ -87,18 +98,18 @@ class ImagePage extends Article { $showmeta = false; } - if ( $this->displayImg->exists() ) - $wgOut->addHTML( $this->showTOC($showmeta) ); + if ( !$diff && $this->displayImg->exists() ) + $wgOut->addHTML( $this->showTOC( $showmeta ) ); - $this->openShowImage(); + if ( !$diff ) + $this->openShowImage(); # No need to display noarticletext, we use our own message, output in openShowImage() if ( $this->getID() ) { - Article::view(); + parent::view(); } else { # Just need to set the right headers $wgOut->setArticleFlag( true ); - $wgOut->setRobotpolicy( 'noindex,nofollow' ); $wgOut->setPageTitle( $this->mTitle->getPrefixedText() ); $this->viewUpdates(); } @@ -106,39 +117,45 @@ class ImagePage extends Article { # Show shared description, if needed if ( $this->mExtraDescription ) { $fol = wfMsgNoTrans( 'shareddescriptionfollows' ); - if( $fol != '-' && !wfEmptyMsg( 'shareddescriptionfollows', $fol ) ) { + if ( $fol != '-' && !wfEmptyMsg( 'shareddescriptionfollows', $fol ) ) { $wgOut->addWikiText( $fol ); } - $wgOut->addHTML( '
' . $this->mExtraDescription . '
' ); - } else { - $this->checkSharedConflict(); + $wgOut->addHTML( '
' . $this->mExtraDescription . "
\n" ); } $this->closeShowImage(); $this->imageHistory(); // TODO: Cleanup the following - - $wgOut->addHTML( Xml::element( 'h2', - array( 'id' => 'filelinks' ), + + $wgOut->addHTML( Xml::element( 'h2', + array( 'id' => 'filelinks' ), wfMsg( 'imagelinks' ) ) . "\n" ); $this->imageDupes(); - // TODO: We may want to find local images redirecting to a foreign - // file: "The following local files redirect to this file" - if ( $this->img->isLocal() ) { - $this->imageRedirects(); - } + # 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 ) + $wgOut->addHTML( $html ); if ( $showmeta ) { - global $wgStylePath, $wgStyleVersion; - $expand = htmlspecialchars( wfEscapeJsString( wfMsg( 'metadata-expand' ) ) ); - $collapse = htmlspecialchars( wfEscapeJsString( wfMsg( 'metadata-collapse' ) ) ); - $wgOut->addHTML( Xml::element( 'h2', array( 'id' => 'metadata' ), wfMsg( 'metadata' ) ). "\n" ); + $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->addScriptFile( 'metadata.js' ); + $wgOut->addModules( array( 'mediawiki.legacy.metadata' ) ); $wgOut->addHTML( "\n" ); } + + $css = $this->repo->getDescriptionStylesheetUrl(); + if ( $css ) { + $wgOut->addStyle( $css ); + } } public function getRedirectTarget() { @@ -150,9 +167,9 @@ class ImagePage extends Article { $from = $this->img->getRedirected(); $to = $this->img->getName(); if ( $from == $to ) { - return null; + return null; } - return $this->mRedirectTarget = Title::makeTitle( NS_IMAGE, $to ); + return $this->mRedirectTarget = Title::makeTitle( NS_FILE, $to ); } public function followRedirect() { $this->loadFile(); @@ -162,9 +179,9 @@ class ImagePage extends Article { $from = $this->img->getRedirected(); $to = $this->img->getName(); if ( $from == $to ) { - return false; + return false; } - return Title::makeTitle( NS_IMAGE, $to ); + return Title::makeTitle( NS_FILE, $to ); } public function isRedirect( $text = false ) { $this->loadFile(); @@ -191,7 +208,7 @@ class ImagePage extends Article { public function getDuplicates() { $this->loadFile(); - if ( !is_null($this->dupes) ) { + if ( !is_null( $this->dupes ) ) { return $this->dupes; } if ( !( $hash = $this->img->getSha1() ) ) { @@ -199,10 +216,10 @@ class ImagePage extends Article { } $dupes = RepoGroup::singleton()->findBySha1( $hash ); // Remove duplicates with self and non matching file sizes - $self = $this->img->getRepoName().':'.$this->img->getName(); + $self = $this->img->getRepoName() . ':' . $this->img->getName(); $size = $this->img->getSize(); foreach ( $dupes as $index => $file ) { - $key = $file->getRepoName().':'.$file->getName(); + $key = $file->getRepoName() . ':' . $file->getName(); if ( $key == $self ) unset( $dupes[$index] ); if ( $file->getSize() != $size ) @@ -216,20 +233,22 @@ class ImagePage extends Article { /** * Create the TOC * - * @access private - * - * @param bool $metadata Whether or not to show the metadata link - * @return string + * @param $metadata Boolean: whether or not to show the metadata link + * @return String */ - function showTOC( $metadata ) { - global $wgLang; - $r = ''; - return $r; + protected function showTOC( $metadata ) { + $r = array( + '
  • ' . wfMsgHtml( 'file-anchor-link' ) . '
  • ', + '
  • ' . wfMsgHtml( 'filehist' ) . '
  • ', + '
  • ' . wfMsgHtml( 'imagelinks' ) . '
  • ', + ); + if ( $metadata ) { + $r[] = '
  • ' . wfMsgHtml( 'metadata' ) . '
  • '; + } + + wfRunHooks( 'ImagePageShowTOC', array( $this, &$r ) ); + + return ''; } /** @@ -237,26 +256,26 @@ class ImagePage extends Article { * * FIXME: bad interface, see note on MediaHandler::formatMetadata(). * - * @access private - * - * @param array $exif The array containing the EXIF data - * @return string + * @param $metadata Array: the array containing the EXIF data + * @return String */ - function makeMetadataTable( $metadata ) { - $r = wfMsg( 'metadata-help' ) . "\n\n"; - $r .= "{| id=mw_metadata class=mw_metadata\n"; + protected function makeMetadataTable( $metadata ) { + $r = "
    "; + $r .= wfMsgNoTrans( 'metadata-help' ); + $r .= "\n"; foreach ( $metadata as $type => $stuff ) { foreach ( $stuff as $v ) { + # FIXME, why is this using escapeId for a class?! $class = Sanitizer::escapeId( $v['id'] ); - if( $type == 'collapsed' ) { + if ( $type == 'collapsed' ) { $class .= ' collapsable'; } - $r .= "|- class=\"$class\"\n"; - $r .= "!| {$v['name']}\n"; - $r .= "|| {$v['value']}\n"; + $r .= "\n"; + $r .= "\n"; + $r .= "\n"; } } - $r .= '|}'; + $r .= "
    {$v['name']}{$v['value']}
    \n
    \n"; return $r; } @@ -266,28 +285,29 @@ class ImagePage extends Article { * Omit noarticletext if sharedupload; text will be fetched from the * shared upload server if possible. */ - function getContent() { + public function getContent() { $this->loadFile(); - if( $this->img && !$this->img->isLocal() && 0 == $this->getID() ) { + if ( $this->img && !$this->img->isLocal() && 0 == $this->getID() ) { return ''; } return Article::getContent(); } - function openShowImage() { - global $wgOut, $wgUser, $wgImageLimits, $wgRequest, $wgLang, $wgContLang; + protected function openShowImage() { + global $wgOut, $wgUser, $wgImageLimits, $wgRequest, + $wgLang, $wgContLang, $wgEnableUploads; $this->loadFile(); $full_url = $this->displayImg->getURL(); $linkAttribs = false; - $sizeSel = intval( $wgUser->getOption( 'imagesize') ); - if( !isset( $wgImageLimits[$sizeSel] ) ) { + $sizeSel = intval( $wgUser->getOption( 'imagesize' ) ); + if ( !isset( $wgImageLimits[$sizeSel] ) ) { $sizeSel = User::getDefaultOption( 'imagesize' ); // The user offset might still be incorrect, specially if // $wgImageLimits got changed (see bug #8858). - if( !isset( $wgImageLimits[$sizeSel] ) ) { + if ( !isset( $wgImageLimits[$sizeSel] ) ) { // Default to the first offset in $wgImageLimits $sizeSel = 0; } @@ -307,40 +327,42 @@ class ImagePage extends Article { } else { $params = array( 'page' => $page ); } - $width_orig = $this->displayImg->getWidth(); + $width_orig = $this->displayImg->getWidth( $page ); $width = $width_orig; - $height_orig = $this->displayImg->getHeight(); + $height_orig = $this->displayImg->getHeight( $page ); $height = $height_orig; $mime = $this->displayImg->getMimeType(); $showLink = false; $linkAttribs = array( 'href' => $full_url ); $longDesc = $this->displayImg->getLongDesc(); - wfRunHooks( 'ImageOpenShowImageInlineBefore', array( &$this , &$wgOut ) ) ; + wfRunHooks( 'ImageOpenShowImageInlineBefore', array( &$this, &$wgOut ) ); if ( $this->displayImg->allowInlineDisplay() ) { # 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, $sk->formatSize( $this->displayImg->getSize() ), $mime ); # We'll show a thumbnail of this image if ( $width > $maxWidth || $height > $maxHeight ) { # Calculate the thumbnail size. # First case, the limiting factor is the width, not the height. if ( $width / $height >= $maxWidth / $maxHeight ) { - $height = round( $height * $maxWidth / $width); + $height = round( $height * $maxWidth / $width ); $width = $maxWidth; # Note that $height <= $maxHeight now. } else { - $newwidth = floor( $width * $maxHeight / $height); + $newwidth = floor( $width * $maxHeight / $height ); $height = round( $height * $newwidth / $width ); $width = $newwidth; # Note that $height <= $maxHeight now, but might not be identical # because of rounding. } $msgbig = wfMsgHtml( 'show-big-image' ); - $msgsmall = wfMsgExt( 'show-big-image-thumb', - array( 'parseinline' ), $wgLang->formatNum( $width ), $wgLang->formatNum( $height ) ); + $msgsmall = wfMsgExt( 'show-big-image-thumb', 'parseinline', + $wgLang->formatNum( $width ), + $wgLang->formatNum( $height ) + ); } else { # Image is small enough to show full size on image page $msgbig = htmlspecialchars( $this->displayImg->getName() ); @@ -351,7 +373,7 @@ class ImagePage extends Article { $thumbnail = $this->displayImg->transform( $params ); $anchorclose = "
    "; - if( $this->displayImg->mustRender() ) { + if ( $this->displayImg->mustRender() ) { $showLink = true; } else { $anchorclose .= @@ -359,7 +381,8 @@ class ImagePage extends Article { '
    ' . Xml::tags( 'a', $linkAttribs, $msgbig ) . "$dirmark " . $longDesc; } - if ( $this->displayImg->isMultipage() ) { + $isMulti = $this->displayImg->isMultipage() && $this->displayImg->pageCount() > 1; + if ( $isMulti ) { $wgOut->addHTML( '' - . ( $this->current->isLocal() && ($wgUser->isAllowed('delete') || $wgUser->isAllowed('deleterevision') ) ? '' : '' ) + . ( $this->current->isLocal() && ( $wgUser->isAllowed( 'delete' ) || $wgUser->isAllowed( 'deletedhistory' ) ) ? '' : '' ) . '' + . ( $this->showThumb ? '' : '' ) . '' - . '' - . '' + . '' + . '' . "\n"; } - public function endImageHistoryList() { - return "
    ' ); } @@ -370,15 +393,21 @@ class ImagePage extends Article { ); $wgOut->addHTML( '' ); + $anchorclose . "\n" ); } - if ( $this->displayImg->isMultipage() ) { + if ( $isMulti ) { $count = $this->displayImg->pageCount(); if ( $page > 1 ) { $label = $wgOut->parse( wfMsg( 'imgmultipageprev' ), false ); - $link = $sk->makeKnownLinkObj( $this->mTitle, $label, 'page='. ($page-1) ); + $link = $sk->link( + $this->mTitle, + $label, + array(), + array( 'page' => $page - 1 ), + array( 'known', 'noclasses' ) + ); $thumb1 = $sk->makeThumbLinkObj( $this->mTitle, $this->displayImg, $link, $label, 'none', array( 'page' => $page - 1 ) ); } else { @@ -387,7 +416,13 @@ class ImagePage extends Article { if ( $page < $count ) { $label = wfMsg( 'imgmultipagenext' ); - $link = $sk->makeKnownLinkObj( $this->mTitle, $label, 'page='. ($page+1) ); + $link = $sk->link( + $this->mTitle, + $label, + array(), + array( 'page' => $page + 1 ), + array( 'known', 'noclasses' ) + ); $thumb2 = $sk->makeThumbLinkObj( $this->mTitle, $this->displayImg, $link, $label, 'none', array( 'page' => $page + 1 ) ); } else { @@ -402,9 +437,8 @@ class ImagePage extends Article { 'onchange' => 'document.pageselector.submit();', ); - $option = array(); - for ( $i=1; $i <= $count; $i++ ) { - $options[] = Xml::option( $wgLang->formatNum($i), $i, $i == $page ); + for ( $i = 1; $i <= $count; $i++ ) { + $options[] = Xml::option( $wgLang->formatNum( $i ), $i, $i == $page ); } $select = Xml::tags( 'select', array( 'id' => 'pageselector', 'name' => 'page' ), @@ -421,216 +455,169 @@ class ImagePage extends Article { ); } } else { - #if direct link is allowed but it's not a renderable image, show an icon. + # if direct link is allowed but it's not a renderable image, show an icon. if ( $this->displayImg->isSafeFile() ) { - $icon= $this->displayImg->iconThumb(); + $icon = $this->displayImg->iconThumb(); $wgOut->addHTML( '' ); + $icon->toHtml( array( 'file-link' => true ) ) . + "\n" ); } $showLink = true; } - if ($showLink) { + if ( $showLink ) { $filename = wfEscapeWikiText( $this->displayImg->getName() ); + $medialink = "[[Media:$filename|$filename]]"; if ( !$this->displayImg->isSafeFile() ) { $warning = wfMsgNoTrans( 'mediawarning' ); $wgOut->addWikiText( << -[[Media:$filename|$filename]]$dirmark - $longDesc +{$medialink}$dirmark +$longDesc -
    $warning
    EOT ); } else { $wgOut->addWikiText( << -[[Media:$filename|$filename]]$dirmark $longDesc +{$medialink}{$dirmark} +$longDesc EOT - ); + ); } } - if( !$this->displayImg->isLocal() ) { + if ( !$this->displayImg->isLocal() ) { $this->printSharedImageText(); } } else { # Image does not exist - - $title = SpecialPage::getTitleFor( 'Upload' ); - $link = $sk->makeKnownLinkObj($title, wfMsgHtml('noimage-linktext'), - 'wpDestFile=' . urlencode( $this->displayImg->getName() ) ); - $wgOut->addHTML( wfMsgWikiHtml( 'noimage', $link ) ); + if ( $wgEnableUploads && $wgUser->isAllowed( 'upload' ) ) { + // Only show an upload link if the user can upload + $uploadTitle = SpecialPage::getTitleFor( 'Upload' ); + $nofile = array( + 'filepage-nofile-link', + $uploadTitle->getFullUrl( array( 'wpDestFile' => $this->img->getName() ) ) + ); + } + else + { + $nofile = 'filepage-nofile'; + } + $wgOut->setRobotPolicy( 'noindex,nofollow' ); + $wgOut->wrapWikiMsg( "", $nofile ); } } /** * Show a notice that the file is from a shared repository */ - function printSharedImageText() { - global $wgOut, $wgUser; + protected function printSharedImageText() { + global $wgOut; $this->loadFile(); $descUrl = $this->img->getDescriptionUrl(); $descText = $this->img->getDescriptionText(); - $s = "
    " . wfMsgWikiHtml( 'sharedupload' ); - if ( $descUrl ) { - $sk = $wgUser->getSkin(); - $link = $sk->makeExternalLink( $descUrl, wfMsg( 'shareduploadwiki-linktext' ) ); - $msg = ( $descText ) ? 'shareduploadwiki-desc' : 'shareduploadwiki'; - $msg = wfMsgExt( $msg, array( 'parseinline', 'replaceafter' ), $link ); - if ( $msg != '-' ) { - # Show message only if not voided by local sysops - $s .= $msg; - } - } - $s .= "
    "; - $wgOut->addHTML( $s ); - - if ( $descText ) { - $this->mExtraDescription = $descText; - } - } - - /* - * Check for files with the same name on the foreign repos. - */ - function checkSharedConflict() { - global $wgOut, $wgUser; - - $repoGroup = RepoGroup::singleton(); - if( !$repoGroup->hasForeignRepos() ) { - return; - } - - $this->loadFile(); - if( !$this->img->isLocal() ) { - return; - } - $this->dupFile = null; - $repoGroup->forEachForeignRepo( array( $this, 'checkSharedConflictCallback' ) ); - - if( !$this->dupFile ) - return; - $dupfile = $this->dupFile; - $same = ( - ($this->img->getSha1() == $dupfile->getSha1()) && - ($this->img->getSize() == $dupfile->getSize()) - ); + $wrap = "
    \n$1\n
    \n"; + $repo = $this->img->getRepo()->getDisplayName(); - $sk = $wgUser->getSkin(); - $descUrl = $dupfile->getDescriptionUrl(); - if( $same ) { - $link = $sk->makeExternalLink( $descUrl, wfMsg( 'shareduploadduplicate-linktext' ) ); - $wgOut->addHTML( '
    ' . wfMsgWikiHtml( 'shareduploadduplicate', $link ) . '
    ' ); + if ( $descUrl && $descText && wfMsgNoTrans( 'sharedupload-desc-here' ) !== '-' ) { + $wgOut->wrapWikiMsg( $wrap, array( 'sharedupload-desc-here', $repo, $descUrl ) ); + } elseif ( $descUrl && wfMsgNoTrans( 'sharedupload-desc-there' ) !== '-' ) { + $wgOut->wrapWikiMsg( $wrap, array( 'sharedupload-desc-there', $repo, $descUrl ) ); } else { - $link = $sk->makeExternalLink( $descUrl, wfMsg( 'shareduploadconflict-linktext' ) ); - $wgOut->addHTML( '
    ' . wfMsgWikiHtml( 'shareduploadconflict', $link ) . '
    ' ); + $wgOut->wrapWikiMsg( $wrap, array( 'sharedupload', $repo ), ''/*BACKCOMPAT*/ ); } - } - function checkSharedConflictCallback( $repo ) { - $this->loadFile(); - $dupfile = $repo->newFile( $this->img->getTitle() ); - if( $dupfile && $dupfile->exists() ) { - $this->dupFile = $dupfile; - return $dupfile->exists(); + if ( $descText ) { + $this->mExtraDescription = $descText; } - return false; } - function getUploadUrl() { + public function getUploadUrl() { $this->loadFile(); $uploadTitle = SpecialPage::getTitleFor( 'Upload' ); - return $uploadTitle->getFullUrl( 'wpDestFile=' . urlencode( $this->img->getName() ) ); + return $uploadTitle->getFullUrl( array( + 'wpDestFile' => $this->img->getName(), + 'wpForReUpload' => 1 + ) ); } /** * Print out the various links at the bottom of the image page, e.g. reupload, * external editing (and instructions link) etc. */ - function uploadLinksBox() { - global $wgUser, $wgOut; + protected function uploadLinksBox() { + global $wgUser, $wgOut, $wgEnableUploads, $wgUseExternalEditor; + + if ( !$wgEnableUploads ) { return; } $this->loadFile(); - if( !$this->img->isLocal() ) + if ( !$this->img->isLocal() ) return; $sk = $wgUser->getSkin(); - $wgOut->addHtml( '
      ' ); + $wgOut->addHTML( "
        \n" ); # "Upload a new version of this file" link - if( UploadForm::userCanReUpload($wgUser,$this->img->name) ) { + if ( UploadBase::userCanReUpload( $wgUser, $this->img->name ) ) { $ulink = $sk->makeExternalLink( $this->getUploadUrl(), wfMsg( 'uploadnewversion-linktext' ) ); - $wgOut->addHtml( "
      • " ); + $wgOut->addHTML( "
      • {$ulink}
      • \n" ); } - # Link to Special:FileDuplicateSearch - $dupeLink = $sk->makeKnownLinkObj( SpecialPage::getTitleFor( 'FileDuplicateSearch', $this->mTitle->getDBkey() ), wfMsgHtml( 'imagepage-searchdupe' ) ); - $wgOut->addHtml( "
      • {$dupeLink}
      • " ); - # External editing link - $elink = $sk->makeKnownLinkObj( $this->mTitle, wfMsgHtml( 'edit-externally' ), 'action=edit&externaledit=true&mode=file' ); - $wgOut->addHtml( '
      • ' . $elink . '
        ' . wfMsgWikiHtml( 'edit-externally-help' ) . '
      • ' ); + if ( $wgUseExternalEditor ) { + $elink = $sk->link( + $this->mTitle, + wfMsgHtml( 'edit-externally' ), + array(), + array( + 'action' => 'edit', + 'externaledit' => 'true', + 'mode' => 'file' + ), + array( 'known', 'noclasses' ) + ); + $wgOut->addHTML( '
      • ' . $elink . ' ' . wfMsgExt( 'edit-externally-help', array( 'parseinline' ) ) . "
      • \n" ); + } - $wgOut->addHtml( '
      ' ); + $wgOut->addHTML( "
    \n" ); } - function closeShowImage() - { - # For overloading - - } + protected function closeShowImage() { } # For overloading /** * If the page we've just displayed is in the "Image" namespace, * we follow it with an upload history of the image and its usage. */ - function imageHistory() - { - global $wgOut, $wgUseExternalEditor; + protected function imageHistory() { + global $wgOut; $this->loadFile(); - if ( $this->img->exists() ) { - $list = new ImageHistoryList( $this ); - $file = $this->img; - $dims = $file->getDimensionsString(); - $s = $list->beginImageHistoryList(); - $s .= $list->imageHistoryLine( true, $file ); - // old image versions - $hist = $this->img->getHistory(); - foreach( $hist as $file ) { - $dims = $file->getDimensionsString(); - $s .= $list->imageHistoryLine( false, $file ); - } - $s .= $list->endImageHistoryList(); - } else { $s=''; } - $wgOut->addHTML( $s ); + $pager = new ImageHistoryPseudoPager( $this ); + $wgOut->addHTML( $pager->getBody() ); - $this->img->resetHistory(); // free db resources + $this->img->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( $wgUseExternalEditor && $this->img->exists() ) { + if ( $this->img->exists() ) { $this->uploadLinksBox(); } - } - function imageLinks() - { - global $wgUser, $wgOut; - + protected function imageLinks() { + global $wgUser, $wgOut, $wgLang; + $limit = 100; $dbr = wfGetDB( DB_SLAVE ); @@ -640,60 +627,90 @@ EOT array( 'page_namespace', 'page_title' ), array( 'il_to' => $this->mTitle->getDBkey(), 'il_from = page_id' ), __METHOD__, - array( 'LIMIT' => $limit + 1) + array( 'LIMIT' => $limit + 1 ) ); - - if ( 0 == $dbr->numRows( $res ) ) { - $wgOut->addHTML( "
    \n" ); - $wgOut->addWikiMsg( 'nolinkstoimage' ); - $wgOut->addHTML( "
    \n" ); + $count = $dbr->numRows( $res ); + if ( $count == 0 ) { + $wgOut->wrapWikiMsg( Html::rawElement( 'div', array ( 'id' => 'mw-imagepage-nolinkstoimage' ), "\n$1\n" ), 'nolinkstoimage' ); return; } + $wgOut->addHTML( "
    \n" ); - $wgOut->addWikiMsg( 'linkstoimage' ); - $wgOut->addHTML( "
      \n" ); + if ( $count <= $limit - 1 ) { + $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() + ); + } + $wgOut->addHTML( Html::openElement( 'ul', array( 'class' => 'mw-imagepage-linkstoimage' ) ) . "\n" ); $sk = $wgUser->getSkin(); $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 - $name = Title::makeTitle( $s->page_namespace, $s->page_title ); - $link = $sk->makeKnownLinkObj( $name, "" ); - $wgOut->addHTML( "
    • {$link}
    • \n" ); + $elements[] = $s; } } - $wgOut->addHTML( "
    \n" ); + + // Sort the list by namespace:title + usort ( $elements, 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( + 'li', + array( 'id' => 'mw-imagepage-linkstoimage-ns' . $element->page_namespace ), + $link + ) . "\n" + ); + + }; + $wgOut->addHTML( Html::closeElement( 'ul' ) . "\n" ); $res->free(); - + // Add a links to [[Special:Whatlinkshere]] - if ( $count > $limit ) + if ( $count > $limit ) { $wgOut->addWikiMsg( 'morelinkstoimage', $this->mTitle->getPrefixedDBkey() ); + } + $wgOut->addHTML( Html::closeElement( 'div' ) . "\n" ); } - function imageRedirects() - { - global $wgUser, $wgOut; - - $redirects = $this->getTitle()->getRedirectsHere( NS_IMAGE ); + protected function imageRedirects() { + global $wgUser, $wgOut, $wgLang; + + $redirects = $this->getTitle()->getRedirectsHere( NS_FILE ); if ( count( $redirects ) == 0 ) return; $wgOut->addHTML( "
    \n" ); - $wgOut->addWikiMsg( 'redirectstofile' ); + $wgOut->addWikiMsg( 'redirectstofile', + $wgLang->formatNum( count( $redirects ) ) + ); $wgOut->addHTML( "
      \n" ); $sk = $wgUser->getSkin(); foreach ( $redirects as $title ) { - $link = $sk->makeKnownLinkObj( $title, "" ); + $link = $sk->link( + $title, + null, + array(), + array( 'redirect' => 'no' ), + array( 'known', 'noclasses' ) + ); $wgOut->addHTML( "
    • {$link}
    • \n" ); } $wgOut->addHTML( "
    \n" ); } - - function imageDupes() { - global $wgOut, $wgUser; + + protected function imageDupes() { + global $wgOut, $wgUser, $wgLang; $this->loadFile(); @@ -701,17 +718,28 @@ EOT if ( count( $dupes ) == 0 ) return; $wgOut->addHTML( "
    \n" ); - $wgOut->addWikiMsg( 'duplicatesoffile' ); + $wgOut->addWikiMsg( 'duplicatesoffile', + $wgLang->formatNum( count( $dupes ) ), $this->mTitle->getDBkey() + ); $wgOut->addHTML( "
      \n" ); $sk = $wgUser->getSkin(); foreach ( $dupes as $file ) { - if ( $file->isLocal() ) - $link = $sk->makeKnownLinkObj( $file->getTitle(), "" ); - else - $link = $sk->makeExternalLink( $file->getDescriptionUrl(), + $fromSrc = ''; + if ( $file->isLocal() ) { + $link = $sk->link( + $file->getTitle(), + null, + array(), + array(), + array( 'known', 'noclasses' ) + ); + } else { + $link = $sk->makeExternalLink( $file->getDescriptionUrl(), $file->getTitle()->getPrefixedText() ); - $wgOut->addHTML( "
    • {$link}
    • \n" ); + $fromSrc = wfMsg( 'shared-repo-from', $file->getRepo()->getDisplayName() ); + } + $wgOut->addHTML( "
    • {$link} {$fromSrc}
    • \n" ); } $wgOut->addHTML( "
    \n" ); } @@ -720,10 +748,17 @@ EOT * Delete the file, or an earlier version of it */ public function delete() { + global $wgUploadMaintenance; + if ( $wgUploadMaintenance && $this->mTitle && $this->mTitle->getNamespace() == NS_FILE ) { + global $wgOut; + $wgOut->wrapWikiMsg( "
    \n$1\n
    \n", array( 'filedelete-maintenance' ) ); + return; + } + $this->loadFile(); - if( !$this->img->exists() || !$this->img->isLocal() || $this->img->getRedirected() ) { + if ( !$this->img->exists() || !$this->img->isLocal() || $this->img->getRedirected() ) { // Standard article deletion - Article::delete(); + parent::delete(); return; } $deleter = new FileDeleteForm( $this->img ); @@ -742,16 +777,19 @@ EOT /** * Override handling of action=purge */ - function doPurge() { + public function doPurge() { $this->loadFile(); - if( $this->img->exists() ) { + 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\n" ); + 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(); } @@ -762,12 +800,28 @@ EOT function showError( $description ) { global $wgOut; $wgOut->setPageTitle( wfMsg( "internalerror" ) ); - $wgOut->setRobotpolicy( "noindex,nofollow" ); + $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 + */ + protected function compare( $a, $b ) { + if ( $a->page_namespace == $b->page_namespace ) { + return strcmp( $a->page_title, $b->page_title ); + } else { + return $a->page_namespace - $b->page_namespace; + } + } } /** @@ -777,155 +831,172 @@ EOT */ class ImageHistoryList { - protected $imagePage, $img, $skin, $title, $repo; + protected $imagePage, $img, $skin, $title, $repo, $showThumb; public function __construct( $imagePage ) { - global $wgUser; + global $wgUser, $wgShowArchiveThumbnails; $this->skin = $wgUser->getSkin(); $this->current = $imagePage->getFile(); $this->img = $imagePage->getDisplayedFile(); $this->title = $imagePage->getTitle(); $this->imagePage = $imagePage; + $this->showThumb = $wgShowArchiveThumbnails && $this->img->canRender(); } - function getImagePage() { + public function getImagePage() { return $this->imagePage; } - function getSkin() { + public function getSkin() { return $this->skin; } - function getFile() { + public function getFile() { return $this->img; } - public function beginImageHistoryList() { + public function beginImageHistoryList( $navLinks = '' ) { global $wgOut, $wgUser; - return Xml::element( 'h2', array( 'id' => 'filehistory' ), wfMsg( 'filehist' ) ) + return Xml::element( 'h2', array( 'id' => 'filehistory' ), wfMsg( 'filehist' ) ) . "\n" + . "
    \n" . $wgOut->parse( wfMsgNoTrans( 'filehist-help' ) ) - . Xml::openElement( 'table', array( 'class' => 'filehistory' ) ) . "\n" + . $navLinks . "\n" + . Xml::openElement( 'table', array( 'class' => 'wikitable filehistory' ) ) . "\n" . '
    ' . wfMsgHtml( 'filehist-datetime' ) . '' . wfMsgHtml( 'filehist-thumb' ) . '' . wfMsgHtml( 'filehist-dimensions' ) . '' . wfMsgHtml( 'filehist-user' ) . '' . wfMsgHtml( 'filehist-comment' ) . '' . wfMsgHtml( 'filehist-user' ) . '' . wfMsgHtml( 'filehist-comment' ) . '
    \n"; + public function endImageHistoryList( $navLinks = '' ) { + return "\n$navLinks\n\n"; } public function imageHistoryLine( $iscur, $file ) { - global $wgUser, $wgLang, $wgContLang, $wgTitle; + global $wgUser, $wgLang; - $timestamp = wfTimestamp(TS_MW, $file->getTimestamp()); + $timestamp = wfTimestamp( TS_MW, $file->getTimestamp() ); $img = $iscur ? $file->getName() : $file->getArchiveName(); - $user = $file->getUser('id'); - $usertext = $file->getUser('text'); - $size = $file->getSize(); + $user = $file->getUser( 'id' ); + $usertext = $file->getUser( 'text' ); $description = $file->getDescription(); - $dims = $file->getDimensionsString(); - $sha1 = $file->getSha1(); $local = $this->current->isLocal(); - $row = $css = $selected = ''; + $row = $selected = ''; // Deletion link - if( $local && ($wgUser->isAllowed('delete') || $wgUser->isAllowed('deleterevision') ) ) { + if ( $local && ( $wgUser->isAllowed( 'delete' ) || $wgUser->isAllowed( 'deletedhistory' ) ) ) { $row .= ''; # Link to remove from history - if( $wgUser->isAllowed( 'delete' ) ) { - $q = array(); - $q[] = 'action=delete'; - if( !$iscur ) - $q[] = 'oldimage=' . urlencode( $img ); - $row .= $this->skin->makeKnownLinkObj( + if ( $wgUser->isAllowed( 'delete' ) ) { + $q = array( 'action' => 'delete' ); + if ( !$iscur ) + $q['oldimage'] = $img; + $row .= $this->skin->link( $this->title, wfMsgHtml( $iscur ? 'filehist-deleteall' : 'filehist-deleteone' ), - implode( '&', $q ) + array(), $q, array( 'known' ) ); } - # Link to hide content - if( $wgUser->isAllowed( 'deleterevision' ) ) { - if( $wgUser->isAllowed('delete') ) { - $row .= '
    '; + # Link to hide content. Don't show useless link to people who cannot hide revisions. + $canHide = $wgUser->isAllowed( 'deleterevision' ); + if ( $canHide || ( $wgUser->isAllowed( 'deletedhistory' ) && $file->getVisibility() ) ) { + if ( $wgUser->isAllowed( 'delete' ) ) { + $row .= '
    '; } - $revdel = SpecialPage::getTitleFor( 'Revisiondelete' ); // If file is top revision or locked from this user, don't link - if( $iscur || !$file->userCan(File::DELETED_RESTRICTED) ) { - $del = wfMsgHtml( 'rev-delundel' ); + if ( $iscur || !$file->userCan( File::DELETED_RESTRICTED ) ) { + $del = $this->skin->revDeleteLinkDisabled( $canHide ); } else { - // If the file was hidden, link to sha-1 - list($ts,$name) = explode('!',$img,2); - $del = $this->skin->makeKnownLinkObj( $revdel, wfMsg( 'rev-delundel' ), - 'target=' . urlencode( $wgTitle->getPrefixedText() ) . - '&oldimage=' . urlencode( $ts ) ); - // Bolden oversighted content - if( $file->isDeleted(File::DELETED_RESTRICTED) ) - $del = "$del"; + list( $ts, $name ) = explode( '!', $img, 2 ); + $query = array( + 'type' => 'oldimage', + 'target' => $this->title->getPrefixedText(), + 'ids' => $ts, + ); + $del = $this->skin->revDeleteLink( $query, + $file->isDeleted( File::DELETED_RESTRICTED ), $canHide ); } - $row .= "$del"; + $row .= $del; } $row .= ''; } // Reversion link/current indicator $row .= ''; - if( $iscur ) { + if ( $iscur ) { $row .= wfMsgHtml( 'filehist-current' ); - } elseif( $local && $wgUser->isLoggedIn() && $this->title->userCan( 'edit' ) ) { - if( $file->isDeleted(File::DELETED_FILE) ) { - $row .= wfMsgHtml('filehist-revert'); + } elseif ( $local && $wgUser->isLoggedIn() && $this->title->userCan( 'edit' ) ) { + if ( $file->isDeleted( File::DELETED_FILE ) ) { + $row .= wfMsgHtml( 'filehist-revert' ); } else { - $q = array(); - $q[] = 'action=revert'; - $q[] = 'oldimage=' . urlencode( $img ); - $q[] = 'wpEditToken=' . urlencode( $wgUser->editToken( $img ) ); - $row .= $this->skin->makeKnownLinkObj( $this->title, + $row .= $this->skin->link( + $this->title, wfMsgHtml( 'filehist-revert' ), - implode( '&', $q ) ); + array(), + array( + 'action' => 'revert', + 'oldimage' => $img, + 'wpEditToken' => $wgUser->editToken( $img ) + ), + array( 'known', 'noclasses' ) + ); } } $row .= ''; // Date/time and image link - if( $file->getTimestamp() === $this->img->getTimestamp() ) { + if ( $file->getTimestamp() === $this->img->getTimestamp() ) { $selected = "class='filehistory-selected'"; } $row .= ""; - if( !$file->userCan(File::DELETED_FILE) ) { + if ( !$file->userCan( File::DELETED_FILE ) ) { # Don't link to unviewable files $row .= '' . $wgLang->timeAndDate( $timestamp, true ) . ''; - } else if( $file->isDeleted(File::DELETED_FILE) ) { + } elseif ( $file->isDeleted( File::DELETED_FILE ) ) { $revdel = SpecialPage::getTitleFor( 'Revisiondelete' ); # Make a link to review the image - $url = $this->skin->makeKnownLinkObj( $revdel, $wgLang->timeAndDate( $timestamp, true ), - "target=".$wgTitle->getPrefixedText()."&file=$sha1.".$this->current->getExtension() ); - $row .= ''.$url.''; + $url = $this->skin->link( + $revdel, + $wgLang->timeAndDate( $timestamp, true ), + array(), + array( + 'target' => $this->title->getPrefixedText(), + 'file' => $img, + 'token' => $wgUser->editToken( $img ) + ), + array( 'known', 'noclasses' ) + ); + $row .= '' . $url . ''; } else { $url = $iscur ? $this->current->getUrl() : $this->current->getArchiveUrl( $img ); $row .= Xml::element( 'a', array( 'href' => $url ), $wgLang->timeAndDate( $timestamp, true ) ); } + $row .= ""; - $row .= ""; - - // Image dimensions - $row .= htmlspecialchars( $dims ); + // Thumbnail + if ( $this->showThumb ) { + $row .= '' . $this->getThumbForLine( $file ) . ''; + } - // File size - $row .= " (" . $this->skin->formatSize( $size ) . ')'; + // Image dimensions + size + $row .= ''; + $row .= htmlspecialchars( $file->getDimensionsString() ); + $row .= " (" . $this->skin->formatSize( $file->getSize() ) . ')'; + $row .= ''; // Uploading user - $row .= ''; - if( $local ) { + $row .= ''; + if ( $local ) { // Hide deleted usernames - if( $file->isDeleted(File::DELETED_USER) ) { + if ( $file->isDeleted( File::DELETED_USER ) ) { $row .= '' . wfMsgHtml( 'rev-deleted-user' ) . ''; } else { - $row .= $this->skin->userLink( $user, $usertext ) . " " . + $row .= $this->skin->userLink( $user, $usertext ) . " " . $this->skin->userToolLinks( $user, $usertext ) . ""; } } else { @@ -934,8 +1005,8 @@ class ImageHistoryList { $row .= ''; // Don't show deleted descriptions - if ( $file->isDeleted(File::DELETED_COMMENT) ) { - $row .= '' . wfMsgHtml('rev-deleted-comment') . ''; + if ( $file->isDeleted( File::DELETED_COMMENT ) ) { + $row .= '' . wfMsgHtml( 'rev-deleted-comment' ) . ''; } else { $row .= $this->skin->commentBlock( $description, $this->title ); } @@ -946,4 +1017,156 @@ class ImageHistoryList { return "{$row}\n"; } + + protected function getThumbForLine( $file ) { + global $wgLang; + + if ( $file->allowInlineDisplay() && $file->userCan( File::DELETED_FILE ) && !$file->isDeleted( File::DELETED_FILE ) ) { + $params = array( + 'width' => '120', + 'height' => '120', + ); + $timestamp = wfTimestamp( TS_MW, $file->getTimestamp() ); + + $thumbnail = $file->transform( $params ); + $options = array( + 'alt' => wfMsg( 'filehist-thumbtext', + $wgLang->timeAndDate( $timestamp, true ), + $wgLang->date( $timestamp, true ), + $wgLang->time( $timestamp, true ) ), + 'file-link' => true, + ); + + if ( !$thumbnail ) return wfMsgHtml( 'filehist-nothumb' ); + + return $thumbnail->toHtml( $options ); + } else { + return wfMsgHtml( 'filehist-nothumb' ); + } + } +} + +class ImageHistoryPseudoPager extends ReverseChronologicalPager { + function __construct( $imagePage ) { + parent::__construct(); + $this->mImagePage = $imagePage; + $this->mTitle = clone ( $imagePage->getTitle() ); + $this->mTitle->setFragment( '#filehistory' ); + $this->mImg = null; + $this->mHist = array(); + $this->mRange = array( 0, 0 ); // display range + } + + function getTitle() { + return $this->mTitle; + } + + function getQueryInfo() { + return false; + } + + function getIndexField() { + return ''; + } + + function formatRow( $row ) { + return ''; + } + + function getBody() { + $s = ''; + $this->doQuery(); + if ( count( $this->mHist ) ) { + $list = new ImageHistoryList( $this->mImagePage ); + # Generate prev/next links + $navLink = $this->getNavigationBar(); + $s = $list->beginImageHistoryList( $navLink ); + // Skip rows there just for paging links + for ( $i = $this->mRange[0]; $i <= $this->mRange[1]; $i++ ) { + $file = $this->mHist[$i]; + $s .= $list->imageHistoryLine( !$file->isOld(), $file ); + } + $s .= $list->endImageHistoryList( $navLink ); + } + return $s; + } + + function doQuery() { + if ( $this->mQueryDone ) return; + $this->mImg = $this->mImagePage->getFile(); // ensure loading + if ( !$this->mImg->exists() ) { + return; + } + $queryLimit = $this->mLimit + 1; // limit plus extra row + if ( $this->mIsBackwards ) { + // Fetch the file history + $this->mHist = $this->mImg->getHistory( $queryLimit, null, $this->mOffset, false ); + // The current rev may not meet the offset/limit + $numRows = count( $this->mHist ); + if ( $numRows <= $this->mLimit && $this->mImg->getTimestamp() > $this->mOffset ) { + $this->mHist = array_merge( array( $this->mImg ), $this->mHist ); + } + } else { + // The current rev may not meet the offset + if ( !$this->mOffset || $this->mImg->getTimestamp() < $this->mOffset ) { + $this->mHist[] = $this->mImg; + } + // Old image versions (fetch extra row for nav links) + $oiLimit = count( $this->mHist ) ? $this->mLimit : $this->mLimit + 1; + // Fetch the file history + $this->mHist = array_merge( $this->mHist, + $this->mImg->getHistory( $oiLimit, $this->mOffset, null, false ) ); + } + $numRows = count( $this->mHist ); // Total number of query results + if ( $numRows ) { + # Index value of top item in the list + $firstIndex = $this->mIsBackwards ? + $this->mHist[$numRows - 1]->getTimestamp() : $this->mHist[0]->getTimestamp(); + # Discard the extra result row if there is one + if ( $numRows > $this->mLimit && $numRows > 1 ) { + if ( $this->mIsBackwards ) { + # Index value of item past the index + $this->mPastTheEndIndex = $this->mHist[0]->getTimestamp(); + # Index value of bottom item in the list + $lastIndex = $this->mHist[1]->getTimestamp(); + # Display range + $this->mRange = array( 1, $numRows - 1 ); + } else { + # Index value of item past the index + $this->mPastTheEndIndex = $this->mHist[$numRows - 1]->getTimestamp(); + # Index value of bottom item in the list + $lastIndex = $this->mHist[$numRows - 2]->getTimestamp(); + # Display range + $this->mRange = array( 0, $numRows - 2 ); + } + } else { + # Setting indexes to an empty string means that they will be + # omitted if they would otherwise appear in URLs. It just so + # happens that this is the right thing to do in the standard + # UI, in all the relevant cases. + $this->mPastTheEndIndex = ''; + # Index value of bottom item in the list + $lastIndex = $this->mIsBackwards ? + $this->mHist[0]->getTimestamp() : $this->mHist[$numRows - 1]->getTimestamp(); + # Display range + $this->mRange = array( 0, $numRows - 1 ); + } + } else { + $firstIndex = ''; + $lastIndex = ''; + $this->mPastTheEndIndex = ''; + } + if ( $this->mIsBackwards ) { + $this->mIsFirst = ( $numRows < $queryLimit ); + $this->mIsLast = ( $this->mOffset == '' ); + $this->mLastShown = $firstIndex; + $this->mFirstShown = $lastIndex; + } else { + $this->mIsFirst = ( $this->mOffset == '' ); + $this->mIsLast = ( $numRows < $queryLimit ); + $this->mLastShown = $lastIndex; + $this->mFirstShown = $firstIndex; + } + $this->mQueryDone = true; + } }