PatrolLogFormatter: Tag curid and previd as numbers for API
authorBrad Jorsch <bjorsch@wikimedia.org>
Sat, 6 Jun 2015 12:16:28 +0000 (08:16 -0400)
committerUmherirrender <umherirrender_de.wp@web.de>
Sat, 6 Jun 2015 18:28:06 +0000 (18:28 +0000)
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

index 0033743..7fe0143 100644 (file)
@@ -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 ) {