r51493 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r51492‎ | r51493 | r51494 >
Date:07:33, 5 June 2009
Author:siebrand
Status:ok
Tags:
Comment:
Follow-up to r51084:
* address case sensitivity issue raised by Splarka on CR
* also change to edit link on Special:Block
Modified paths:
  • /trunk/phase3/includes/FileDeleteForm.php (modified) (history)
  • /trunk/phase3/includes/ProtectionForm.php (modified) (history)
  • /trunk/phase3/includes/specials/SpecialBlockip.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/ProtectionForm.php
@@ -447,7 +447,7 @@
448448 $out .= Xml::closeElement( 'fieldset' );
449449
450450 if ( $wgUser->isAllowed( 'editinterface' ) ) {
451 - $title = Title::newFromText( 'protect-dropdown', NS_MEDIAWIKI );
 451+ $title = Title::newFromText( 'Protect-dropdown', NS_MEDIAWIKI );
452452 $link = $wgUser->getSkin()->link(
453453 $title,
454454 wfMsgHtml( 'protect-edit-reasonlist' ),
Index: trunk/phase3/includes/FileDeleteForm.php
@@ -194,7 +194,7 @@
195195
196196 if ( $wgUser->isAllowed( 'editinterface' ) ) {
197197 $skin = $wgUser->getSkin();
198 - $title = Title::newFromText( 'filedelete-reason-dropdown', NS_MEDIAWIKI );
 198+ $title = Title::newFromText( 'Filedelete-reason-dropdown', NS_MEDIAWIKI );
199199 $link = $skin->link(
200200 $title,
201201 wfMsgHtml( 'filedelete-edit-reasonlist' ),
Index: trunk/phase3/includes/specials/SpecialBlockip.php
@@ -636,7 +636,13 @@
637637 $links[] = $this->getContribsLink( $skin );
638638 $links[] = $this->getUnblockLink( $skin );
639639 $links[] = $this->getBlockListLink( $skin );
640 - $links[] = $skin->makeLink ( 'MediaWiki:Ipbreason-dropdown', wfMsgHtml( 'ipb-edit-dropdown' ) );
 640+ $title = Title::newFromText( 'Ipbreason-dropdown', NS_MEDIAWIKI );
 641+ $links[] = $skin->link(
 642+ $title,
 643+ wfMsgHtml( 'ipb-edit-dropdown' ),
 644+ array(),
 645+ array( 'action' => 'edit' )
 646+ );
641647 return '<p class="mw-ipb-conveniencelinks">' . $wgLang->pipeList( $links ) . '</p>';
642648 }
643649

Follow-up revisions

RevisionCommit summaryAuthorDate
r51502Fix for r51493, r51084 and realted: use Title::makeTitle() rather than Title:...ialex13:12, 5 June 2009

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r51084Fix a usability issue on protect and [file ]delete forms. "Edit ... reasons" ...siebrand20:35, 27 May 2009

Status & tagging log