Page MenuHomePhabricator

PermissionsError should be instantiatable with PermissionStatus returned by Authority
Closed, ResolvedPublic

Description

PermissionError constructor now takes an array of permission errors. Instead it should be able to take the PermissionStatus from Authority directly.

Might want to base it on https://gerrit.wikimedia.org/r/c/mediawiki/core/+/666239/4/includes/Permissions/PermissionStatus.php for PermissionStatus::toLegacyErrorArray

Event Timeline

Pchelolo renamed this task from PermissionError should be instantiatable with StatusValue returned by Authority to PermissionError should be instantiatable with PermissionStatus returned by Authority.Jan 14 2021, 8:11 PM
Pchelolo renamed this task from PermissionError should be instantiatable with PermissionStatus returned by Authority to PermissionsError should be instantiatable with PermissionStatus returned by Authority.
Pchelolo assigned this task to Peter.ovchyn.
Pchelolo raised the priority of this task from Low to High.
Pchelolo added a subscriber: Peter.ovchyn.

We need this to convert special pages to Authority.

Change 667693 had a related patch set uploaded (by Cicalese; owner: Cicalese):
[mediawiki/core@master] Allow PermissionError to be constructed with a PermissionStatus

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

Change 667693 merged by jenkins-bot:
[mediawiki/core@master] Allow PermissionError to be constructed with a PermissionStatus

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

Pchelolo reopened this task as Open.
Pchelolo closed this task as Resolved.