Merge pull request #272 from matrix-org/rav/fix_roomname

Make sure that we update the room name
pull/21833/head
Richard van der Hoff 2016-04-17 18:12:18 +01:00
commit 336d91dadb
1 changed files with 16 additions and 2 deletions

View File

@ -53,6 +53,13 @@ module.exports = React.createClass({
componentDidMount: function() { componentDidMount: function() {
var cli = MatrixClientPeg.get(); var cli = MatrixClientPeg.get();
cli.on("RoomState.events", this._onRoomStateEvents); cli.on("RoomState.events", this._onRoomStateEvents);
// When a room name occurs, RoomState.events is fired *before*
// room.name is updated. So we have to listen to Room.name as well as
// RoomState.events.
if (this.props.room) {
this.props.room.on("Room.name", this._onRoomNameChange);
}
}, },
componentDidUpdate: function() { componentDidUpdate: function() {
@ -62,6 +69,9 @@ module.exports = React.createClass({
}, },
componentWillUnmount: function() { componentWillUnmount: function() {
if (this.props.room) {
this.props.room.removeListener("Room.name", this._onRoomNameChange);
}
var cli = MatrixClientPeg.get(); var cli = MatrixClientPeg.get();
if (cli) { if (cli) {
cli.removeListener("RoomState.events", this._onRoomStateEvents); cli.removeListener("RoomState.events", this._onRoomStateEvents);
@ -77,6 +87,10 @@ module.exports = React.createClass({
this.forceUpdate(); this.forceUpdate();
}, },
_onRoomNameChange: function(room) {
this.forceUpdate();
},
onAvatarPickerClick: function(ev) { onAvatarPickerClick: function(ev) {
if (this.refs.file_label) { if (this.refs.file_label) {
this.refs.file_label.click(); this.refs.file_label.click();
@ -98,7 +112,7 @@ module.exports = React.createClass({
description: "Failed to set avatar. " + errMsg description: "Failed to set avatar. " + errMsg
}); });
}).done(); }).done();
}, },
/** /**
* After editing the settings, get the new name for the room * After editing the settings, get the new name for the room
@ -272,7 +286,7 @@ module.exports = React.createClass({
var right_row; var right_row;
if (!this.props.editing) { if (!this.props.editing) {
right_row = right_row =
<div className="mx_RoomHeader_rightRow"> <div className="mx_RoomHeader_rightRow">
{ forget_button } { forget_button }
{ leave_button } { leave_button }