From: Bartosz Dziewoński <matma.rex@gmail.com>
Date: Sun, 24 Nov 2013 17:31:42 +0000 (+0100)
Subject: SpecialRecentChanges: Use $wgRecentChangesFlags in makeLegend()
X-Git-Tag: 1.31.0-rc.0~17919^2
X-Git-Url: http://git.cyclocoop.org//%27%40script%40/%27?a=commitdiff_plain;h=d8990fada1b14a05b2f457595edaf226a85fed1c;p=lhc%2Fweb%2Fwiklou.git

SpecialRecentChanges: Use $wgRecentChangesFlags in makeLegend()

Follow-up to Iabf4873f.

Change-Id: I3741723c59575d265353108ebe189f5b0aaaf141
---

diff --git a/includes/specials/SpecialRecentchanges.php b/includes/specials/SpecialRecentchanges.php
index e8e35aa093..d306744a7c 100644
--- a/includes/specials/SpecialRecentchanges.php
+++ b/includes/specials/SpecialRecentchanges.php
@@ -661,35 +661,31 @@ class SpecialRecentChanges extends IncludableSpecialPage {
 	 * @return string
 	 */
 	public static function makeLegend( IContextSource $context ) {
+		global $wgRecentChangesFlags;
 		$user = $context->getUser();
 		# The legend showing what the letters and stuff mean
 		$legend = Xml::openElement( 'dl', array( 'class' => 'mw-changeslist-legend' ) ) . "\n";
 		# Iterates through them and gets the messages for both letter and tooltip
-		# Messages:
-		# * minoreditletter
-		# * boteditletter
-		# * newpageletter
-		# * unpatrolledletter
-		# * recentchanges-label-minor
-		# * recentchanges-label-bot
-		# * recentchanges-label-newpage
-		# * recentchanges-label-unpatrolled
-		$legendItems = array( 'newpage' => 'newpage', 'minor' => 'minoredit', 'bot' => 'botedit' );
-		if ( $user->useRCPatrol() ) {
-			$legendItems['unpatrolled'] = 'unpatrolled';
-		}
-		foreach ( $legendItems as $label => $letter ) { # generate items of the legend
+		$legendItems = $wgRecentChangesFlags;
+		if ( !$user->useRCPatrol() ) {
+			unset( $legendItems['unpatrolled'] );
+		}
+		foreach ( $legendItems as $key => $legendInfo ) { # generate items of the legend
+			$label = $legendInfo['title'];
+			$letter = $legendInfo['letter'];
+			$cssClass = isset( $legendInfo['class'] ) ? $legendInfo['class'] : $key;
+
 			$legend .= Xml::element( 'dt',
-				array( 'class' => $label ), $context->msg( $letter . 'letter' )->text()
+				array( 'class' => $cssClass ), $context->msg( $letter )->text()
 			) . "\n";
-			if ( $letter === 'newpage' ) {
+			if ( $key === 'newpage' ) {
 				$legend .= Xml::openElement( 'dd' );
-				$legend .= $context->msg( "recentchanges-label-$label" )->escaped();
+				$legend .= $context->msg( $label )->escaped();
 				$legend .= ' ' . $context->msg( 'recentchanges-legend-newpage' )->parse();
 				$legend .= Xml::closeElement( 'dd' ) . "\n";
 			} else {
 				$legend .= Xml::element( 'dd', array(),
-					$context->msg( "recentchanges-label-$label" )->text()
+					$context->msg( $label )->text()
 				) . "\n";
 			}
 		}