From: Chad Horohoe <chadh@wikimedia.org>
Date: Thu, 12 Mar 2015 18:53:21 +0000 (-0700)
Subject: Pass if log entry is legacy to ApiQueryLogEvents::addLogParams()
X-Git-Tag: 1.31.0-rc.0~12125
X-Git-Url: http://git.cyclocoop.org/%7B%24www_url%7Dadmin/compta/operations/?a=commitdiff_plain;h=cc2eb92668cd731594fa7b3025768c533c7846f4;p=lhc%2Fweb%2Fwiklou.git

Pass if log entry is legacy to ApiQueryLogEvents::addLogParams()

Querying via ApiQueryRecentChanges and ApiQueryWatchlist did not
pass if a particular log entry was a legacy log entry.

Ultimately CheckUser should stop writing legacy style log entries
too but we can at least behave properly when we hit them.

Bug: T91466
Change-Id: I1564ec7e599e2f9d78775e63e06eca84db497a43
---

diff --git a/includes/api/ApiQueryRecentChanges.php b/includes/api/ApiQueryRecentChanges.php
index 14820340ed..aa222641db 100644
--- a/includes/api/ApiQueryRecentChanges.php
+++ b/includes/api/ApiQueryRecentChanges.php
@@ -542,7 +542,8 @@ class ApiQueryRecentChanges extends ApiQueryGeneratorBase {
 					$logEntry->getParameters(),
 					$logEntry->getType(),
 					$logEntry->getSubtype(),
-					$logEntry->getTimestamp()
+					$logEntry->getTimestamp(),
+					$logEntry->isLegacy()
 				);
 			}
 		}
diff --git a/includes/api/ApiQueryWatchlist.php b/includes/api/ApiQueryWatchlist.php
index 11008cf883..3857a08f0f 100644
--- a/includes/api/ApiQueryWatchlist.php
+++ b/includes/api/ApiQueryWatchlist.php
@@ -419,7 +419,8 @@ class ApiQueryWatchlist extends ApiQueryGeneratorBase {
 					$logEntry->getParameters(),
 					$logEntry->getType(),
 					$logEntry->getSubtype(),
-					$logEntry->getTimestamp()
+					$logEntry->getTimestamp(),
+					$logEntry->isLegacy()
 				);
 			}
 		}