Page MenuHomePhabricator
Differential D13258

Add a "Thread Members" object policy and some unit tests
ClosedPublic
Actions

Authored by epriestley on Jun 11 2015, 11:52 PM.
Tags
None
Referenced Files
F19514814: D13258.diff
Wed, Jan 14, 5:22 AM
F19364927: D13258.diff
Fri, Dec 26, 12:21 PM
F19095823: D13258.diff
Dec 4 2025, 6:21 AM
F19053681: D13258.diff
Nov 28 2025, 9:41 AM
F18943791: D13258.id.diff
Nov 11 2025, 2:41 PM
F18918597: D13258.id32081.diff
Nov 9 2025, 11:00 AM
F18838109: D13258.diff
Oct 27 2025, 9:20 AM
F18787188: D13258.diff
Oct 14 2025, 10:28 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T5681: Allow applications to provide custom policy rules
T8488: Remove "room" vs "message" distinction
Commits
Restricted Diffusion Commit
rP893c7a26c196: Add a "Thread Members" object policy and some unit tests
Summary

Ref T8488. Ref T5681.

Now you can just use id(new ConpherenceThreadMembersPolicyRule())->getObjectPolicyFullKey() as a policy.

Added tests for TaskAuthor + ThreadMembers.

Test Plan
  • Ran tests.
  • Set a thread policy to "Members of thread'.

Diff Detail

Repository
rP Phabricator
Branch
apolicy4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6700
Build 6722: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 32045.Jun 11 2015, 11:52 PM
epriestley retitled this revision from to Add a "Thread Members" object policy and some unit tests.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added tasks: T8488: Remove "room" vs "message" distinction, T5681: Allow applications to provide custom policy rules.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 11 2015, 11:52 PM
btrahan accepted this revision.Jun 12 2015, 2:26 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 12 2015, 2:26 PM
Closed by commit rP893c7a26c196: Add a "Thread Members" object policy and some unit tests (authored by epriestley, committed by epriestley). · Explain WhyJun 13 2015, 10:45 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
conpherence/
policyrule/
42 lines
policy/
__tests__/
59 lines
query/
2 lines
rule/
14 lines

Diff 32045

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/conpherence/policyrule/ConpherenceThreadMembersPolicyRule.php

Loading...
View Options

src/applications/policy/__tests__/PhabricatorPolicyDataTestCase.php

Loading...
View Options

src/applications/policy/query/PhabricatorPolicyQuery.php

Loading...
View Options

src/applications/policy/rule/PhabricatorPolicyRule.php

Loading...
Phabricator · Built by Phacility