Page MenuHomePhabricator

OutputPage::permissionRequired() should suggest groups with the needed permission
Closed, ResolvedPublic

Description

$wgOut->sysopRequired(); need to be rewrote/replaced to show a message saying
that the user is lacking "soandso" right.


Version: unspecified
Severity: enhancement

Details

Commits
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Reference
bz769

Related Objects

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Event Timeline

bzimport raised the priority of this task from to Normal.Nov 21 2014, 7:00 PM
bzimport added a project: MediaWiki-Interface.
bzimport set Reference to bz769.
hashar created this task.Oct 24 2004, 2:19 AM
brion added a comment.Nov 29 2005, 9:52 AM

There is $wgOut->permissionRequired() now, though it's not always used.

However the permissions are internal to the code and configuration; they are not
otherwise exposed to users. Knowing which permission is required doesn't necessarily
help know what user groups would provide it. It might be enhanced to display this...

rotemliss wrote:

Added in r15994. Now, the function scans $wgGroupPermissions, finds the group
which has the permission, and shows it. The permissions themselves are hidden.

ayg wrote:

Show all groups, not just one

Why not show all groups instead of just the first encountered? (Patch tested
on local install, appears not to explode.)

Attached:

rotemliss wrote:

Applied the patch to r15997.

epriestley added a commit: Unknown Object (Diffusion Commit).Mar 4 2015, 8:22 AM
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).
epriestley added a commit: Unknown Object (Diffusion Commit).