Push msg from ContextSource/RequestContext into IContextSource

[lhc/web/wiklou.git] / includes / actions / CreditsAction.php

diff --git a/includes/actions/CreditsAction.php b/includes/actions/CreditsAction.php
index 26bb70b..3b2f77e 100644 (file)
--- a/includes/actions/CreditsAction.php
+++ b/includes/actions/CreditsAction.php
@@ -29,30 +29,32 @@ class CreditsAction extends FormlessAction {
                return 'credits';
        }
 
-       public function getRestriction() {
-               return null;
+       protected function getDescription() {
+               return wfMsg( 'creditspage' );
        }
 
        /**
         * This is largely cadged from PageHistory::history
+        *
+        * @return String HTML
         */
        public function onView() {
                wfProfileIn( __METHOD__ );
 
                if ( $this->page->getID() == 0 ) {
-                       $s = wfMsg( 'nocredits' );
+                       $s = $this->msg( 'nocredits' )->parse();
                } else {
                        $s = $this->getCredits( -1 );
                }
 
                wfProfileOut( __METHOD__ );
 
-               return $s;
+               return Html::rawElement( 'div', array( 'id' => 'mw-credits' ), $s );
        }
 
        /**
-        * Get a list of contributors of $article
-        * @param $article Article object
+        * Get a list of contributors
+        *
         * @param $cnt Int: maximum list of contributors to show
         * @param $showIfMax Bool: whether to contributors if there more than $cnt
         * @return String: html
@@ -61,8 +63,8 @@ class CreditsAction extends FormlessAction {
                wfProfileIn( __METHOD__ );
                $s = '';
 
-               if ( isset( $cnt ) && $cnt != 0 ) {
-                       $s = self::getAuthor( $this->page );
+               if ( $cnt != 0 ) {
+                       $s = $this->getAuthor( $this->page );
                        if ( $cnt > 1 || $cnt < 0 ) {
                                $s .= ' ' . $this->getContributors( $cnt - 1, $showIfMax );
                        }
@@ -75,32 +77,32 @@ class CreditsAction extends FormlessAction {
        /**
         * Get the last author with the last modification time
         * @param $article Article object
+        * @return String HTML
         */
-       protected static function getAuthor( Article $article ) {
-               global $wgLang;
-
-               $user = User::newFromId( $article->getUser() );
+       protected function getAuthor( Page $article ) {
+               $user = User::newFromName( $article->getUserText(), false );
 
                $timestamp = $article->getTimestamp();
                if ( $timestamp ) {
-                       $d = $wgLang->date( $article->getTimestamp(), true );
-                       $t = $wgLang->time( $article->getTimestamp(), true );
+                       $lang = $this->getLanguage();
+                       $d = $lang->date( $article->getTimestamp(), true );
+                       $t = $lang->time( $article->getTimestamp(), true );
                } else {
                        $d = '';
                        $t = '';
                }
-               return wfMessage( 'lastmodifiedatby', $d, $t )->rawParams( self::userLink( $user ) )->params( $user->getName() )->escaped();
+               return $this->msg( 'lastmodifiedatby', $d, $t )->rawParams(
+                       $this->userLink( $user ) )->params( $user->getName() )->escaped();
        }
 
        /**
         * Get a list of contributors of $article
-        * @param $article Article object
         * @param $cnt Int: maximum list of contributors to show
         * @param $showIfMax Bool: whether to contributors if there more than $cnt
         * @return String: html
         */
        protected function getContributors( $cnt, $showIfMax ) {
-               global $wgLang, $wgHiddenPrefs;
+               global $wgHiddenPrefs;
 
                $contributors = $this->page->getContributors();
 
@@ -110,7 +112,8 @@ class CreditsAction extends FormlessAction {
                if ( $cnt > 0 && $contributors->count() > $cnt ) {
                        $others_link = $this->othersLink();
                        if ( !$showIfMax )
-                               return wfMessage( 'othercontribs' )->rawParams( $others_link )->params( $contributors->count() )->escaped();
+                               return $this->msg( 'othercontribs' )->rawParams(
+                                       $others_link )->params( $contributors->count() )->escaped();
                }
 
                $real_names = array();
@@ -119,16 +122,16 @@ class CreditsAction extends FormlessAction {
 
                # Sift for real versus user names
                foreach ( $contributors as $user ) {
-                       $cnt--;
+                       $cnt--; 
                        if ( $user->isLoggedIn() ) {
-                               $link = self::link( $user );
+                               $link = $this->link( $user );
                                if ( !in_array( 'realname', $wgHiddenPrefs ) && $user->getRealName() ) {
                                        $real_names[] = $link;
                                } else {
                                        $user_names[] = $link;
                                }
                        } else {
-                               $anon_ips[] = self::link( $user );
+                               $anon_ips[] = $this->link( $user );
                        }
 
                        if ( $cnt == 0 ) {
@@ -136,22 +139,24 @@ class CreditsAction extends FormlessAction {
                        }
                }
 
+               $lang = $this->getLanguage();
+
                if ( count( $real_names ) ) {
-                       $real = $wgLang->listToText( $real_names );
+                       $real = $lang->listToText( $real_names );
                } else {
                        $real = false;
                }
 
                # "ThisSite user(s) A, B and C"
                if ( count( $user_names ) ) {
-                       $user = wfMessage( 'siteusers' )->rawParams( $wgLang->listToText( $user_names ) )->params(
+                       $user = $this->msg( 'siteusers' )->rawParams( $lang->listToText( $user_names ) )->params(
                                count( $user_names ) )->escaped();
                } else {
                        $user = false;
                }
 
                if ( count( $anon_ips ) ) {
-                       $anon = wfMessage( 'anonusers' )->rawParams( $wgLang->listToText( $anon_ips ) )->params(
+                       $anon = $this->msg( 'anonusers' )->rawParams( $lang->listToText( $anon_ips ) )->params(
                                count( $anon_ips ) )->escaped();
                } else {
                        $anon = false;
@@ -168,8 +173,8 @@ class CreditsAction extends FormlessAction {
                $count = count( $fulllist );
                # "Based on work by ..."
                return $count
-                       ? wfMessage( 'othercontribs' )->rawParams(
-                               $wgLang->listToText( $fulllist ) )->params( $count )->escaped()
+                       ? $this->msg( 'othercontribs' )->rawParams(
+                               $lang->listToText( $fulllist ) )->params( $count )->escaped()
                        : '';
        }
 
@@ -178,7 +183,7 @@ class CreditsAction extends FormlessAction {
         * @param $user User object
         * @return String: html
         */
-       protected static function link( User $user ) {
+       protected function link( User $user ) {
                global $wgHiddenPrefs;
                if ( !in_array( 'realname', $wgHiddenPrefs ) && !$user->isAnon() ) {
                        $real = $user->getRealName();
@@ -198,16 +203,16 @@ class CreditsAction extends FormlessAction {
         * @param $user User object
         * @return String: html
         */
-       protected static function userLink( User $user ) {
-               $link = self::link( $user );
+       protected function userLink( User $user ) {
+               $link = $this->link( $user );
                if ( $user->isAnon() ) {
-                       return wfMsgExt( 'anonuser', array( 'parseinline', 'replaceafter' ), $link );
+                       return $this->msg( 'anonuser' )->rawParams( $link )->parse();
                } else {
                        global $wgHiddenPrefs;
                        if ( !in_array( 'realname', $wgHiddenPrefs ) && $user->getRealName() ) {
                                return $link;
                        } else {
-                               return wfMessage( 'siteuser' )->rawParams( $link )->params( $user->getName() )->escaped();
+                               return $this->msg( 'siteuser' )->rawParams( $link )->params( $user->getName() )->escaped();
                        }
                }
        }
@@ -218,12 +223,11 @@ class CreditsAction extends FormlessAction {
         * @return String: html
         */
        protected function othersLink() {
-               return Linker::link(
+               return Linker::linkKnown(
                        $this->getTitle(),
-                       wfMsgHtml( 'others' ),
+                       $this->msg( 'others' )->escaped(),
                        array(),
-                       array( 'action' => 'credits' ),
-                       array( 'known' )
+                       array( 'action' => 'credits' )
                );
        }
 }