r42665 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r42664‎ | r42665 | r42666 >
Date:18:27, 27 October 2008
Author:demon
Status:old
Tags:
Comment:
(bug 15806) Add html IDs to tags. This time let's follow naming standards and not put a tag on the <li>'s, but rather on its parent <ul>
Modified paths:
  • /trunk/extensions/CentralAuth/SpecialCentralAuth.php (modified) (history)

Diff [purge]

Index: trunk/extensions/CentralAuth/SpecialCentralAuth.php
@@ -245,7 +245,7 @@
246246 'registered' => $wgLang->timeanddate( $reg ) . " ($age)",
247247 'locked' => $globalUser->isLocked() ? wfMsg( 'centralauth-admin-yes' ) : wfMsg( 'centralauth-admin-no' ),
248248 'hidden' => $globalUser->isHidden() ? wfMsg( 'centralauth-admin-yes' ) : wfMsg( 'centralauth-admin-no' ) );
249 - $out = '<ul>';
 249+ $out = '<ul id="mw-centralauth-info">';
250250 foreach( $attribs as $tag => $data ) {
251251 $out .= Xml::element( 'li', array(), wfMsg( "centralauth-admin-info-$tag" ) . ' ' . $data );
252252 }
@@ -273,7 +273,7 @@
274274
275275 function listRemainder( $list ) {
276276 ksort( $list );
277 - $s = '<ul>';
 277+ $s = '<ul id="mw-centralauth-unattached">';
278278 foreach ( $list as $row ) {
279279 $s .= '<li>' . $this->foreignUserLink( $row['wiki'] ) . "</li>\n";
280280 }
@@ -288,7 +288,8 @@
289289 Xml::openElement( 'form',
290290 array(
291291 'method' => 'post',
292 - 'action' => $this->getTitle( $this->mUserName )->getLocalUrl( 'action=submit' ) ) ) .
 292+ 'action' => $this->getTitle( $this->mUserName )->getLocalUrl( 'action=submit' ),
 293+ 'id' => 'mw-centralauth-merged' ) ) .
293294 Xml::hidden( 'wpMethod', $action ) .
294295 Xml::hidden( 'wpEditToken', $wgUser->editToken() ) .
295296 '<table>' .
@@ -359,7 +360,8 @@
360361 Xml::element( 'h2', array(), wfMsg( "centralauth-admin-{$action}-title" ) ) .
361362 Xml::openElement( 'form', array(
362363 'method' => 'POST',
363 - 'action' => $this->getTitle()->getFullUrl( 'target=' . urlencode( $this->mUserName ) ) ) ) .
 364+ 'action' => $this->getTitle()->getFullUrl( 'target=' . urlencode( $this->mUserName ) ),
 365+ 'id' => "mw-centralauth-$action" ) ) .
364366 Xml::hidden( 'wpMethod', $action ) .
365367 Xml::hidden( 'wpEditToken', $wgUser->editToken() ) .
366368 wfMsgExt( "centralauth-admin-{$action}-description", 'parse' ) .

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r42264(bug 15806) Added html ids to certain tags. Patch by Pathoschild.btongminh14:21, 20 October 2008
r42431Revert r42264 "(bug 15806) Added html ids to certain tags. Patch by Pathoschi...brion23:26, 23 October 2008

Status & tagging log