Comment 2 for bug 925702

Revision history for this message
John S. Gruber (jsjgruber) wrote :

They should be ignored.

The code tries to append an error message to the chatroom stream, and fails because the error message is appended without specifying a sender, which defaults to none. There's code above the line above that handles the sender==None case, but it isn't handled here, so there is a problem. The code in this function should probably be reviewed for any other sender==None cases.