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

	Unknown Object (File)
	Thu, Apr 25, 3:27 AM

	Unknown Object (File)
	Fri, Apr 19, 6:25 PM

	Unknown Object (File)
	Fri, Apr 19, 10:14 AM

	Unknown Object (File)
	Mon, Apr 1, 5:35 PM

	Unknown Object (File)
	Mon, Apr 1, 2:33 AM

	Unknown Object (File)
	Sun, Mar 31, 9:34 AM

	Unknown Object (File)
	Sat, Mar 30, 4:33 AM

	Unknown Object (File)
	Feb 14 2024, 10:51 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 16606
Build 22142: Run Core Tests
Build 22141: 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
32	This can be simplified slightly to `$this->newDialog()` in modern code. Probably needs a `addCancelButton()`?
37	(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

105 lines

ConpherenceUpdateController.php

106 lines

Diff 42671

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