* (bug 14258, 14368) Fix for subpage renames in replication environments

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

diff --git a/includes/SpecialMostlinked.php b/includes/SpecialMostlinked.php
index f50120c..a56ac26 100644 (file)
--- a/includes/SpecialMostlinked.php
+++ b/includes/SpecialMostlinked.php
@@ -1,10 +1,14 @@
 <?php
+/**
+ * @file
+ * @ingroup SpecialPage
+ */
 
 /**
- * A special page to show pages ordered by the number of pages linking to them
+ * A special page to show pages ordered by the number of pages linking to them.
+ * Implements Special:Mostlinked
  *
- * @package MediaWiki
- * @subpackage SpecialPage
+ * @ingroup SpecialPage
  *
  * @author Ævar Arnfjörð Bjarmason <avarab@gmail.com>
  * @author Rob Church <robchur@gmail.com>
@@ -12,11 +16,6 @@
  * @copyright © 2006 Rob Church
  * @license http://www.gnu.org/copyleft/gpl.html GNU General Public License 2.0 or later
  */
-
-/**
- * @package MediaWiki
- * @subpackage SpecialPage
- */
 class MostlinkedPage extends QueryPage {
 
        function getName() { return 'Mostlinked'; }
@@ -27,8 +26,8 @@ class MostlinkedPage extends QueryPage {
         * Note: Getting page_namespace only works if $this->isCached() is false
         */
        function getSQL() {
-               $dbr =& wfGetDB( DB_SLAVE );
-               extract( $dbr->tableNames( 'pagelinks', 'page' ) );
+               $dbr = wfGetDB( DB_SLAVE );
+               list( $pagelinks, $page ) = $dbr->tableNamesN( 'pagelinks', 'page' );
                return
                        "SELECT 'Mostlinked' AS type,
                                pl_namespace AS namespace,
@@ -44,12 +43,12 @@ class MostlinkedPage extends QueryPage {
        /**
         * Pre-fill the link cache
         */
-       function preprocessResults( &$dbr, $res ) {
-               if( $dbr->numRows( $res ) > 0 ) {
+       function preprocessResults( $db, $res ) {
+               if( $db->numRows( $res ) > 0 ) {
                        $linkBatch = new LinkBatch();
-                       while( $row = $dbr->fetchObject( $res ) )
-                               $linkBatch->addObj( Title::makeTitleSafe( $row->namespace, $row->title ) );
-                       $dbr->dataSeek( $res, 0 );
+                       while( $row = $db->fetchObject( $res ) )
+                               $linkBatch->add( $row->namespace, $row->title );
+                       $db->dataSeek( $res, 0 );
                        $linkBatch->execute();
                }
        }
@@ -58,26 +57,28 @@ class MostlinkedPage extends QueryPage {
         * Make a link to "what links here" for the specified title
         *
         * @param $title Title being queried
+        * @param $skin Skin to use
         * @return string
         */
-       function makeWlhLink( &$title, $caption ) {
+       function makeWlhLink( &$title, $caption, &$skin ) {
                $wlh = SpecialPage::getTitleFor( 'Whatlinkshere', $title->getPrefixedDBkey() );
-               return Linker::makeKnownLinkObj( $wlh, $caption );
+               return $skin->makeKnownLinkObj( $wlh, $caption );
        }
 
        /**
         * Make links to the page corresponding to the item, and the "what links here" page for it
         *
+        * @param $skin Skin to be used
         * @param $result Result row
         * @return string
         */
-       function formatResult( $result ) {
+       function formatResult( $skin, $result ) {
                global $wgLang;
                $title = Title::makeTitleSafe( $result->namespace, $result->title );
-               $link = Linker::makeLinkObj( $title );
+               $link = $skin->makeLinkObj( $title );
                $wlh = $this->makeWlhLink( $title,
                        wfMsgExt( 'nlinks', array( 'parsemag', 'escape'),
-                               $wgLang->formatNum( $result->value ) ) );
+                               $wgLang->formatNum( $result->value ) ), $skin );
                return wfSpecialList( $link, $wlh );
        }
 }
@@ -92,5 +93,3 @@ function wfSpecialMostlinked() {
 
        $wpp->doQuery( $offset, $limit );
 }
-
-?>