From cc1f72fc747a44228626fbe55c3ffd07e06979eb Mon Sep 17 00:00:00 2001
From: Aryeh Gregor <simetrical@users.mediawiki.org>
Date: Thu, 8 May 2008 16:19:01 +0000
Subject: [PATCH] As Brion pointed out, this param is a boolean, and the
 existing code only works kind of by chance.

---
 includes/SpecialMovepage.php | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/includes/SpecialMovepage.php b/includes/SpecialMovepage.php
index 6c5232cec7..33f043d699 100644
--- a/includes/SpecialMovepage.php
+++ b/includes/SpecialMovepage.php
@@ -144,8 +144,6 @@ class MovePageForm {
 			$wgOut->addHTML( $errMsg );
 		}
 
-		$moveTalkChecked = $this->moveTalk ? ' checked="checked"' : '';
-
 		$wgOut->addHTML(
 			 Xml::openElement( 'form', array( 'method' => 'post', 'action' => $titleObj->getLocalURL( 'action=submit' ), 'id' => 'movepage' ) ) .
 			 Xml::openElement( 'fieldset' ) .
@@ -183,7 +181,7 @@ class MovePageForm {
 				<tr>
 					<td></td>
 					<td class='mw-input'>" .
-						Xml::checkLabel( wfMsg( 'movetalk' ), 'wpMovetalk', 'wpMovetalk', $moveTalkChecked ) .
+						Xml::checkLabel( wfMsg( 'movetalk' ), 'wpMovetalk', 'wpMovetalk', $this->moveTalk ) .
 					"</td>
 				</tr>"
 			);
-- 
2.20.1