jenkins-bot has submitted this change and it was merged.

Change subject: ApiPurge: Remove setTriggeringUser call.
......................................................................


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
---
M includes/api/ApiPurge.php
1 file changed, 0 insertions(+), 5 deletions(-)

Approvals:
  Legoktm: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/api/ApiPurge.php b/includes/api/ApiPurge.php
index 8be523e..8bbd88d 100644
--- a/includes/api/ApiPurge.php
+++ b/includes/api/ApiPurge.php
@@ -103,11 +103,6 @@
                                                $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 );
                                                }
 

-- 
To view, visit https://gerrit.wikimedia.org/r/316720
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9891da628f968970247edc37cccd4a666950273b
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Ppchelko <ppche...@wikimedia.org>
Gerrit-Reviewer: Anomie <bjor...@wikimedia.org>
Gerrit-Reviewer: Legoktm <legoktm.wikipe...@gmail.com>
Gerrit-Reviewer: Mobrovac <mobro...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to