isExpensive get default value

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

diff --git a/includes/SpecialWantedpages.php b/includes/SpecialWantedpages.php
index acb11c6..bf2d630 100644 (file)
--- a/includes/SpecialWantedpages.php
+++ b/includes/SpecialWantedpages.php
@@ -1,44 +1,53 @@
-<?
+<?php
 
-include_once ( "QueryPage.php" ) ;
+require_once ( "QueryPage.php" ) ;
 
 class WantedPagesPage extends QueryPage {
-    
-    function getName() {
-       return "Wantedpages";
-    }
-    
-    function isExpensive() {
-       return 1;
-    }
-    
-    function getSQL( $offset, $limit ) {
-       return "SELECT bl_to, COUNT( DISTINCT bl_from ) as nlinks " .
-         "FROM brokenlinks GROUP BY bl_to HAVING nlinks > 1 " .
-         "ORDER BY nlinks DESC LIMIT {$offset}, {$limit}";
-    }
-
-    function formatResult( $skin, $result ) {
-       global $wgLang;
-       
-       $nt = Title::newFromDBkey( $result->bl_to );
-
-       $plink = $skin->makeBrokenLink( $nt->getPrefixedText(), "" );
-       $nl = wfMsg( "nlinks", $result->nlinks );
-       $nlink = $skin->makeKnownLink( $wgLang->specialPage( "Whatlinkshere" ), $nl,
-                                    "target=" . $nt->getPrefixedURL() );
-       
-       return "{$plink} ({$nlink})";
-    }
+
+       function getName() {
+               return "Wantedpages";
+       }
+
+       function isExpensive() {
+               return true;
+       }
+
+       function getSQL() {
+               # We cheat and return the full-text from bl_to in the title.
+               # In the future, a pre-parsed name will be available.
+               return
+                       "SELECT 'Wantedpages' as type,
+                               0 as namespace,
+                               bl_to as title,
+                               COUNT(DISTINCT bl_from) as value
+                       FROM brokenlinks
+                       GROUP BY bl_to
+                       HAVING value > 1";
+       }
+
+       function formatResult( $skin, $result ) {
+               global $wgLang;
+
+               $nt = Title::newFromDBkey( $result->title );
+               if( is_null( $nt ) ) {
+                       return "<!-- Bad title '" . htmlspecialchars( $result->title ) . "' -->";
+               }
+               $plink = $skin->makeBrokenLink( $nt->getPrefixedText(), "" );
+               $nl = wfMsg( "nlinks", $result->value );
+               $nlink = $skin->makeKnownLink( $wgLang->specialPage( "Whatlinkshere" ), $nl,
+                 "target=" . $nt->getPrefixedURL() );
+
+               return "{$plink} ({$nlink})";
+       }
 }
 
 function wfSpecialWantedpages()
 {
-    list( $limit, $offset ) = wfCheckLimits();
-    
-    $wpp = new WantedPagesPage();
-    
-    $wpp->doQuery( $offset, $limit );
+       list( $limit, $offset ) = wfCheckLimits();
+
+       $wpp = new WantedPagesPage();
+
+       $wpp->doQuery( $offset, $limit );
 }
 
 ?>