Bug 27255

I discussed with smcv about the tp_mixin_message_sent, our general conclusion was that the function should be called as soon as the CM determines it's theoretically possible to send the message (So in salut call it basically right away, the only reason to call it with an error is when the contact is offline)..  Patches for the tp-glib doc and the spec to clarify this are appreciated :)

The text channel gets the id from the stanza's id, but afaik salut doesn't set the id on messages?

+  token = g_strdup_printf ("%d", g_random_int ());

That's not really random enough :) Although that really is a dummy value. Probably best to grab the id generation function for gabble (which uses libuuid optionally) and use that for the id/token generation in salut as well.

Updated per your review comments.

Patch to clarify tp-glib documentation: http://bugs.freedesktop.org/show_bug.cgi?id=27281
Patch to clarify tp-spec specification: http://bugs.freedesktop.org/show_bug.cgi?id=27282

Updated once again per your review comments in #telepathy.

Even with the updated delivery reports, it doesn't quite act the same as before porting. Before porting, if a user was offline and you'd send them a message, it wouldn't display the message normally, it would just say there was an error sending it (because they were offline). After porting, it shows the message normally, then says there was an error sending it (also because they were offline). This is due to tp_message_mixin_sent being sent earlier than its equivalent before, and because it would send sent or error, now it sends both.

Not that it's a big deal, just thought it would be good to mention.

That's not unfixable, but i don't think it's worth the trouble atm. please merge :)

Merged.