Page MenuHomePhabricator

Allow custom policies to be loaded and exeucuted by the policy filter
ClosedPublic

Authored by epriestley on Oct 13 2013, 1:21 AM.
Tags
None
Referenced Files
F22750718: D7292.id16445.diff
Sun, Oct 4, 9:40 PM
Unknown Object (File)
Aug 24 2026, 9:59 PM
Unknown Object (File)
Jul 9 2026, 6:49 PM
Unknown Object (File)
Jul 5 2026, 4:13 AM
Unknown Object (File)
Jun 26 2026, 4:33 PM
Unknown Object (File)
Jun 26 2026, 9:19 AM
Unknown Object (File)
Jun 25 2026, 10:33 AM
Unknown Object (File)
Jun 25 2026, 7:56 AM
Subscribers

Details

Summary

Ref T603. Adds code to actually execute custom policies. (There's still no way to select them in the UI.)

Test Plan
  • Added and executed unit tests.
  • Edited policies in existing applications.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Although I'm sure your review was very thorough, it might be best for @btrahan to take a look too.

btrahan added inline comments.
src/applications/policy/query/PhabricatorPolicyQuery.php
73–75

you could lose this else or lose the initialization on line 66

204

s/thought/though/

epriestley updated this revision to Unknown Object (????).Oct 14 2013, 6:46 PM
  • Remove extra initialization.
  • Fix comment typo.