r104499 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r104498‎ | r104499 | r104500 >
Date:22:33, 28 November 2011
Author:preilly
Status:ok
Tags:
Comment:
remove heredoc syntax
Modified paths:
  • /trunk/extensions/MobileFrontend/MobileFrontend.php (modified) (history)

Diff [purge]

Index: trunk/extensions/MobileFrontend/MobileFrontend.php
@@ -1084,47 +1084,62 @@
10851085 $username = self::$messages['mobile-frontend-username'];
10861086 $password = self::$messages['mobile-frontend-password'];
10871087 $login = self::$messages['mobile-frontend-login'];
1088 - $loginHtml = Html::input( 'wpName', null, 'text', array(
1089 - 'class' => 'loginText',
1090 - 'id' => 'wpName1',
1091 - 'tabindex' => '1',
1092 - 'size' => '20',
1093 - 'required') );
1094 - $passwordHtml = Html::input( 'wpPassword', null, 'password', array(
1095 - 'class' => 'loginPassword',
1096 - 'id' => 'wpPassword1',
1097 - 'tabindex' => '2',
1098 - 'size' => '20') );
1099 - $buttonHtml = Html::input( 'wpLoginAttempt', $login, 'submit', array(
1100 - 'id' => 'wpLoginAttempt',
1101 - 'tabindex' => '3') );
1102 - $form = <<<EOT
1103 - <form name="userlogin" method="post" action="{$action}">
1104 - <table class="user-login">
1105 - <tbody>
1106 - <tr>
1107 - <td class="mw-label"><label for="wpName1">{$username}</label></td>
1108 - </tr>
1109 - <tr>
1110 - <td class="mw-input">{$loginHtml}</td>
1111 - </tr>
1112 - <tr>
1113 - <td class="mw-label"><label for="wpPassword1">{$password}</label></td>
1114 - </tr>
1115 - <tr>
1116 - <td class="mw-input">{$passwordHtml}</td>
1117 - </tr>
1118 - <tr>
1119 - <td></td>
1120 - </tr>
1121 - <tr>
1122 - <td class="mw-submit">{$buttonHtml}</td>
1123 - </tr>
1124 - </tbody>
1125 - </table>
1126 - <input type="hidden" name="wpLoginToken" value="{$token}">
1127 - </form>
1128 -EOT;
 1088+ $form = Html::openElement( 'form',
 1089+ array( 'name' => 'userlogin',
 1090+ 'method' => 'post',
 1091+ 'action' => $action ) ) .
 1092+ Html::openElement( 'table',
 1093+ array( 'class' => 'user-login' ) ) .
 1094+ Html::openElement( 'tbody' ) .
 1095+ Html::openElement( 'tr' ) .
 1096+ Html::openElement( 'td',
 1097+ array( 'class' => 'mw-label' ) ) .
 1098+ Html::element( 'label',
 1099+ array( 'for' => 'wpName1' ), $username ) .
 1100+ Html::closeElement( 'td' ) .
 1101+ Html::closeElement( 'tr' ) .
 1102+ Html::openElement( 'tr' ) .
 1103+ Html::openElement( 'td' ) .
 1104+ Html::input( 'wpName', null, 'text',
 1105+ array( 'class' => 'loginText',
 1106+ 'id' => 'wpName1',
 1107+ 'tabindex' => '1',
 1108+ 'size' => '20',
 1109+ 'required') ) .
 1110+ Html::closeElement( 'td' ) .
 1111+ Html::closeElement( 'tr' ) .
 1112+ Html::openElement( 'tr' ) .
 1113+ Html::openElement( 'td',
 1114+ array( 'class' => 'mw-label' ) ) .
 1115+ Html::element( 'label',
 1116+ array( 'for' => 'wpPassword1' ), $password ) .
 1117+ Html::closeElement( 'td' ) .
 1118+ Html::closeElement( 'tr' ) .
 1119+ Html::openElement( 'tr' ) .
 1120+ Html::openElement( 'td',
 1121+ array( 'class' => 'mw-input' ) ) .
 1122+ Html::input( 'wpPassword', null, 'password',
 1123+ array( 'class' => 'loginPassword',
 1124+ 'id' => 'wpPassword1',
 1125+ 'tabindex' => '2',
 1126+ 'size' => '20') ) .
 1127+ Html::closeElement( 'td' ) .
 1128+ Html::closeElement( 'tr' ) .
 1129+ Html::openElement( 'tr' ) .
 1130+ Html::element( 'td' ) .
 1131+ Html::closeElement( 'tr' ) .
 1132+ Html::openElement( 'tr' ) .
 1133+ Html::openElement( 'td',
 1134+ array( 'class' => 'mw-submit' ) ) .
 1135+ Html::input( 'wpLoginAttempt', $login, 'submit',
 1136+ array( 'id' => 'wpLoginAttempt',
 1137+ 'tabindex' => '3') ) .
 1138+ Html::closeElement( 'td' ) .
 1139+ Html::closeElement( 'tr' ) .
 1140+ Html::closeElement( 'tbody' ) .
 1141+ Html::closeElement( 'table' ) .
 1142+ Html::input( 'wpLoginToken', $token, 'hidden' ) .
 1143+ Html::closeElement( 'form' );
11291144 return $this->getDomDocumentNodeByTagName( $form, 'form' );
11301145 }
11311146

Status & tagging log