Page MenuHomePhabricator

isAllowed hook
Closed, InvalidPublic

Description

I've been looking at the code surrounding security, and I'm wondering why the userCan hook isn't called from isAllowed since most of the calls made to determine whether an action can be performed call userCan, yet all of them call isAllowed. Even if userCan is best where it is, wouldn't it be a good idea to have an isAllowed hook?


Version: unspecified
Severity: enhancement

Details

Reference
bz11609

Event Timeline

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

(sorry I meant most *dont* call userCan)

Silly me, there's no need for such a hook since the result it returns is determined by the global array which can be adjusted. Sorry :-/