In the SmartFoxClient.as in the joinRoom function, why not to add:
Code: Select all
this.onJoinRoomError('Requested room to join does not exist!')
right after the trace:
Code: Select all
trace("SmartFoxError: requested room to join does not exist!")
I have seen some posts where developers complain about the missing error. And this one seems to give the best answers:
viewtopic.php?t=5630&highlight=onjoinroomerror
But still, I do not get why only the trace to the ide?
This is no hack-attack prevention right?
And I see Lapo's point
The RoomList should be the only source from which the client can decide where to go.
But right now I have the following case:
any user can make a "private game", a room which should not be shown in the interface's room list. Then the user can say to his friend "buddy, I am waiting you in the room /MyGameName/". His friend can hit a "joinPrivateRoom" button and enter the "MyGameName" string and try to join the hidden room. Everything is alright until you try to join non exsisting room.
I couldn't find any other useful and easy solution than modifying the SmartFoxClient.as itself with this extra line.
I would patch it for myself on every new SmartFoxClient.as update if I have to. And if my case is so rare.
But isn't it better this error event to be fired?
I really cannot understand what is the benefit of not firing it?
~