Revert revert of setSingleton(), unrelated to broken, accidentally committed code...

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

diff --git a/includes/SpecialStatistics.php b/includes/SpecialStatistics.php
index 6343edd..a29811d 100644 (file)
--- a/includes/SpecialStatistics.php
+++ b/includes/SpecialStatistics.php
@@ -1,18 +1,19 @@
 <?php
+
 /**
-*
-* @addtogroup SpecialPage
-*/
+ * Special page lists various statistics, including the contents of
+ * `site_stats`, plus page view details if enabled
+ *
+ * @addtogroup SpecialPage
+ */
 
 /**
-* constructor
-*/
-function wfSpecialStatistics() {
+ * Show the special page
+ *
+ * @param mixed $par (not used)
+ */
+function wfSpecialStatistics( $par = '' ) {
        global $wgOut, $wgLang, $wgRequest;
-       $fname = 'wfSpecialStatistics';
-
-       $action = $wgRequest->getVal( 'action' );
-
        $dbr = wfGetDB( DB_SLAVE );
 
        $views = SiteStats::views();
@@ -24,14 +25,15 @@ function wfSpecialStatistics() {
        $admins = SiteStats::admins();
        $numJobs = SiteStats::jobs();
 
-       if ($action == 'raw') {
+       if( $wgRequest->getVal( 'action' ) == 'raw' ) {
                $wgOut->disable();
                header( 'Pragma: nocache' );
                echo "total=$total;good=$good;views=$views;edits=$edits;users=$users;admins=$admins;images=$images;jobs=$numJobs\n";
                return;
        } else {
-               $text = '==' . wfMsg( 'sitestats' ) . "==\n" ;
-               $text .= wfMsgExt( 'sitestatstext', array ( 'parsemag' ),
+               $text = "__NOTOC__\n";
+               $text .= '==' . wfMsg( 'sitestats' ) . "==\n";
+               $text .= wfMsgExt( 'sitestatstext', array( 'parsemag' ),
                        $wgLang->formatNum( $total ),
                        $wgLang->formatNum( $good ),
                        $wgLang->formatNum( $views ),
@@ -40,44 +42,52 @@ function wfSpecialStatistics() {
                        $wgLang->formatNum( sprintf( '%.2f', $edits ? $views / $edits : 0 ) ),
                        $wgLang->formatNum( $numJobs ),
                        $wgLang->formatNum( $images )
-               );
-
-               $text .= "\n==" . wfMsg( 'userstats' ) . "==\n";
+               )."\n";
 
+               $text .= "==" . wfMsg( 'userstats' ) . "==\n";
                $text .= wfMsgExt( 'userstatstext', array ( 'parsemag' ),
                        $wgLang->formatNum( $users ),
                        $wgLang->formatNum( $admins ),
                        '[[' . wfMsgForContent( 'grouppage-sysop' ) . ']]', # TODO somehow remove, kept for backwards compatibility
                        $wgLang->formatNum( sprintf( '%.2f', $admins / $users * 100 ) ),
                        User::makeGroupLinkWiki( 'sysop' )
-               );
-
-               $wgOut->addWikiText( $text );
+               )."\n";
 
                global $wgDisableCounters, $wgMiserMode, $wgUser, $wgLang, $wgContLang;
                if( !$wgDisableCounters && !$wgMiserMode ) {
-                       $page = $dbr->tableName( 'page' );
-                       $sql = "SELECT page_namespace, page_title, page_counter FROM {$page} WHERE page_is_redirect = 0 AND page_counter > 0 ORDER BY page_counter DESC";
-                       $sql = $dbr->limitResult($sql, 10, 0);
-                       $res = $dbr->query( $sql, $fname );
-                       if( $res ) {
-                               $wgOut->addHtml( '<h2>' . wfMsgHtml( 'statistics-mostpopular' ) . '</h2>' );
-                               $skin = $wgUser->getSkin();
-                               $wgOut->addHtml( '<ol>' );
-                               while( $row = $dbr->fetchObject( $res ) ) {
-                                       $link = $skin->makeKnownLinkObj( Title::makeTitleSafe( $row->page_namespace, $row->page_title ) );
-                                       $dirmark = $wgContLang->getDirMark();
-                                       $wgOut->addHtml( '<li>' . $link . $dirmark . ' [' . $wgLang->formatNum( $row->page_counter ) . ']</li>' );
+                       $res = $dbr->select(
+                               'page',
+                               array(
+                                       'page_namespace',
+                                       'page_title',
+                                       'page_counter',
+                               ),
+                               array(
+                                       'page_is_redirect' => 0,
+                                       'page_counter > 0',
+                               ),
+                               __METHOD__,
+                               array(
+                                       'ORDER BY' => 'page_counter DESC',
+                                       'LIMIT' => 10,
+                               )
+                       );
+                       if( $res->numRows() > 0 ) {
+                               $text .= "==" . wfMsg( 'statistics-mostpopular' ) . "==\n";
+                               while( $row = $res->fetchObject() ) {
+                                       $title = Title::makeTitleSafe( $row->page_namespace, $row->page_title );
+                                       if( $title instanceof Title )
+                                               $text .= '* [[:' . $title->getPrefixedText() . ']] (' . $wgLang->formatNum( $row->page_counter ) . ")\n";
                                }
-                               $wgOut->addHtml( '</ol>' );
-                               $dbr->freeResult( $res );
+                               $res->free();
                        }
                }
                
                $footer = wfMsg( 'statistics-footer' );
                if( !wfEmptyMsg( 'statistics-footer', $footer ) && $footer != '' )
-                       $wgOut->addWikiText( $footer );
-               
+                       $text .= "\n" . $footer;
+                       
+               $wgOut->addWikiText( $text );           
        }
-}
-
+       
+}
\ No newline at end of file