Page MenuHomePhabricator

Conpherence - add storage for view / edit / join policy
ClosedPublic

Authored by btrahan on Mar 17 2015, 11:10 PM.
Tags
None
Referenced Files
F19074131: D12103.id29120.diff
Dec 1 2025, 4:29 AM
F18898796: D12103.id.diff
Nov 7 2025, 7:12 PM
F18879513: D12103.diff
Nov 6 2025, 6:31 PM
F18827505: D12103.id29119.diff
Oct 24 2025, 10:32 AM
F18753496: D12103.id.diff
Oct 4 2025, 7:07 PM
F18749209: D12103.id29114.diff
Oct 4 2025, 2:52 AM
F18748248: D12103.diff
Oct 3 2025, 8:06 PM
F18658759: D12103.diff
Sep 23 2025, 8:13 AM
Subscribers

Details

Summary

Ref T7582. Also adds the basic logic for "rooms" implementation. Also makes sure we use the initializeNewThread method as appropriate.

Test Plan

made a new conpherence and it worked!

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Conpherence - add storage for view / edit / join policy.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/conpherence/storage/ConpherenceThread.php
35

This might need to be 1 explicitly in SQL strict mode.

237–239

Oh, I think there might be a special case here: we should probably still give you view permission automatically if you're a member of a room, even if it technically violates the view policy. That is, return false for edit/join, but drop down to the participant check for view.

A use case is "make the view policy 'no one', then invite people you like". This is also consistent with Projects, which have a similar rule (you can always view projects you're a member of).

This revision is now accepted and ready to land.Mar 17 2015, 11:15 PM
btrahan edited edge metadata.

changes as requested

This revision was automatically updated to reflect the committed changes.