From 1dc97335ae258ab7a7b29e9181d652f17c13e250 Mon Sep 17 00:00:00 2001 From: Brad Jorsch Date: Sat, 6 Jun 2015 08:16:28 -0400 Subject: [PATCH] PatrolLogFormatter: Tag curid and previd as numbers for API In the database the values might be strings, depending on the source of the RecentChanges object when the log entry was created. In getParametersForApi(), tag the vars as 'number' so they'll always be output as actual integers. Bug: T101603 Change-Id: I6d3e8fd2653ce810eec59b94bf75063e5fb6bcab --- includes/logging/PatrolLogFormatter.php | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/includes/logging/PatrolLogFormatter.php b/includes/logging/PatrolLogFormatter.php index 00337432cc..7fe0143ffb 100644 --- a/includes/logging/PatrolLogFormatter.php +++ b/includes/logging/PatrolLogFormatter.php @@ -68,9 +68,11 @@ class PatrolLogFormatter extends LogFormatter { $params = $entry->getParameters(); static $map = array( - '4::curid', - '5::previd', + '4:number:curid', + '5:number:previd', '6:bool:auto', + '4::curid' => '4:number:curid', + '5::previd' => '5:number:previd', '6::auto' => '6:bool:auto', ); foreach ( $map as $index => $key ) { -- 2.20.1