Page MenuHomePhabricator

Autopromotion system should be extended to allow removal of granted groups etc.
Closed, ResolvedPublic

Description

Author: llampak

Description:
This report is related to bug 24948, originally submitted by Aryeh Gregor, but that one is marked as concerning FlaggedRevs, so I'm making another report to be found by a core developer.

Currently all autopromoted groups are implicit, cannot be removed via Special:UserRights and the autopromotion is checked on each call of User::getEffectiveGroups() rather than on certain events.

It's clearly not enough for FlaggedRevs which had to develop their own autopromotion system which bypasses the above limitations.

The point is that the core autopromotion system should be upgraded so that FlaggedRevs could use it.

In the original bug report I have attached a patch which I'm also attaching here (for a newer revision). The patch is described at comment 1 for bug 24948. I don't know if it's ok (probably it's not) so don't hesitate to write everything from scratch if you prefer.


Version: 1.17.x
Severity: enhancement

Details

Reference
bz25428

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:12 PM
bzimport set Reference to bz25428.
bzimport added a subscriber: Unknown Object (MLST).

llampak wrote:

the forgotten patch

Attached:

  • This bug has been marked as a duplicate of bug 15702 ***