Merge "Make HTMLCheckField::loadDataFromRequest always return a boolean"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Sat, 23 Apr 2016 17:01:32 +0000 (17:01 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Sat, 23 Apr 2016 17:01:32 +0000 (17:01 +0000)
includes/htmlform/HTMLCheckField.php

index a59b15e..4a6b804 100644 (file)
@@ -111,7 +111,7 @@ class HTMLCheckField extends HTMLFormField {
        /**
         * @param WebRequest $request
         *
-        * @return string
+        * @return bool
         */
        function loadDataFromRequest( $request ) {
                $invert = isset( $this->mParams['invert'] ) && $this->mParams['invert'];
@@ -125,7 +125,7 @@ class HTMLCheckField extends HTMLFormField {
                                ? !$request->getBool( $this->mName )
                                : $request->getBool( $this->mName );
                } else {
-                       return $this->getDefault();
+                       return (bool)$this->getDefault();
                }
        }
 }