Pwirth has uploaded a new change for review.

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

Change subject: BSUserManager: Fixed RemoveGroups calculation
......................................................................

BSUserManager: Fixed RemoveGroups calculation

* Replaced getEffectiveGroups() with getGroups()
=> Needs Cherry-Pick to REL1_23

Change-Id: I2cbb749b8271bbe12f90c1b156eff89205dad544
---
M UserManager/UserManager.class.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceExtensions 
refs/changes/97/304197/1

diff --git a/UserManager/UserManager.class.php 
b/UserManager/UserManager.class.php
index 2c98580..cb55c67 100644
--- a/UserManager/UserManager.class.php
+++ b/UserManager/UserManager.class.php
@@ -277,7 +277,7 @@
                        return Status::newFatal( 
'bs-usermanager-no-self-desysop' );
                }
 
-               $aCurrentGroups = $oUser->getEffectiveGroups();
+               $aCurrentGroups = $oUser->getGroups();
                $aSetGroups = array_diff( $aGroups, $aCurrentGroups );
                $aRemoveGroups = array_diff( $aCurrentGroups, $aGroups );
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2cbb749b8271bbe12f90c1b156eff89205dad544
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceExtensions
Gerrit-Branch: master
Gerrit-Owner: Pwirth <wi...@hallowelt.biz>

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

Reply via email to