Page MenuHomePhabricator

Move room preferences into own controller
ClosedPublic

Authored by chad on Apr 19 2017, 11:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 6, 3:04 AM
Unknown Object (File)
Fri, Oct 4, 3:08 AM
Unknown Object (File)
Thu, Oct 3, 9:14 PM
Unknown Object (File)
Aug 25 2024, 8:13 AM
Unknown Object (File)
Aug 23 2024, 9:42 PM
Unknown Object (File)
Aug 19 2024, 3:34 AM
Unknown Object (File)
Aug 14 2024, 7:05 PM
Unknown Object (File)
Aug 14 2024, 1:43 AM
Subscribers

Details

Summary

Ref T12591. These preferences are per user and we need to reload the whole UI on a change anyways. Simplifies future expansion.

Test Plan

Option click into new window, set preferences. View in Conpherence, see saved settings, change sound. Hear new sound.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

v. nice

src/applications/conpherence/controller/ConpherenceRoomPreferencesController.php
33

This can be simplified slightly to $this->newDialog() in modern code.

Probably needs a addCancelButton()?

38

(Consider adding getURI() to Thread if it doesn't already exist?)

This revision is now accepted and ready to land.Apr 19 2017, 11:25 PM
chad marked an inline comment as done.
  • update per comments
This revision was automatically updated to reflect the committed changes.