Siebrand has submitted this change and it was merged. Change subject: Reduce queries in WorkflowStatesMessageGroup ......................................................................
Reduce queries in WorkflowStatesMessageGroup With TranslateUtils::getContents we can load all definitions with one query. We can then loop over the results to see if any are missing, avoiding the queries caused by Title:: exists(). Also changes it to gather states from all groups, not just from the default configuration. The creation of missing state pages should use jobs to avoid recursion issues, but that is not done in this commit. Change-Id: If8bffc93903d5a77d93c90187c4da83c5cd347d0 --- M messagegroups/WorkflowStatesMessageGroup.php 1 file changed, 13 insertions(+), 12 deletions(-) Approvals: Siebrand: Looks good to me, approved jenkins-bot: Verified -- To view, visit https://gerrit.wikimedia.org/r/30150 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8bffc93903d5a77d93c90187c4da83c5cd347d0 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Translate Gerrit-Branch: master Gerrit-Owner: Nikerabbit <niklas.laxst...@gmail.com> Gerrit-Reviewer: Siebrand <siebr...@wikimedia.org> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
