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
F15403282: D12103.diff
Tue, Mar 18, 2:35 AM
F15379459: D12103.id.diff
Thu, Mar 13, 9:18 PM
F15369989: D12103.diff
Wed, Mar 12, 8:59 AM
F15345043: D12103.id29120.diff
Mon, Mar 10, 8:16 AM
Unknown Object (File)
Fri, Feb 21, 5:48 AM
Unknown Object (File)
Fri, Feb 21, 4:42 AM
Unknown Object (File)
Fri, Feb 21, 3:38 AM
Unknown Object (File)
Feb 12 2025, 3:19 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.