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
Unknown Object (File)
Wed, Apr 24, 6:40 PM
Unknown Object (File)
Mon, Apr 22, 5:15 PM
Unknown Object (File)
Fri, Apr 12, 4:58 AM
Unknown Object (File)
Mon, Apr 1, 10:49 PM
Unknown Object (File)
Mon, Apr 1, 3:05 PM
Unknown Object (File)
Mon, Apr 1, 9:39 AM
Unknown Object (File)
Feb 8 2024, 11:47 AM
Unknown Object (File)
Jan 20 2024, 4:42 PM
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.