ulith/webxoss-fixed
1
0
Fork 0
forked from mirrors/webxoss-core
Code Activity

fix bugs of renaming room

Browse source 
recovery previous room name when fail to rename room
add `onblur` listener to #room-name element
This commit is contained in:
deardrops 2017-05-23 18:30:53 +08:00
parent 4b384f1b9d
commit 37782768f7
3 changed files with 13 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
.gitignore vendored
View file

@ -2,4 +2,5 @@ node_modules
trash trash
.DS_Store .DS_Store
*.sublime-* *.sublime-*
yarn.lock yarn.lock
.vscode/

10
RoomManager.js
View file

@ -218,16 +218,26 @@ RoomManager.prototype.renameRoom = function (client,cfg) {
} }
var oldRoomName = client.room.name; var oldRoomName = client.room.name;
var newRoomName = cfg.roomName; var newRoomName = cfg.roomName;
if (newRoomName === oldRoomName) {
return;
}
if (!errMsg) { if (!errMsg) {
errMsg = this.checkRoomName(newRoomName); errMsg = this.checkRoomName(newRoomName);
} }
var room; var room;
if (!errMsg) {
if (newRoomName in this.roomMap) {
errMsg = 'ROOM_ALREADY_EXISTS';
client.room.update();
}
}
if (!errMsg) { if (!errMsg) {
room = this.roomMap[oldRoomName]; room = this.roomMap[oldRoomName];
if (!room) { if (!room) {
errMsg = 'ROOM_DOES_NOT_EXIST'; errMsg = 'ROOM_DOES_NOT_EXIST';
} else if (client.getPosition() !== 'host') { } else if (client.getPosition() !== 'host') {
errMsg = 'YOU_ARE_NOT_ROOM_HOST'; errMsg = 'YOU_ARE_NOT_ROOM_HOST';
client.room.update();
} }
} }
if (errMsg) { if (errMsg) {

2
webxoss-client

@ -1 +1 @@
Subproject commit 51a710f84c2931e55ca71009ad469468b5662863 Subproject commit 36727dd62e19545ca9a0abcc25124e7858c788aa