Anomie has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/216411

Change subject: PatrolLogFormatter: Tag curid and previd as numbers for API
......................................................................

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
---
M includes/logging/PatrolLogFormatter.php
1 file changed, 4 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/11/216411/1

diff --git a/includes/logging/PatrolLogFormatter.php 
b/includes/logging/PatrolLogFormatter.php
index 0033743..7fe0143 100644
--- a/includes/logging/PatrolLogFormatter.php
+++ b/includes/logging/PatrolLogFormatter.php
@@ -68,9 +68,11 @@
                $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 ) {

-- 
To view, visit https://gerrit.wikimedia.org/r/216411
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6d3e8fd2653ce810eec59b94bf75063e5fb6bcab
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Anomie <bjor...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to