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

	F13284504: D17740.diff
	Mon, Jun 3, 5:44 AM

	F13283335: D17740.diff
	Sun, Jun 2, 2:06 PM

	F13269695: D17740.diff
	Wed, May 29, 6:49 AM

	F13269527: D17740.diff
	Wed, May 29, 6:25 AM

	F13262867: D17740.id42673.diff
	Mon, May 27, 4:33 AM

	F13254919: D17740.diff
	Sat, May 25, 3:55 AM

	F13234728: D17740.diff
	Tue, May 21, 3:41 AM

	F13212685: D17740.diff
	Fri, May 17, 6:58 AM

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...