From 2bec7ec981b9a83b31c438920b85eec7169dcf16 Mon Sep 17 00:00:00 2001
From: Erik Johnston <erikj@matrix.org>
Date: Tue, 21 Jul 2015 11:02:36 +0100
Subject: [PATCH] Add spinner while uploading state

---
 skins/base/views/organisms/RoomView.js | 64 ++++++++++++++++++++------
 src/controllers/organisms/RoomView.js  |  1 +
 2 files changed, 52 insertions(+), 13 deletions(-)

diff --git a/skins/base/views/organisms/RoomView.js b/skins/base/views/organisms/RoomView.js
index 4f19b9b3..74c3f1ee 100644
--- a/skins/base/views/organisms/RoomView.js
+++ b/skins/base/views/organisms/RoomView.js
@@ -23,6 +23,7 @@ var MatrixClientPeg = require("../../../../src/MatrixClientPeg");
 var ComponentBroker = require('../../../../src/ComponentBroker');
 var classNames = require("classnames");
 var filesize = require('filesize');
+var q = require('q');
 
 var MessageTile = ComponentBroker.get('molecules/MessageTile');
 var RoomHeader = ComponentBroker.get('molecules/RoomHeader');
@@ -44,7 +45,10 @@ module.exports = React.createClass({
     },
 
     onSaveClick: function() {
-        this.setState({editingRoomSettings: false});
+        this.setState({
+            editingRoomSettings: false,
+            uploadingRoomSettings: true,
+        });
 
         var new_name = this.refs.header.getRoomName();
         var new_topic = this.refs.room_settings.getTopic();
@@ -75,36 +79,65 @@ module.exports = React.createClass({
             old_history_visibility = "shared";
         }
 
+        var deferreds = [];
 
         if (old_name != new_name && new_name != undefined) {
-            MatrixClientPeg.get().setRoomName(this.state.room.roomId, new_name);
+            deferreds.push(
+                MatrixClientPeg.get().setRoomName(this.state.room.roomId, new_name)
+            );
         }
 
         if (old_topic != new_topic && new_topic != undefined) {
-            MatrixClientPeg.get().setRoomTopic(this.state.room.roomId, new_topic);
+            deferreds.push(
+                MatrixClientPeg.get().setRoomTopic(this.state.room.roomId, new_topic)
+            );
         }
 
         if (old_join_rule != new_join_rule && new_join_rule != undefined) {
-            MatrixClientPeg.get().sendStateEvent(
-                this.state.room.roomId, "m.room.join_rules", {
-                    join_rule: new_join_rule,
-                }, ""
+            deferreds.push(
+                MatrixClientPeg.get().sendStateEvent(
+                    this.state.room.roomId, "m.room.join_rules", {
+                        join_rule: new_join_rule,
+                    }, ""
+                )
             );
         }
 
         if (old_history_visibility != new_history_visibility && new_history_visibility != undefined) {
-            MatrixClientPeg.get().sendStateEvent(
-                this.state.room.roomId, "m.room.history_visibility", {
-                    history_visibility: new_history_visibility,
-                }, ""
+            deferreds.push(
+                MatrixClientPeg.get().sendStateEvent(
+                    this.state.room.roomId, "m.room.history_visibility", {
+                        history_visibility: new_history_visibility,
+                    }, ""
+                )
             );
         }
 
         if (new_power_levels) {
-            MatrixClientPeg.get().sendStateEvent(
-                this.state.room.roomId, "m.room.power_levels", new_power_levels, ""
+            deferreds.push(
+                MatrixClientPeg.get().sendStateEvent(
+                    this.state.room.roomId, "m.room.power_levels", new_power_levels, ""
+                )
             );
         }
+
+        console.log("deferreds " + deferreds.length);
+
+        if (deferreds.length) {
+            var self = this;
+            q.all(deferreds).fail(function(err) {
+                // TODO: Handle err
+            }).finally(function() {
+                self.setState({
+                    uploadingRoomSettings: false,
+                });
+            });
+        } else {
+            this.setState({
+                editingRoomSettings: false,
+                uploadingRoomSettings: false,
+            });
+        }
     },
 
     render: function() {
@@ -179,6 +212,11 @@ module.exports = React.createClass({
                 roomEdit = <RoomSettings ref="room_settings" room={this.state.room} />;
             }
 
+            if (this.state.uploadingRoomSettings) {
+                console.log("Uploading");
+                roomEdit = <Loader/>;
+            }
+
             return (
                 <div className="mx_RoomView">
                     <RoomHeader ref="header" room={this.state.room} editing={this.state.editingRoomSettings}
diff --git a/src/controllers/organisms/RoomView.js b/src/controllers/organisms/RoomView.js
index 4f6c4d3c..b02b2937 100644
--- a/src/controllers/organisms/RoomView.js
+++ b/src/controllers/organisms/RoomView.js
@@ -46,6 +46,7 @@ module.exports = {
             room: this.props.roomId ? MatrixClientPeg.get().getRoom(this.props.roomId) : null,
             messageCap: INITIAL_SIZE,
             editingRoomSettings: false,
+            uploadingRoomSettings: false,
         }
     },