Bug in PublicMessageRequest validation

Post here your questions about the Objective-C API for SFS2X

Moderators: Bax, Lapo, A51Integrated

Bug in PublicMessageRequest validation

Postby mmilen » 26 Oct 2011, 14:17

line 57 in GenericMessageRequest.m
-(void)validatePublicMessage:(SmartFoxiPhoneClient *)sfs errors:(NSMutableArray *)errors {
if(_message == nil || [_message length] == 0) {
[errors addObject:@"Public message is empty!"];
}

if (_room != nil && ![sfs.mySelf isJoinedInRoom:_room]) {
[errors addObject:[NSString stringWithFormat:@"You are not joined in the target Room: %@", _room]];
}
}
[/code]

in some circumstances
Code: Select all
[sfs.mySelf isJoinedInRoom:_room]
shows the client not joined, while the server has it right and ROOM isJoined property is also correct, every other message will go out just fine. Replacing this check with
Code: Select all
[_room isJoined]
]
fixes the problem and it is much faster, as it checks single boolean value vs going through isJoinedInRoom , which involves search in array.[/code]
mmilen
 
Posts: 200
Joined: 09 Nov 2010, 00:48

Postby Lapo » 01 Nov 2011, 19:54

Thanks for reporting
Lapo
--
gotoAndPlay()
...addicted to flash games
User avatar
Lapo
Site Admin
 
Posts: 15796
Joined: 21 Mar 2005, 09:50
Location: Italy


Return to SFS2X iPhone / iPad / OSX API

Who is online

Users browsing this forum: No registered users and 2 guests