SearchResult: MW version number should be a string in wfDeprecated call
authorDerick Alangi <alangiderick@gmail.com>
Tue, 14 May 2019 11:28:35 +0000 (12:28 +0100)
committerReedy <reedy@wikimedia.org>
Tue, 14 May 2019 12:36:30 +0000 (12:36 +0000)
Seems this was a typo and I think 1.32 which is a double/float will be
implicitly converted to true (bool) because it will resolve 1.32 to 1 as
integer and then 1 which maps to true (bool).

To avoid this, use '1.32' instead of the integer form of the version.

Change-Id: I2420396e110284f582cd79820ffc6064e247b4b9

includes/search/SearchResult.php

index bd19a84..f85c58f 100644 (file)
@@ -285,7 +285,7 @@ class SearchResult {
                if ( $extensionData instanceof Closure ) {
                        $this->extensionData = $extensionData;
                } elseif ( is_array( $extensionData ) ) {
-                       wfDeprecated( __METHOD__ . ' with array argument', 1.32 );
+                       wfDeprecated( __METHOD__ . ' with array argument', '1.32' );
                        $this->extensionData = function () use ( $extensionData ) {
                                return $extensionData;
                        };