jenkins-bot has submitted this change and it was merged.
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(-)
Approvals:
Umherirrender: Looks good to me, approved
jenkins-bot: Verified
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: merged
Gerrit-Change-Id: I6d3e8fd2653ce810eec59b94bf75063e5fb6bcab
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Anomie <bjor...@wikimedia.org>
Gerrit-Reviewer: Umherirrender <umherirrender_de...@web.de>
Gerrit-Reviewer: Waldir <wal...@email.com>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
