* (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.

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 12e9c77..adc893d 100644 (file)
--- 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 acc2762..1c7fcbc 100644 (file)
--- a/includes/specials/SpecialMovepage.php
+++ b/includes/specials/SpecialMovepage.php
@@ -184,7 +184,7 @@ class MovePageForm {
                                        Xml::label( wfMsg( 'newtitle' ), 'wpNewTitle' ) .
                                "</td>
                                <td class='mw-input'>" .
-                                       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() ) .
                                "</td>
                        </tr>
@@ -193,7 +193,8 @@ class MovePageForm {
                                        Xml::label( wfMsg( 'movereason' ), 'wpReason' ) .
                                "</td>
                                <td class='mw-input'>" .
-                                       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 ) ) .
                                "</td>
                        </tr>"
                );