Merge "Improve description of paths/urls in the INSTALL file."

[lhc/web/wiklou.git] / includes / specials / SpecialFewestrevisions.php

diff --git a/includes/specials/SpecialFewestrevisions.php b/includes/specials/SpecialFewestrevisions.php
index 3de1c34..7e4bc9c 100644 (file)
--- a/includes/specials/SpecialFewestrevisions.php
+++ b/includes/specials/SpecialFewestrevisions.php
@@ -1,5 +1,22 @@
 <?php
 /**
+ * Implements Special:Fewestrevisions
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
+ *
  * @file
  * @ingroup SpecialPage
  */
@@ -12,8 +29,8 @@
  */
 class FewestrevisionsPage extends QueryPage {
 
-       function getName() {
-               return 'Fewestrevisions';
+       function __construct( $name = 'Fewestrevisions' ) {
+               parent::__construct( $name );
        }
 
        function isExpensive() {
@@ -24,62 +41,57 @@ class FewestrevisionsPage extends QueryPage {
                return false;
        }
 
-       function getSql() {
-               $dbr = wfGetDB( DB_SLAVE );
-               list( $revision, $page ) = $dbr->tableNamesN( 'revision', 'page' );
-
-               return "SELECT 'Fewestrevisions' as type,
-                               page_namespace as namespace,
-                               page_title as title,
-                               page_is_redirect as redirect,
-                               COUNT(*) as value
-                       FROM $revision
-                       JOIN $page ON page_id = rev_page
-                       WHERE page_namespace = " . NS_MAIN . "
-                       GROUP BY page_namespace, page_title, page_is_redirect
-                       HAVING COUNT(*) > 1";
+       function getQueryInfo() {
+               return array (
+                       'tables' => array ( 'revision', 'page' ),
+                       'fields' => array ( 'namespace' => 'page_namespace',
+                                       'title' => 'page_title',
+                                       'value' => 'COUNT(*)',
+                                       'redirect' => 'page_is_redirect' ),
+                       'conds' => array ( 'page_namespace' => MWNamespace::getContentNamespaces(),
+                                       'page_id = rev_page' ),
+                       'options' => array ( 'HAVING' => 'COUNT(*) > 1',
                        // ^^^ This was probably here to weed out redirects.
                        // Since we mark them as such now, it might be
                        // useful to remove this. People _do_ create pages
                        // and never revise them, they aren't necessarily
                        // redirects.
+                       'GROUP BY' => array( 'page_namespace', 'page_title', 'page_is_redirect' ) )
+               );
        }
 
+
        function sortDescending() {
                return false;
        }
 
+       /**
+        * @param $skin Skin object
+        * @param $result Object: database row
+        * @return String
+        */
        function formatResult( $skin, $result ) {
-               global $wgLang, $wgContLang;
+               global $wgContLang;
 
                $nt = Title::makeTitleSafe( $result->namespace, $result->title );
                if( !$nt ) {
-                       return '<!-- bad title -->';
+                       return Html::element( 'span', array( 'class' => 'mw-invalidtitle' ),
+                               Linker::getInvalidTitleDescription( $this->getContext(), $result->namespace, $result->title ) );
                }
-               
-               $text = $wgContLang->convert( $nt->getPrefixedText() );
 
-               $plink = $skin->linkKnown(
-                       $nt,
-                       $text
-               );
+               $text = htmlspecialchars( $wgContLang->convert( $nt->getPrefixedText() ) );
+               $plink = Linker::linkKnown( $nt, $text );
 
-               $nl = wfMsgExt( 'nrevisions', array( 'parsemag', 'escape' ),
-                       $wgLang->formatNum( $result->value ) );
-               $redirect = $result->redirect ? ' - ' . wfMsgHtml( 'isredirect' ) : '';
-               $nlink = $skin->linkKnown(
+               $nl = $this->msg( 'nrevisions' )->numParams( $result->value )->escaped();
+               $redirect = isset( $result->redirect ) && $result->redirect ?
+                       ' - ' . $this->msg( 'isredirect' )->escaped() : '';
+               $nlink = Linker::linkKnown(
                        $nt,
                        $nl,
                        array(),
                        array( 'action' => 'history' )
                ) . $redirect;
 
-               return wfSpecialList( $plink, $nlink );
+               return $this->getLanguage()->specialList( $plink, $nlink );
        }
 }
-
-function wfSpecialFewestrevisions() {
-       list( $limit, $offset ) = wfCheckLimits();
-       $frp = new FewestrevisionsPage();
-       $frp->doQuery( $offset, $limit );
-}