ApiPurge: Remove setTriggeringUser call.
authorPetr Pchelko <ppchelko@wikimedia.org>
Tue, 18 Oct 2016 23:32:07 +0000 (16:32 -0700)
committerLegoktm <legoktm.wikipedia@gmail.com>
Wed, 19 Oct 2016 03:37:27 +0000 (03:37 +0000)
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

index 8be523e..8bbd88d 100644 (file)
@@ -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 );
                                                }