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)
Tue, Apr 30, 4:55 PM
Unknown Object (File)
Tue, Apr 30, 4:55 PM
Unknown Object (File)
Tue, Apr 30, 4:55 PM
Unknown Object (File)
Tue, Apr 30, 4:55 PM
Unknown Object (File)
Mon, Apr 29, 2:24 PM
Unknown Object (File)
Wed, Apr 24, 6:40 PM
Unknown Object (File)
Mon, Apr 22, 5:15 PM
Unknown Object (File)
Apr 12 2024, 4:58 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
Branch
T7582
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4917
Build 4935: [Placeholder Plan] Wait for 30 Seconds

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.