* Call Linker methods statically

* Use local context instead of global variables

diff --git a/includes/specials/SpecialFewestrevisions.php b/includes/specials/SpecialFewestrevisions.php
index aa0611c..bbeceae 100644 (file)
--- a/includes/specials/SpecialFewestrevisions.php
+++ b/includes/specials/SpecialFewestrevisions.php
@@ -70,24 +70,20 @@ class FewestrevisionsPage extends QueryPage {
         * @param $result Object: database row
         */
        function formatResult( $skin, $result ) {
-               global $wgLang, $wgContLang;
+               global $wgContLang;
 
                $nt = Title::makeTitleSafe( $result->namespace, $result->title );
                if( !$nt ) {
                        return '<!-- bad title -->';
                }
 
-               $text = $wgContLang->convert( $nt->getPrefixedText() );
+               $text = htmlspecialchars( $wgContLang->convert( $nt->getPrefixedText() ) );
+               $plink = Linker::linkKnown( $nt, $text );
 
-               $plink = $skin->linkKnown(
-                       $nt,
-                       $text
-               );
-
-               $nl = wfMsgExt( 'nrevisions', array( 'parsemag', 'escape' ),
-                       $wgLang->formatNum( $result->value ) );
-               $redirect = isset( $result->redirect ) ? ' - ' . wfMsgHtml( 'isredirect' ) : '';
-               $nlink = $skin->linkKnown(
+               $nl = $this->msg( 'nrevisions' )->numParams( $result->value )->escaped();
+               $redirect = isset( $result->redirect ) && $result->redirect ?
+                       ' - ' . wfMsgHtml( 'isredirect' ) : '';
+               $nlink = Linker::linkKnown(
                        $nt,
                        $nl,
                        array(),