Page MenuHomePhabricator
Differential D21754

Give Phame blog posts configurable interact policies, with a default policy of "Same as Blog"
ClosedPublic
Actions

Authored by epriestley on Apr 1 2022, 7:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 7, 6:54 AM
Unknown Object (File)
Sun, Apr 7, 5:52 AM
Unknown Object (File)
Sun, Apr 7, 4:23 AM
Unknown Object (File)
Sat, Apr 6, 10:53 AM
Unknown Object (File)
Tue, Apr 2, 9:24 AM
Unknown Object (File)
Sat, Mar 30, 9:20 PM
Unknown Object (File)
Sat, Mar 30, 10:20 AM
Unknown Object (File)
Feb 9 2024, 5:03 AM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13661: Give Phame configurable interact policies
Commits
rPaae23f020476: Give Phame blog posts configurable interact policies, with a default policy of…
Summary

Ref T13661. This allows posts to have comments disabled (or restricted) on a per-post basis, and makes them inherit the containing blog policy by default.

Test Plan

Locked a post by editing its policy explicitly; locked a post by editing the containing blog policy.

Diff Detail

Repository
rP Phabricator
Branch
phameint7
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/phame/policyrule/PhameInheritBlogPolicyRule.php:35XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 25638
Build 35466: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 1 2022, 7:41 PM
Owners added a subscriber: Restricted Owners Package.Apr 1 2022, 7:41 PM
epriestley requested review of this revision.Apr 1 2022, 7:41 PM
Harbormaster completed remote builds in B25638: Diff 51853.Apr 1 2022, 7:41 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 1 2022, 7:53 PM
Closed by commit rPaae23f020476: Give Phame blog posts configurable interact policies, with a default policy of… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPaae23f020476: Give Phame blog posts configurable interact policies, with a default policy of….

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
6 lines
src/
applications/
phame/
editor/
2 lines
policyrule/
51 lines
storage/
13 lines

Diff 51853

View Options

resources/sql/autopatches/20220401.phameinteract.04.postinteract.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/phame/editor/PhamePostEditor.php

Loading...
View Options

src/applications/phame/policyrule/PhameInheritBlogPolicyRule.php

Loading...
View Options

src/applications/phame/storage/PhamePost.php

Loading...
Phabricator · Built by Phacility