From db60af7ea57ce7bd637dbf475aa86ad164fd57e6 Mon Sep 17 00:00:00 2001
From: Chad Little <chad@phacility.com>
Date: Wed, 19 Apr 2017 14:18:43 -0700
Subject: [PATCH] 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
---
 .../conpherence/controller/ConpherenceUpdateController.php  | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/applications/conpherence/controller/ConpherenceUpdateController.php b/src/applications/conpherence/controller/ConpherenceUpdateController.php
index 1d17134a53..51f40fef92 100644
--- a/src/applications/conpherence/controller/ConpherenceUpdateController.php
+++ b/src/applications/conpherence/controller/ConpherenceUpdateController.php
@@ -120,8 +120,10 @@ final class ConpherenceUpdateController
           if (!$participant) {
             return id(new Aphront404Response());
           }
-          $participant->setSettings(array('notifications' => $notifications));
-          $participant->setSettings(array('sounds' => $sounds));
+          $participant->setSettings(array(
+            'notifications' => $notifications,
+            'sounds' => $sounds,
+          ));
           $participant->save();
           return id(new AphrontRedirectResponse())
             ->setURI('/'.$conpherence->getMonogram());