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
F18378919: D20807.id49611.diff
Thu, Aug 28, 3:03 PM
F18378765: D20807.id49603.diff
Thu, Aug 28, 2:51 PM
F18376192: D20807.diff
Thu, Aug 28, 10:53 AM
F18375552: D20807.id.diff
Thu, Aug 28, 10:06 AM
F18374759: D20807.id49611.diff
Thu, Aug 28, 9:11 AM
F18374671: D20807.diff
Thu, Aug 28, 9:06 AM
F18100703: D20807.diff
Sat, Aug 9, 12:01 PM
F17944061: D20807.id.diff
Thu, Jul 31, 9:16 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