Siebrand has submitted this change and it was merged. Change subject: Fixes to avoid double counting ......................................................................
Fixes to avoid double counting Two issues: * I had method to recursively expand aggregate groups, but it also included the aggregate groups themselves! * The groups were expanded, but then the list of groups was thrown away and the direct subgroups were iterated instead Change-Id: Ida5b44302fcdfb7ec365c53e26be5511cd7adc04 --- M utils/MessageGroupStats.php 1 file changed, 11 insertions(+), 8 deletions(-) Approvals: Siebrand: Verified; Looks good to me, approved -- To view, visit https://gerrit.wikimedia.org/r/20278 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ida5b44302fcdfb7ec365c53e26be5511cd7adc04 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Translate Gerrit-Branch: master Gerrit-Owner: Nikerabbit <niklas.laxst...@gmail.com> Gerrit-Reviewer: Siebrand <siebr...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
