| Index: trunk/extensions/CheckUser/CheckUser_body.php |
| — | — | @@ -163,16 +163,19 @@ |
| 164 | 164 | '|' . wfMsg( 'checkuser-showlog' ) . ']]' |
| 165 | 165 | ); |
| 166 | 166 | } |
| 167 | | - // FIXME: use more Xml/Html methods |
| 168 | | - $form = "<form name='checkuserform' id='checkuserform' action=\"$action\" method='post'>"; |
| | 167 | + |
| | 168 | + $form = Xml::openElement( 'form', array ( 'action' => $action, 'name' => 'checkuserform', 'id' => 'checkuserform', 'method' => 'post' ) ); |
| 169 | 169 | $form .= '<fieldset><legend>' . wfMsgHtml( 'checkuser-query' ) . '</legend>'; |
| 170 | | - $form .= '<table border="0" cellpadding="2"><tr>'; |
| | 170 | + $form .= Xml::openElement( 'table', array( 'border' => '0', 'cellpadding' => '2' ) ); |
| | 171 | + $form .= '<tr>'; |
| 171 | 172 | $form .= '<td>' . wfMsgHtml( 'checkuser-target' ) . '</td>'; |
| 172 | 173 | $form .= '<td>' . Xml::input( 'user', 46, $user, array( 'id' => 'checktarget' ) ); |
| 173 | 174 | $form .= ' ' . $this->getPeriodMenu( $period ) . '</td>'; |
| 174 | 175 | $form .= '</tr><tr>'; |
| 175 | | - $form .= '<td></td><td class="checkuserradios"><table border="0" cellpadding="3"><tr>'; |
| 176 | | - $form .= '<td>' . Xml::radio( 'checktype', 'subuserips', $encuserips, array( 'id' => 'subuserips' ) ); |
| | 176 | + $form .= '<td></td>'; |
| | 177 | + $form .= Xml::openElement('td', array( 'class' => 'checkuserradios' ) ); |
| | 178 | + $form .= Xml::openElement( 'table', array( 'border' => '0', 'cellpadding' => '3' ) ); |
| | 179 | + $form .= '<tr><td>' . Xml::radio( 'checktype', 'subuserips', $encuserips, array( 'id' => 'subuserips' ) ); |
| 177 | 180 | $form .= ' ' . Xml::label( wfMsg( 'checkuser-ips' ), 'subuserips' ) . '</td>'; |
| 178 | 181 | $form .= '<td>' . Xml::radio( 'checktype', 'subedits', $encedits, array( 'id' => 'subedits' ) ); |
| 179 | 182 | $form .= ' ' . Xml::label( wfMsg( 'checkuser-edits' ), 'subedits' ) . '</td>'; |