-<?
+<?php
+/**
+ *
+ * @package MediaWiki
+ * @subpackage SpecialPage
+ */
-include_once( "QueryPage.php" );
+/**
+ *
+ */
+require_once( "QueryPage.php" );
+/**
+ *
+ * @package MediaWiki
+ * @subpackage SpecialPage
+ */
class PopularPagesPage extends QueryPage {
- function getName() {
- return "Popularpages";
- }
-
- function isExpensive() {
- return 1;
- }
-
- function getSQL( $offset, $limit ) {
- return "SELECT DISTINCT cur_title, cur_counter FROM cur " .
- "WHERE cur_namespace=0 AND cur_is_redirect=0 ORDER BY " .
- "cur_counter DESC LIMIT {$offset}, {$limit}";
- }
-
- function formatResult( $skin, $result ) {
- $link = $skin->makeKnownLink( $result->cur_title, "" );
- $nv = wfMsg( "nviews", $result->cur_counter );
- return "{$link} ({$nv})";
- }
+ function getName() {
+ return "Popularpages";
+ }
+
+ function isExpensive() {
+ # cur_counter is not indexed
+ return true;
+ }
+
+ function getSQL() {
+ $dbr =& wfGetDB( DB_SLAVE );
+ $cur = $dbr->tableName( 'cur' );
+
+ return
+ "SELECT 'Popularpages' as type,
+ cur_namespace as namespace,
+ cur_title as title,
+ cur_counter as value
+ FROM $cur
+ WHERE cur_namespace=0 AND cur_is_redirect=0";
+ }
+
+ function formatResult( $skin, $result ) {
+ global $wgLang;
+ $link = $skin->makeKnownLink( $result->title, "" );
+ $nv = wfMsg( "nviews", $wgLang->formatNum( $result->value ) );
+ return "{$link} ({$nv})";
+ }
}
-function wfSpecialPopularpages()
-{
+/**
+ * Constructor
+ */
+function wfSpecialPopularpages() {
list( $limit, $offset ) = wfCheckLimits();
$ppp = new PopularPagesPage();