* $wgDebugTidy feature

[lhc/web/wiklou.git] / includes / SpecialShortpages.php

diff --git a/includes/SpecialShortpages.php b/includes/SpecialShortpages.php
index 70ca503..973656d 100644 (file)
--- a/includes/SpecialShortpages.php
+++ b/includes/SpecialShortpages.php
@@ -1,20 +1,13 @@
 <?php
 /**
  *
- * @package MediaWiki
- * @subpackage SpecialPage
+ * @addtogroup SpecialPage
  */
 
-/**
- *
- */
-require_once("QueryPage.php");
-
 /**
  * SpecialShortpages extends QueryPage. It is used to return the shortest
  * pages in the database.
- * @package MediaWiki
- * @subpackage SpecialPage
+ * @addtogroup SpecialPage
  */
 class ShortPagesPage extends QueryPage {
 
@@ -26,37 +19,62 @@ class ShortPagesPage extends QueryPage {
         * This query is indexed as of 1.5
         */
        function isExpensive() {
-               return false;
+               return true;
        }
-       
+
        function isSyndicated() {
                return false;
        }
 
        function getSQL() {
-               $dbr =& wfGetDB( DB_SLAVE );
+               $dbr = wfGetDB( DB_SLAVE );
                $page = $dbr->tableName( 'page' );
                $name = $dbr->addQuotes( $this->getName() );
-               
+
+               $forceindex = $dbr->useIndexClause("page_len");
                return
                        "SELECT $name as type,
-                                       page_namespace as namespace,
+                               page_namespace as namespace,
                                page_title as title,
                                page_len AS value
-                       FROM $page
+                       FROM $page $forceindex
                        WHERE page_namespace=".NS_MAIN." AND page_is_redirect=0";
        }
-       
+
+       function preprocessResults( &$db, &$res ) {
+               # There's no point doing a batch check if we aren't caching results;
+               # the page must exist for it to have been pulled out of the table
+               if( $this->isCached() ) {
+                       $batch = new LinkBatch();
+                       while( $row = $db->fetchObject( $res ) )
+                               $batch->addObj( Title::makeTitleSafe( $row->namespace, $row->title ) );
+                       $batch->execute();
+                       if( $db->numRows( $res ) > 0 )
+                               $db->dataSeek( $res, 0 );
+               }
+       }
+
        function sortDescending() {
                return false;
        }
 
        function formatResult( $skin, $result ) {
                global $wgLang, $wgContLang;
-               $nb = htmlspecialchars( wfMsg( "nbytes", $wgLang->formatNum( $result->value ) ) );
-               $title = Title::makeTitle( $result->namespace, $result->title );
-               $link = $skin->makeKnownLinkObj( $title, $wgContLang->convert( $title->getPrefixedText() ) );
-               return "{$link} ({$nb})";
+               $dm = $wgContLang->getDirMark();
+               
+               $title = Title::makeTitleSafe( $result->namespace, $result->title );
+               if ( !$title ) {
+                       return '<!-- Invalid title ' .  htmlspecialchars( "{$result->namespace}:{$result->title}" ). '-->';
+               }
+               $hlink = $skin->makeKnownLinkObj( $title, wfMsgHtml( 'hist' ), 'action=history' );
+               $plink = $this->isCached()
+                                       ? $skin->makeLinkObj( $title )
+                                       : $skin->makeKnownLinkObj( $title );
+               $size = wfMsgExt( 'nbytes', array( 'parsemag', 'escape' ), $wgLang->formatNum( htmlspecialchars( $result->value ) ) );
+               
+               return $title->exists()
+                               ? "({$hlink}) {$dm}{$plink} {$dm}[{$size}]"
+                               : "<s>({$hlink}) {$dm}{$plink} {$dm}[{$size}]</s>";
        }
 }
 
@@ -71,4 +89,4 @@ function wfSpecialShortpages() {
        return $spp->doQuery( $offset, $limit );
 }
 
-?>
+