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
F19506238: D7292.id16445.diff
Jan 9 2026, 4:42 PM
F19134429: D7292.diff
Dec 10 2025, 8:11 AM
F19066212: D7292.id.diff
Nov 30 2025, 4:54 AM
F19063292: D7292.diff
Nov 29 2025, 6:16 PM
F18848483: D7292.id.diff
Oct 30 2025, 4:56 AM
F18846453: D7292.diff
Oct 29 2025, 6:57 PM
F18715765: D7292.id16445.diff
Sep 29 2025, 10:43 AM
F18684548: D7292.diff
Sep 26 2025, 9:41 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.