Merge "Fix wrong @return type hints in Language::tsTo… methods"
[lhc/web/wiklou.git] / includes / htmlform / HTMLForm.php
index bf46e55..1cc41f7 100644 (file)
  *                             Some field types support multi-level arrays.
  *    'options-messages'    -- associative array mapping message keys to values.
  *                             Some field types support multi-level arrays.
- *    'options-message'     -- message key to be parsed to extract the list of
+ *    'options-message'     -- message key or object to be parsed to extract the list of
  *                             options (like 'ipbreason-dropdown').
- *    'label-message'       -- message key for a message to use as the label.
+ *    'label-message'       -- message key or object for a message to use as the label.
  *                             can be an array of msg key and then parameters to
  *                             the message.
  *    'label'               -- alternatively, a raw text message. Overridden by
  *                             label-message
  *    'help'                -- message text for a message to use as a help text.
- *    'help-message'        -- message key for a message to use as a help text.
+ *    'help-message'        -- message key or object for a message to use as a help text.
  *                             can be an array of msg key and then parameters to
  *                             the message.
  *                             Overwrites 'help-messages' and 'help'.
- *    'help-messages'       -- array of message key. As above, each item can
+ *    'help-messages'       -- array of message keys/objects. As above, each item can
  *                             be an array of msg key and then parameters.
  *                             Overwrites 'help'.
  *    'required'            -- passed through to the object, indicating that it
  *                             is a required field.
  *    'size'                -- the length of text fields
- *    'filter-callback      -- a function name to give you the chance to
+ *    'filter-callback'     -- a function name to give you the chance to
  *                             massage the inputted value before it's processed.
  *                             @see HTMLFormField::filter()
  *    'validation-callback' -- a function name to give you the chance
@@ -180,6 +180,7 @@ class HTMLForm extends ContextSource {
        protected $mSectionFooters = [];
        protected $mPost = '';
        protected $mId;
+       protected $mName;
        protected $mTableId = '';
 
        protected $mSubmitID;
@@ -294,7 +295,7 @@ class HTMLForm extends ContextSource {
                        $this->setContext( $context );
                        $this->mTitle = false; // We don't need them to set a title
                        $this->mMessagePrefix = $messagePrefix;
-               } elseif ( is_null( $context ) && $messagePrefix !== '' ) {
+               } elseif ( $context === null && $messagePrefix !== '' ) {
                        $this->mMessagePrefix = $messagePrefix;
                } elseif ( is_string( $context ) && $messagePrefix === '' ) {
                        // B/C since 1.18
@@ -319,7 +320,7 @@ class HTMLForm extends ContextSource {
                                ? $info['section']
                                : '';
 
-                       if ( isset( $info['type'] ) && $info['type'] == 'file' ) {
+                       if ( isset( $info['type'] ) && $info['type'] === 'file' ) {
                                $this->mUseMultipart = true;
                        }
 
@@ -359,14 +360,14 @@ class HTMLForm extends ContextSource {
         */
        public function setDisplayFormat( $format ) {
                if (
-                       in_array( $format, $this->availableSubclassDisplayFormats ) ||
-                       in_array( $this->displayFormat, $this->availableSubclassDisplayFormats )
+                       in_array( $format, $this->availableSubclassDisplayFormats, true ) ||
+                       in_array( $this->displayFormat, $this->availableSubclassDisplayFormats, true )
                ) {
                        throw new MWException( 'Cannot change display format after creation, ' .
                                'use HTMLForm::factory() instead' );
                }
 
-               if ( !in_array( $format, $this->availableDisplayFormats ) ) {
+               if ( !in_array( $format, $this->availableDisplayFormats, true ) ) {
                        throw new MWException( 'Display format must be one of ' .
                                print_r( $this->availableDisplayFormats, true ) );
                }
@@ -458,9 +459,7 @@ class HTMLForm extends ContextSource {
                # @todo This will throw a fatal error whenever someone try to use
                # 'class' to feed a CSS class instead of 'cssclass'. Would be
                # great to avoid the fatal error and show a nice error.
-               $obj = new $class( $descriptor );
-
-               return $obj;
+               return new $class( $descriptor );
        }
 
        /**
@@ -472,10 +471,10 @@ class HTMLForm extends ContextSource {
         * @throws MWException
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function prepareForm() {
+       public function prepareForm() {
                # Check if we have the info we need
                if ( !$this->mTitle instanceof Title && $this->mTitle !== false ) {
-                       throw new MWException( "You must call setTitle() on an HTMLForm" );
+                       throw new MWException( 'You must call setTitle() on an HTMLForm' );
                }
 
                # Load data from the request.
@@ -488,15 +487,15 @@ class HTMLForm extends ContextSource {
         * Try submitting, with edit token check first
         * @return Status|bool
         */
-       function tryAuthorizedSubmit() {
+       public function tryAuthorizedSubmit() {
                $result = false;
 
                $submit = false;
-               if ( $this->getMethod() != 'post' ) {
+               if ( $this->getMethod() !== 'post' ) {
                        $submit = true; // no session check needed
                } elseif ( $this->getRequest()->wasPosted() ) {
                        $editToken = $this->getRequest()->getVal( 'wpEditToken' );
-                       if ( $this->getUser()->isLoggedIn() || $editToken != null ) {
+                       if ( $this->getUser()->isLoggedIn() || $editToken !== null ) {
                                // Session tokens for logged-out users have no security value.
                                // However, if the user gave one, check it in order to give a nice
                                // "session expired" error instead of "permission denied" or such.
@@ -520,7 +519,7 @@ class HTMLForm extends ContextSource {
         * errors
         * @return bool|Status Whether submission was successful.
         */
-       function show() {
+       public function show() {
                $this->prepareForm();
 
                $result = $this->tryAuthorizedSubmit();
@@ -538,7 +537,7 @@ class HTMLForm extends ContextSource {
         * added to the output, no matter, if the validation was good or not.
         * @return bool|Status Whether submission was successful.
         */
-       function showAlways() {
+       public function showAlways() {
                $this->prepareForm();
 
                $result = $this->tryAuthorizedSubmit();
@@ -559,7 +558,7 @@ class HTMLForm extends ContextSource {
         *       object, an HTML string, or an array of arrays (message keys and
         *       params) or strings (message keys)
         */
-       function trySubmit() {
+       public function trySubmit() {
                $valid = true;
                $hoistedErrors = [];
                $hoistedErrors[] = isset( $this->mValidationErrorMessage )
@@ -630,7 +629,7 @@ class HTMLForm extends ContextSource {
         * @since 1.23
         * @return bool
         */
-       function wasSubmitted() {
+       public function wasSubmitted() {
                return $this->mWasSubmitted;
        }
 
@@ -644,7 +643,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setSubmitCallback( $cb ) {
+       public function setSubmitCallback( $cb ) {
                $this->mSubmitCallback = $cb;
 
                return $this;
@@ -658,7 +657,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setValidationErrorMessage( $msg ) {
+       public function setValidationErrorMessage( $msg ) {
                $this->mValidationErrorMessage = $msg;
 
                return $this;
@@ -671,7 +670,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setIntro( $msg ) {
+       public function setIntro( $msg ) {
                $this->setPreText( $msg );
 
                return $this;
@@ -685,7 +684,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setPreText( $msg ) {
+       public function setPreText( $msg ) {
                $this->mPre = $msg;
 
                return $this;
@@ -698,7 +697,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function addPreText( $msg ) {
+       public function addPreText( $msg ) {
                $this->mPre .= $msg;
 
                return $this;
@@ -712,8 +711,8 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function addHeaderText( $msg, $section = null ) {
-               if ( is_null( $section ) ) {
+       public function addHeaderText( $msg, $section = null ) {
+               if ( $section === null ) {
                        $this->mHeader .= $msg;
                } else {
                        if ( !isset( $this->mSectionHeaders[$section] ) ) {
@@ -734,8 +733,8 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setHeaderText( $msg, $section = null ) {
-               if ( is_null( $section ) ) {
+       public function setHeaderText( $msg, $section = null ) {
+               if ( $section === null ) {
                        $this->mHeader = $msg;
                } else {
                        $this->mSectionHeaders[$section] = $msg;
@@ -751,8 +750,8 @@ class HTMLForm extends ContextSource {
         * @since 1.26
         * @return string HTML
         */
-       function getHeaderText( $section = null ) {
-               if ( is_null( $section ) ) {
+       public function getHeaderText( $section = null ) {
+               if ( $section === null ) {
                        return $this->mHeader;
                } else {
                        return isset( $this->mSectionHeaders[$section] ) ? $this->mSectionHeaders[$section] : '';
@@ -767,8 +766,8 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function addFooterText( $msg, $section = null ) {
-               if ( is_null( $section ) ) {
+       public function addFooterText( $msg, $section = null ) {
+               if ( $section === null ) {
                        $this->mFooter .= $msg;
                } else {
                        if ( !isset( $this->mSectionFooters[$section] ) ) {
@@ -789,8 +788,8 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setFooterText( $msg, $section = null ) {
-               if ( is_null( $section ) ) {
+       public function setFooterText( $msg, $section = null ) {
+               if ( $section === null ) {
                        $this->mFooter = $msg;
                } else {
                        $this->mSectionFooters[$section] = $msg;
@@ -806,8 +805,8 @@ class HTMLForm extends ContextSource {
         * @since 1.26
         * @return string
         */
-       function getFooterText( $section = null ) {
-               if ( is_null( $section ) ) {
+       public function getFooterText( $section = null ) {
+               if ( $section === null ) {
                        return $this->mFooter;
                } else {
                        return isset( $this->mSectionFooters[$section] ) ? $this->mSectionFooters[$section] : '';
@@ -821,7 +820,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function addPostText( $msg ) {
+       public function addPostText( $msg ) {
                $this->mPost .= $msg;
 
                return $this;
@@ -834,7 +833,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setPostText( $msg ) {
+       public function setPostText( $msg ) {
                $this->mPost = $msg;
 
                return $this;
@@ -849,7 +848,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       public function addHiddenField( $name, $value, $attribs = [] ) {
+       public function addHiddenField( $name, $value, array $attribs = [] ) {
                $attribs += [ 'name' => $name ];
                $this->mHiddenFields[] = [ $value, $attribs ];
 
@@ -955,7 +954,7 @@ class HTMLForm extends ContextSource {
         *
         * @return void Nothing, should be last call
         */
-       function displayForm( $submitResult ) {
+       public function displayForm( $submitResult ) {
                $this->getOutput()->addHTML( $this->getHTML( $submitResult ) );
        }
 
@@ -966,7 +965,7 @@ class HTMLForm extends ContextSource {
         *
         * @return string HTML
         */
-       function getHTML( $submitResult ) {
+       public function getHTML( $submitResult ) {
                # For good measure (it is the default)
                $this->getOutput()->preventClickjacking();
                $this->getOutput()->addModules( 'mediawiki.htmlform' );
@@ -1000,12 +999,15 @@ class HTMLForm extends ContextSource {
                        'method' => $this->getMethod(),
                        'enctype' => $encType,
                ];
-               if ( !empty( $this->mId ) ) {
+               if ( $this->mId ) {
                        $attribs['id'] = $this->mId;
                }
-               if ( !empty( $this->mAutocomplete ) ) {
+               if ( $this->mAutocomplete ) {
                        $attribs['autocomplete'] = $this->mAutocomplete;
                }
+               if ( $this->mName ) {
+                       $attribs['name'] = $this->mName;
+               }
                return $attribs;
        }
 
@@ -1016,7 +1018,7 @@ class HTMLForm extends ContextSource {
         *
         * @return string Wrapped HTML.
         */
-       function wrapForm( $html ) {
+       public function wrapForm( $html ) {
                # Include a <fieldset> wrapper for style, if requested.
                if ( $this->mWrapperLegend !== false ) {
                        $legend = is_string( $this->mWrapperLegend ) ? $this->mWrapperLegend : false;
@@ -1034,9 +1036,9 @@ class HTMLForm extends ContextSource {
         * Get the hidden fields that should go inside the form.
         * @return string HTML.
         */
-       function getHiddenFields() {
+       public function getHiddenFields() {
                $html = '';
-               if ( $this->getMethod() == 'post' ) {
+               if ( $this->getMethod() === 'post' ) {
                        $html .= Html::hidden(
                                'wpEditToken',
                                $this->getUser()->getEditToken( $this->mTokenSalt ),
@@ -1046,7 +1048,7 @@ class HTMLForm extends ContextSource {
                }
 
                $articlePath = $this->getConfig()->get( 'ArticlePath' );
-               if ( strpos( $articlePath, '?' ) !== false && $this->getMethod() == 'get' ) {
+               if ( strpos( $articlePath, '?' ) !== false && $this->getMethod() === 'get' ) {
                        $html .= Html::hidden( 'title', $this->getTitle()->getPrefixedText() ) . "\n";
                }
 
@@ -1062,7 +1064,7 @@ class HTMLForm extends ContextSource {
         * Get the submit and (potentially) reset buttons.
         * @return string HTML.
         */
-       function getButtons() {
+       public function getButtons() {
                $buttons = '';
                $useMediaWikiUIEverywhere = $this->getConfig()->get( 'UseMediaWikiUIEverywhere' );
 
@@ -1085,9 +1087,9 @@ class HTMLForm extends ContextSource {
 
                        if ( $useMediaWikiUIEverywhere ) {
                                foreach ( $this->mSubmitFlags as $flag ) {
-                                       array_push( $attribs['class'], 'mw-ui-' . $flag );
+                                       $attribs['class'][] = 'mw-ui-' . $flag;
                                }
-                               array_push( $attribs['class'], 'mw-ui-button' );
+                               $attribs['class'][] = 'mw-ui-button';
                        }
 
                        $buttons .= Xml::submitButton( $this->getSubmitText(), $attribs ) . "\n";
@@ -1099,7 +1101,7 @@ class HTMLForm extends ContextSource {
                                [
                                        'type' => 'reset',
                                        'value' => $this->msg( 'htmlform-reset' )->text(),
-                                       'class' => ( $useMediaWikiUIEverywhere ? 'mw-ui-button' : null ),
+                                       'class' => $useMediaWikiUIEverywhere ? 'mw-ui-button' : null,
                                ]
                        ) . "\n";
                }
@@ -1154,7 +1156,7 @@ class HTMLForm extends ContextSource {
         * Get the whole body of the form.
         * @return string
         */
-       function getBody() {
+       public function getBody() {
                return $this->displaySection( $this->mFieldTree, $this->mTableId );
        }
 
@@ -1165,7 +1167,7 @@ class HTMLForm extends ContextSource {
         *
         * @return string
         */
-       function getErrors( $errors ) {
+       public function getErrors( $errors ) {
                if ( $errors instanceof Status ) {
                        if ( $errors->isOK() ) {
                                $errorstr = '';
@@ -1220,7 +1222,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setSubmitText( $t ) {
+       public function setSubmitText( $t ) {
                $this->mSubmitText = $t;
 
                return $this;
@@ -1263,10 +1265,8 @@ class HTMLForm extends ContextSource {
         * Get the text for the submit button, either customised or a default.
         * @return string
         */
-       function getSubmitText() {
-               return $this->mSubmitText
-                       ? $this->mSubmitText
-                       : $this->msg( 'htmlform-submit' )->text();
+       public function getSubmitText() {
+               return $this->mSubmitText ?: $this->msg( 'htmlform-submit' )->text();
        }
 
        /**
@@ -1299,7 +1299,7 @@ class HTMLForm extends ContextSource {
         * @todo FIXME: Integrity of $t is *not* validated
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setSubmitID( $t ) {
+       public function setSubmitID( $t ) {
                $this->mSubmitID = $t;
 
                return $this;
@@ -1315,7 +1315,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls
         */
-       function suppressDefaultSubmit( $suppressSubmit = true ) {
+       public function suppressDefaultSubmit( $suppressSubmit = true ) {
                $this->mShowSubmit = !$suppressSubmit;
 
                return $this;
@@ -1347,6 +1347,16 @@ class HTMLForm extends ContextSource {
                return $this;
        }
 
+       /**
+        * @param string $name 'name' attribute for the form
+        * @return HTMLForm $this for chaining calls
+        */
+       public function setName( $name ) {
+               $this->mName = $name;
+
+               return $this;
+       }
+
        /**
         * Prompt the whole form to be wrapped in a "<fieldset>", with
         * this text as its "<legend>" element.
@@ -1391,7 +1401,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setMessagePrefix( $p ) {
+       public function setMessagePrefix( $p ) {
                $this->mMessagePrefix = $p;
 
                return $this;
@@ -1404,7 +1414,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function setTitle( $t ) {
+       public function setTitle( $t ) {
                $this->mTitle = $t;
 
                return $this;
@@ -1414,7 +1424,7 @@ class HTMLForm extends ContextSource {
         * Get the title
         * @return Title
         */
-       function getTitle() {
+       public function getTitle() {
                return $this->mTitle === false
                        ? $this->getContext()->getTitle()
                        : $this->mTitle;
@@ -1477,7 +1487,7 @@ class HTMLForm extends ContextSource {
 
                // Conveniently, PHP method names are case-insensitive.
                // For grep: this can call getDiv, getRaw, getInline, getVForm, getOOUI
-               $getFieldHtmlMethod = $displayFormat == 'table' ? 'getTableRow' : ( 'get' . $displayFormat );
+               $getFieldHtmlMethod = $displayFormat === 'table' ? 'getTableRow' : ( 'get' . $displayFormat );
 
                foreach ( $fields as $key => $value ) {
                        if ( $value instanceof HTMLFormField ) {
@@ -1493,7 +1503,7 @@ class HTMLForm extends ContextSource {
                                        $html[] = $retval;
 
                                        $labelValue = trim( $value->getLabel() );
-                                       if ( $labelValue != '&#160;' && $labelValue !== '' ) {
+                                       if ( $labelValue !== '&#160;' && $labelValue !== '' ) {
                                                $hasLabel = true;
                                        }
 
@@ -1586,7 +1596,7 @@ class HTMLForm extends ContextSource {
        /**
         * Construct the form fields from the Descriptor array
         */
-       function loadData() {
+       public function loadData() {
                $fieldData = [];
 
                foreach ( $this->mFlatFields as $fieldname => $field ) {
@@ -1615,7 +1625,7 @@ class HTMLForm extends ContextSource {
         *
         * @return HTMLForm $this for chaining calls (since 1.20)
         */
-       function suppressReset( $suppressReset = true ) {
+       public function suppressReset( $suppressReset = true ) {
                $this->mShowReset = !$suppressReset;
 
                return $this;
@@ -1630,7 +1640,7 @@ class HTMLForm extends ContextSource {
         *
         * @return array
         */
-       function filterDataForSubmit( $data ) {
+       public function filterDataForSubmit( $data ) {
                return $data;
        }