Alex Monk has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/73990


Change subject: Don't trigger event when user rights change is cross-wiki
......................................................................

Don't trigger event when user rights change is cross-wiki

Bug: 51418
Change-Id: If962ed70d1c29668d676ab378916db4afb7eb14a
---
M Hooks.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo 
refs/changes/90/73990/1

diff --git a/Hooks.php b/Hooks.php
index fea04a8..737a1ca 100644
--- a/Hooks.php
+++ b/Hooks.php
@@ -526,7 +526,7 @@
        public static function onUserRights( &$user, $add, $remove ) {
                global $wgUser;
 
-               if ( !$user->isAnon() && $wgUser->getId() != $user->getId() && 
( $add || $remove ) ) {
+               if ( !( $user instanceof UserRightsProxy ) && !$user->isAnon() 
&& $wgUser->getId() != $user->getId() && ( $add || $remove ) ) {
                        EchoEvent::create(
                                array(
                                        'type' => 'user-rights',

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If962ed70d1c29668d676ab378916db4afb7eb14a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Alex Monk <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to