Catrope has uploaded a new change for review. https://gerrit.wikimedia.org/r/311832
Change subject: Fix moderation controller ...................................................................... Fix moderation controller The change to use DeferredUpdates had a missing use. Change-Id: Ied72608c9b3aa597627aa5ea069c257f506c3c1b (cherry picked from commit ffd9d4bb4d6c2c38f72a125e32ffe4a4a357ac2c) --- M includes/controller/ModerationController.php 1 file changed, 4 insertions(+), 4 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo refs/changes/32/311832/1 diff --git a/includes/controller/ModerationController.php b/includes/controller/ModerationController.php index 0c45357..27241b4 100644 --- a/includes/controller/ModerationController.php +++ b/includes/controller/ModerationController.php @@ -24,12 +24,12 @@ $affectedUserIds = $notificationMapper->fetchUsersWithNotificationsForEvents( $eventIds ); $eventMapper->toggleDeleted( $eventIds, $moderate ); - DeferredUpdates::addCallableUpdate( function () { - // This udate runs after than main transaction round commits. - // Wait for the events deletions to be propagated to replica DBs + DeferredUpdates::addCallableUpdate( function () use ( $affectedUserIds ) { + // This update runs after the main transaction round commits. + // Wait for the event deletions to be propagated to replica DBs $lbFactory = MediaWikiServices::getInstance()->getDBLoadBalancerFactory(); $lbFactory->waitForReplication( [ 'timeout' => 5 ] ); - $lbFactory->flushReplicaSnapshots( __METHOD__ ); + $lbFactory->flushReplicaSnapshots( 'EchoModerationController::moderate' ); // Recompute the notification count for the // users whose notifications have been moderated. foreach ( $affectedUserIds as $userId ) { -- To view, visit https://gerrit.wikimedia.org/r/311832 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ied72608c9b3aa597627aa5ea069c257f506c3c1b Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Echo Gerrit-Branch: wmf/1.28.0-wmf.19 Gerrit-Owner: Catrope <roan.katt...@gmail.com> Gerrit-Reviewer: Mattflaschen <mflasc...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits