Skip to content

Commit

Permalink
Remove /# as its unecessary cause it was fixed in socketio/socket.io#…
Browse files Browse the repository at this point in the history
  • Loading branch information
craighillwood committed Jun 13, 2018
1 parent 26254fc commit b9e5511
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions client.js
Original file line number Diff line number Diff line change
Expand Up @@ -181,9 +181,7 @@ p.getRoomData = function() {

p.setIsInitialized = function() {

// added in /# because it seems that socket.io-client adds /# to the id
// where on the backend socket.id does not have /#
var room = this.getRoom().setVar( 'pv_init/#' + this.io.id, true );
var room = this.getRoom().setVar( 'pv_init' + this.io.id, true );
};

p.play = function() {
Expand Down Expand Up @@ -269,7 +267,7 @@ function checkAllInitialized() {

users.forEach( function( user ) {

isAllInited = Boolean(data[ 'pv_init/#' + user ]) && isAllInited;
isAllInited = Boolean(data[ 'pv_init' + user ]) && isAllInited;
});

if( this.isAllInitialized !== isAllInited ) {
Expand Down

0 comments on commit b9e5511

Please sign in to comment.