Last modified: 2014-09-23 23:59:16 UTC
Created attachment 5202 [details] Proposed patch Proposed patch for this functionality is attached. In brief, there's a new interface for managing group rights on-wiki, which stores the data in the database.
Created attachment 5203 [details] Revised. Oops, forgot to include a changein that diff.
Created attachment 5204 [details] Revised SVN diff from root, instead of from includes.
Brion's comments 23:30 < brion> Werdna: consider splitting some of the code out from both User and SpecialGroupRights to create a proper group-rights-manager backend class. this should encapsulate all the caching and database queries/updates, so User and the Special:GroupRights front-end are clients of the backend class 23:31 < brion> otherwise the basic theory of it looks reasonably sane ;) 23:32 < brion> i also didn't look into too much detail on the UI frontend yet; how is it about indicating default settings, and perhaps reverting to defaults if you've messed something up?
Created attachment 5277 [details] Revised (big) patch This patch is a total rewrite of the permissions system. It still needs to have cross-wiki user rights reimplemented in the new framework, but other than that, it allows current behaviour, plus implementing group rights being changed on-wiki.
Now in /branches/rightsmanager.
Cross-wiki user rights have been implemented! There's some strange problem with logging, but I'm not sure if anybody else can reproduce it. Other than that, it's pretty much ready to be reviewed.
Functionality implemented through Configure extension, and as such this patch is unlikely to be reviewed and applied. Leaving open pending a decision on whether extension functionality is sufficient to close this bug as FIXED.
*** Bug 19352 has been marked as a duplicate of this bug. ***
*Bulk BZ Change: +Patch to open bugs with patches attached that are missing the keyword*
Comment on attachment 5277 [details] Revised (big) patch "as such this patch is unlikely to be reviewed and applied." hence obsoleting the patch and removing the patch keyword.