Paths

Table of Contentst

Differential D17740

Move room preferences into own controller
ClosedPublic
Actions

Authored by chad on Apr 19 2017, 11:23 PM.

Tags

None

Referenced Files

	F15421045: D17740.id42674.diff
	Fri, Mar 21, 8:35 PM

	F15416691: D17740.id42671.diff
	Thu, Mar 20, 12:59 PM

	F15416539: D17740.id42673.diff
	Thu, Mar 20, 11:53 AM

	F15412040: D17740.id.diff
	Wed, Mar 19, 10:59 AM

	F15407717: D17740.diff
	Tue, Mar 18, 7:08 PM

	F15398758: D17740.diff
	Mon, Mar 17, 1:32 AM

	F15394876: D17740.diff
	Sun, Mar 16, 1:47 AM

	F15346705: D17740.id42673.diff
	Mon, Mar 10, 2:03 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T12591: Dismantle ConpherenceUpdateController into more logical controllers

Commits

rP6969e9389f23: Move room preferences into own controller

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

Branch

move-conpherence-preferences (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 16608
Build 22146: Run Core Tests
Build 22145: arc lint + arc unit

Event Timeline

chad created this revision.Apr 19 2017, 11:23 PM

Herald added a subscriber: Korvin. · View Herald TranscriptApr 19 2017, 11:23 PM

Harbormaster completed remote builds in B16606: Diff 42671.Apr 19 2017, 11:24 PM

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

update per comments

Harbormaster completed remote builds in B16608: Diff 42673.Apr 19 2017, 11:46 PM

Closed by commit rP6969e9389f23: Move room preferences into own controller (authored by chad, committed by chad). · Explain WhyApr 19 2017, 11:48 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

applications/

conpherence/

application/

PhabricatorConpherenceApplication.php

2 lines

constants/

ConpherenceUpdateActions.php

1 line

controller/

ConpherenceController.php

14 lines

ConpherenceRoomPreferencesController.php

104 lines

ConpherenceUpdateController.php

106 lines

storage/

ConpherenceThread.php

4 lines

Diff 42673

src/__phutil_library_map__.php

Loading...

src/applications/conpherence/application/PhabricatorConpherenceApplication.php

Loading...

src/applications/conpherence/constants/ConpherenceUpdateActions.php

Loading...

src/applications/conpherence/controller/ConpherenceController.php

Loading...

src/applications/conpherence/controller/ConpherenceRoomPreferencesController.php

Loading...

src/applications/conpherence/controller/ConpherenceUpdateController.php

Loading...

src/applications/conpherence/storage/ConpherenceThread.php

Loading...