Page MenuHomePhabricator

When users encounter a policy exception for a non-view capability with a custom policy, inline the policy rules
ClosedPublic

Authored by epriestley on Sep 12 2019, 4:30 PM.
Tags
None
Referenced Files
F15549718: D20807.id.diff
Sun, Apr 27, 7:31 AM
F15548753: D20807.id.diff
Sun, Apr 27, 1:52 AM
F15548307: D20807.diff
Sat, Apr 26, 10:52 PM
F15483932: D20807.id49603.diff
Wed, Apr 9, 3:22 PM
F15464569: D20807.id49603.diff
Wed, Apr 2, 10:48 AM
F15457246: D20807.diff
Sun, Mar 30, 3:10 PM
F15444244: D20807.id.diff
Mar 27 2025, 8:42 AM
F15442904: D20807.diff
Mar 27 2025, 1:11 AM
Subscribers
None

Details

Summary

Fixes T13411. This looks like the last case where you hit a policy explanation and have permission to see the policy, but we don't currently show you the policy rules.

This implementation is slightly clumsy, but likely harmless.

Test Plan

Screen Shot 2019-09-12 at 9.28.22 AM.png (1×2 px, 357 KB)

Diff Detail

Repository
rP Phabricator
Branch
policy7
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/policy/filter/PhabricatorPolicyFilter.php:656XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 23428
Build 32190: Run Core Tests
Build 32189: arc lint + arc unit