From: Alex Monk <krenair@gmail.com>
Date: Sat, 22 Jun 2013 23:04:22 +0000 (+0100)
Subject: Add ID to action=info table rows
X-Git-Tag: 1.31.0-rc.0~19318^2
X-Git-Url: http://git.cyclocoop.org/%27%2C%20%20%20quote_amp%28%24url%29%2C%20%20%20%27?a=commitdiff_plain;h=187321aded3a70114dea93c404cc294ef0aecdd6;p=lhc%2Fweb%2Fwiklou.git

Add ID to action=info table rows

MZMcBride wants to use these to highlight certain pieces of information.

Bug: 44252
Change-Id: Id4fd17e748c0fe3a9f4c51846abcaa8942fa5c50
---

diff --git a/includes/actions/InfoAction.php b/includes/actions/InfoAction.php
index e3ea6fcc4a..012a65566e 100644
--- a/includes/actions/InfoAction.php
+++ b/includes/actions/InfoAction.php
@@ -121,7 +121,8 @@ class InfoAction extends FormlessAction {
 			foreach ( $infoTable as $infoRow ) {
 				$name = ( $infoRow[0] instanceof Message ) ? $infoRow[0]->escaped() : $infoRow[0];
 				$value = ( $infoRow[1] instanceof Message ) ? $infoRow[1]->escaped() : $infoRow[1];
-				$table = $this->addRow( $table, $name, $value ) . "\n";
+				$id = ( $infoRow[0] instanceof Message ) ? $infoRow[0]->getKey() : null;
+				$table = $this->addRow( $table, $name, $value, $id ) . "\n";
 			}
 			$content = $this->addTable( $content, $table ) . "\n";
 		}
@@ -156,10 +157,11 @@ class InfoAction extends FormlessAction {
 	 * @param string $table The table that will be added to the content
 	 * @param string $name The name of the row
 	 * @param string $value The value of the row
+	 * @param string $id The ID to use for the 'tr' element
 	 * @return string The table with the row added
 	 */
-	protected function addRow( $table, $name, $value ) {
-		return $table . Html::rawElement( 'tr', array(),
+	protected function addRow( $table, $name, $value, $id ) {
+		return $table . Html::rawElement( 'tr', $id === null ? array() : array( 'id' => 'mw-' . $id ),
 			Html::rawElement( 'td', array( 'style' => 'vertical-align: top;' ), $name ) .
 			Html::rawElement( 'td', array(), $value )
 		);