Removing leecruft. No, you sure as hell couldn't defer it.

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

diff --git a/includes/SpecialListredirects.php b/includes/SpecialListredirects.php
index e309a36..1ac7de4 100644 (file)
--- a/includes/SpecialListredirects.php
+++ b/includes/SpecialListredirects.php
@@ -4,7 +4,7 @@
  * @subpackage SpecialPage
  *
  * @author Rob Church <robchur@gmail.com>
- * @copyright © 2006 Rob Church
+ * @copyright © 2006 Rob Church
  * @license http://www.gnu.org/copyleft/gpl.html GNU General Public License 2.0 or later
  */
 
@@ -22,31 +22,39 @@ class ListredirectsPage extends QueryPage {
        function isExpensive() { return( true ); }
        function isSyndicated() { return( false ); }
        function sortDescending() { return( false ); }
-       
+
        function getSQL() {
                $dbr =& wfGetDB( DB_SLAVE );
                extract( $dbr->tableNames( 'page' ) );
                return( 'SELECT page_title AS title, page_namespace AS namespace, page_namespace AS value FROM ' . $page . ' WHERE page_is_redirect = 1' );
        }
-       
+
        function formatResult( $skin, $result ) {
                global $wgContLang;
-               
+
                # Make a link to the redirect itself
                $rd_title = Title::makeTitle( $result->namespace, $result->title );
                $rd_link = $skin->makeKnownLinkObj( $rd_title, '', 'redirect=no' );
-               
+
                # Find out where the redirect leads
-               $rd_page = new Article( &$rd_title, 0 );
-               $rd_text = $rd_page->getContent( true ); # Don't follow the redirect
-               
-               # Make a link to the destination page
-               $tp_title = Title::newFromRedirect( $rd_text );
-               $tp_link = $skin->makeKnownLinkObj( $tp_title );
-               
+               $revision = Revision::newFromTitle( $rd_title );
+               if( $revision ) {
+                       # Make a link to the destination page
+                       $target = Title::newFromRedirect( $revision->getText() );
+                       if( $target ) {
+                               $targetLink = $skin->makeKnownLinkObj( $target );
+                       } else {
+                               /** @todo Put in some decent error display here */
+                               $targetLink = '*';
+                       }
+               } else {
+                       /** @todo Put in some decent error display here */
+                       $targetLink = '*';
+               }
+
                # Format the whole thing and return it
-               return( $rd_link . ' &rarr; ' . $tp_link );
-               
+               return( $rd_link . ' &rarr; ' . $targetLink );
+
        }
 
 }