Fix RoomDirectory to join by alias whenever possible.

This commit is contained in:
David Baker 2016-06-09 16:41:01 +01:00
parent c4b7571c45
commit f61cfbc542

View File

@ -81,20 +81,30 @@ module.exports = React.createClass({
// }); // });
}, },
showRoom: function(roomId) { showRoom: function(roomIdOrAlias) {
// extract the metadata from the publicRooms structure to pass // extract the metadata from the publicRooms structure to pass
// as out-of-band data to view_room, because we get information // as out-of-band data to view_room, because we get information
// here that we can't get other than by joining the room in some // here that we can't get other than by joining the room in some
// cases. // cases.
var room; var room;
if (roomIdOrAlias[0] == '!') {
for (var i = 0; i < this.state.publicRooms.length; ++i) { for (var i = 0; i < this.state.publicRooms.length; ++i) {
if (this.state.publicRooms[i].room_id == roomId) { if (this.state.publicRooms[i].room_id == roomIdOrAlias) {
room = this.state.publicRooms[i]; room = this.state.publicRooms[i];
break; break;
} }
} }
}
var oob_data = {}; var oob_data = {};
if (room) { if (room) {
// pluck the alias out of the room data and use it to join the room, as we cannot
// really join rooms by ID (the HS has no way to get candidate servers). However,
// we still have to do this for room in the public room list that don't have an alias
// since this is currently the only choice.
// (Note we don't just pass the room alias to this function: we still want to be able to
// look up the oob data for which we need the room id).
var alias = room.canonical_alias || (room.aliases ? room.aliases[0] : undefined);
if (alias) roomIdOrAlias = alias;
if (MatrixClientPeg.get().isGuest()) { if (MatrixClientPeg.get().isGuest()) {
if (!room.world_readable && !room.guest_can_join) { if (!room.world_readable && !room.guest_can_join) {
var NeedToRegisterDialog = sdk.getComponent("dialogs.NeedToRegisterDialog"); var NeedToRegisterDialog = sdk.getComponent("dialogs.NeedToRegisterDialog");
@ -114,9 +124,12 @@ module.exports = React.createClass({
}; };
} }
// XXX: this interface needs to change to officially accept room IDs
// or aliases, rather than it happening to work if you pass an alias
// as room_id
dis.dispatch({ dis.dispatch({
action: 'view_room', action: 'view_room',
room_id: roomId, room_id: roomIdOrAlias,
oob_data: oob_data, oob_data: oob_data,
}); });
}, },