Skip to content

Commit db60af7

Browse files
author
Chad Little
committed
Set all room settings at once
Summary: Sets notification and sound preferences in a single array() Test Plan: Change email preference, save, set sound preference, save. Email preference still OK. Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Differential Revision: https://secure.phabricator.com/D17735
1 parent 51485a1 commit db60af7

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/applications/conpherence/controller/ConpherenceUpdateController.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,10 @@ public function handleRequest(AphrontRequest $request) {
120120
if (!$participant) {
121121
return id(new Aphront404Response());
122122
}
123-
$participant->setSettings(array('notifications' => $notifications));
124-
$participant->setSettings(array('sounds' => $sounds));
123+
$participant->setSettings(array(
124+
'notifications' => $notifications,
125+
'sounds' => $sounds,
126+
));
125127
$participant->save();
126128
return id(new AphrontRedirectResponse())
127129
->setURI('/'.$conpherence->getMonogram());

0 commit comments

Comments
 (0)