r56384 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r56383‎ | r56384 | r56385 >
Date:18:37, 15 September 2009
Author:brion
Status:deferred
Tags:
Comment:
merge r56188
Modified paths:
  • /branches/wmf-deployment-work/includes/specials (modified) (history)
  • /branches/wmf-deployment-work/includes/specials/SpecialMovepage.php (modified) (history)

Diff [purge]

Index: branches/wmf-deployment-work/includes/specials/SpecialMovepage.php
@@ -17,8 +17,10 @@
1818 }
1919
2020 $target = isset( $par ) ? $par : $wgRequest->getVal( 'target' );
 21+
 22+ // Yes, the use of getVal() and getText() is wanted, see bug 20365
2123 $oldTitleText = $wgRequest->getVal( 'wpOldTitle', $target );
22 - $newTitleText = $wgRequest->getVal( 'wpNewTitle' );
 24+ $newTitleText = $wgRequest->getText( 'wpNewTitle' );
2325
2426 $oldTitle = Title::newFromText( $oldTitleText );
2527 $newTitle = Title::newFromText( $newTitleText );
@@ -87,7 +89,7 @@
8890 * OutputPage::wrapWikiMsg().
8991 */
9092 function showForm( $err ) {
91 - global $wgOut, $wgUser, $wgFixDoubleRedirects;
 93+ global $wgOut, $wgUser, $wgContLang, $wgFixDoubleRedirects;
9294
9395 $skin = $wgUser->getSkin();
9496
@@ -200,7 +202,7 @@
201203 Xml::label( wfMsg( 'newtitle' ), 'wpNewTitle' ) .
202204 "</td>
203205 <td class='mw-input'>" .
204 - Xml::input( 'wpNewTitle', 40, $newTitle->getPrefixedText(), array( 'type' => 'text', 'id' => 'wpNewTitle' ) ) .
 206+ Xml::input( 'wpNewTitle', 40, $wgContLang->recodeForEdit( $newTitle->getPrefixedText() ), array( 'type' => 'text', 'id' => 'wpNewTitle' ) ) .
205207 Xml::hidden( 'wpOldTitle', $this->oldTitle->getPrefixedText() ) .
206208 "</td>
207209 </tr>
Property changes on: branches/wmf-deployment-work/includes/specials
___________________________________________________________________
Name: svn:mergeinfo
208210 + /branches/REL1_15/phase3/includes/specials:51646
/branches/wmf-deployment/includes/specials:53381
/trunk/phase3/includes/specials:56188

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r56188Fix for r55535, per Brion's comment: convert x-coded strings in the target in...ialex17:17, 11 September 2009

Status & tagging log