From d9139aa0b8065feb1dfac2860b745882db7eb080 Mon Sep 17 00:00:00 2001 From: Petr Pchelko Date: Tue, 18 Oct 2016 16:32:07 -0700 Subject: [PATCH] ApiPurge: Remove setTriggeringUser call. This is a revert of 84ffa5d1242. At that point we've been implementing page-properties-change event for the EventBus extension and thought adding this call would be a good idea. However, it was discovered afterwards that there's much more extensions that don't set the triggering user. Since that call is deprecated and plans are to remove it - revert the change. Change-Id: I9891da628f968970247edc37cccd4a666950273b --- includes/api/ApiPurge.php | 5 ----- 1 file changed, 5 deletions(-) diff --git a/includes/api/ApiPurge.php b/includes/api/ApiPurge.php index 8be523e919..8bbd88dfec 100644 --- a/includes/api/ApiPurge.php +++ b/includes/api/ApiPurge.php @@ -103,11 +103,6 @@ class ApiPurge extends ApiBase { $updates = $content->getSecondaryDataUpdates( $title, null, $forceRecursiveLinkUpdate, $p_result ); foreach ( $updates as $update ) { - # Some extensions, like EventBus, need to know the user - # that performed the purge action, so set it here - if ( $update instanceof LinksUpdate ) { - $update->setTriggeringUser( $user ); - } DeferredUpdates::addUpdate( $update, DeferredUpdates::PRESEND ); } -- 2.20.1