From a95d8b5ed6f320a77e2ae743b7bcf56548d65aa9 Mon Sep 17 00:00:00 2001 From: David Baker Date: Tue, 14 Jun 2016 14:38:45 +0100 Subject: [PATCH] Move init logic into componentWillMount to simplify getInitialState --- src/components/structures/RoomView.js | 25 +++++++++++++------------ 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/src/components/structures/RoomView.js b/src/components/structures/RoomView.js index 4a3d3e7a31..2ff7ed1cbd 100644 --- a/src/components/structures/RoomView.js +++ b/src/components/structures/RoomView.js @@ -102,23 +102,17 @@ module.exports = React.createClass({ }, getInitialState: function() { - var room; - var room_id; - if (this.props.roomAddress[0] == '!') { - room_id = this.props.roomAddress; - room = MatrixClientPeg.get().getRoom(room_id); - } return { - room: room, - roomId: room_id, - roomLoading: !room, + room: null, + roomId: null, + roomLoading: true, editingRoomSettings: false, uploadingRoomSettings: false, numUnreadMessages: 0, draggingFile: false, searching: false, searchResults: null, - hasUnsentMessages: this._hasUnsentMessages(room), + hasUnsentMessages: false, callState: null, guestsCanJoin: false, canPeek: false, @@ -164,15 +158,22 @@ module.exports = React.createClass({ room: room, roomId: result.room_id, roomLoading: !room, + hasUnsentMessages: this._hasUnsentMessages(room), }, this.updatePeeking); }, (err) => { this.setState({ roomLoading: false, }); }); + } else { + var room = MatrixClientPeg.get().getRoom(this.props.roomAddress); + this.setState({ + roomId: this.props.roomAddress, + room: room, + roomLoading: !room, + hasUnsentMessages: this._hasUnsentMessages(room), + }, this.updatePeeking); } - - this.updatePeeking(); }, updatePeeking: function() {