From 4e98a815c55f3a911a1e8c16f24fd5207628ae07 Mon Sep 17 00:00:00 2001 From: Raimond Spekking Date: Wed, 7 Jan 2009 20:34:37 +0000 Subject: [PATCH] * (bug 16921) Add a maxlength for the reason field of the page move form Changed from 'textarea' to 'text' because 'textarea' does not support maxlength. Makes it consistent with all other reason fields. --- RELEASE-NOTES | 1 + includes/specials/SpecialMovepage.php | 5 +++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/RELEASE-NOTES b/RELEASE-NOTES index 12e9c771e4..adc893d1ee 100644 --- a/RELEASE-NOTES +++ b/RELEASE-NOTES @@ -474,6 +474,7 @@ The following extensions are migrated into MediaWiki 1.14: that STDIN can be used for page list * (bug 16560) Special:Random returns a page from ContentNamespaces, and no longer from NS_MAIN +* (bug 16921) Add a maxlength for the reason field of the page move form === API changes in 1.14 === diff --git a/includes/specials/SpecialMovepage.php b/includes/specials/SpecialMovepage.php index acc27625dc..1c7fcbcd45 100644 --- a/includes/specials/SpecialMovepage.php +++ b/includes/specials/SpecialMovepage.php @@ -184,7 +184,7 @@ class MovePageForm { Xml::label( wfMsg( 'newtitle' ), 'wpNewTitle' ) . " " . - Xml::input( 'wpNewTitle', 40, $newTitle->getPrefixedText(), array( 'type' => 'text', 'id' => 'wpNewTitle' ) ) . + Xml::input( 'wpNewTitle', 60, $newTitle->getPrefixedText(), array( 'type' => 'text', 'id' => 'wpNewTitle' ) ) . Xml::hidden( 'wpOldTitle', $this->oldTitle->getPrefixedText() ) . " @@ -193,7 +193,8 @@ class MovePageForm { Xml::label( wfMsg( 'movereason' ), 'wpReason' ) . " " . - Xml::tags( 'textarea', array( 'name' => 'wpReason', 'id' => 'wpReason', 'cols' => 60, 'rows' => 2 ), htmlspecialchars( $this->reason ) ) . + Xml::input( 'wpReason', 60, $this->reason, array( 'type' => 'text', + 'id' => 'wpReason', 'maxlength' => 200 ) ) . " " ); -- 2.20.1