jenkins-bot has submitted this change and it was merged.
Change subject: Track the number of entities requested using wbgetentities
......................................................................
Track the number of entities requested using wbgetentities
Change-Id: I57cf70478a4f0b5dbadb1c4a2bd05596b4988db6
---
M repo/includes/Api/GetEntities.php
1 file changed, 3 insertions(+), 0 deletions(-)
Approvals:
Hoo man: Looks good to me, approved
jenkins-bot: Verified
Objections:
Thiemo Mättig (WMDE): There's a problem with this change, please improve
diff --git a/repo/includes/Api/GetEntities.php
b/repo/includes/Api/GetEntities.php
index b3aebaf..ecf43e3 100644
--- a/repo/includes/Api/GetEntities.php
+++ b/repo/includes/Api/GetEntities.php
@@ -121,6 +121,9 @@
$resolveRedirects = $params['redirects'] === 'yes';
$entityIds = $this->getEntityIdsFromParams( $params );
+
+ $this->getStats()->updateCount(
'wikibase.repo.api.getentities.entities', count( $entityIds ) );
+
$entityRevisions = $this->getEntityRevisionsFromEntityIds(
$entityIds, $resolveRedirects );
foreach ( $entityRevisions as $sourceEntityId =>
$entityRevision ) {
--
To view, visit https://gerrit.wikimedia.org/r/278756
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I57cf70478a4f0b5dbadb1c4a2bd05596b4988db6
Gerrit-PatchSet: 4
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Addshore <[email protected]>
Gerrit-Reviewer: Hoo man <[email protected]>
Gerrit-Reviewer: Thiemo Mättig (WMDE) <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
