From eda82d60c68beb89189f2828edea9c5de696082d Mon Sep 17 00:00:00 2001 From: "This, that and the other" Date: Fri, 29 Jan 2016 10:52:37 +1100 Subject: [PATCH] Title::newFromText: Cast integers to strings This is the cause of the T76305 debug log entries relating to SpecialExport and Echo. Bug: T76305 Bug: T116034 Change-Id: I64d629d31be79c4b4702a4298bce68fd544df6e8 --- includes/Title.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/includes/Title.php b/includes/Title.php index e549037fac..882b7dda78 100644 --- a/includes/Title.php +++ b/includes/Title.php @@ -246,7 +246,7 @@ class Title { * Create a new Title from text, such as what one would find in a link. De- * codes any HTML entities in the text. * - * @param string|null $text The link text; spaces, prefixes, and an + * @param string|int|null $text The link text; spaces, prefixes, and an * initial ':' indicating the main namespace are accepted. * @param int $defaultNamespace The namespace to use if none is specified * by a prefix. If you want to force a specific namespace even if @@ -259,7 +259,8 @@ class Title { if ( is_object( $text ) ) { throw new InvalidArgumentException( '$text must be a string.' ); } - if ( $text !== null && !is_string( $text ) ) { + // DWIM: Integers can be passed in here when page titles are used as array keys. + if ( $text !== null && !is_string( $text ) && !is_int( $text ) ) { wfDebugLog( 'T76305', wfGetAllCallers( 5 ) ); return null; } @@ -268,7 +269,7 @@ class Title { } try { - return Title::newFromTextThrow( $text, $defaultNamespace ); + return Title::newFromTextThrow( strval( $text ), $defaultNamespace ); } catch ( MalformedTitleException $ex ) { return null; } -- 2.20.1