------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
http://bugs.kde.org/show_bug.cgi?id=121507
ogoffart kde org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
Resolution| |FIXED
------- Additional Comments From ogoffart kde org 2006-03-10 19:10 -------
SVN commit 517351 by ogoffart:
Fix bug 121507: Some users shown offline althought they are online.
We need to send our presence info after having _received_ the roster (and not
only asked for it). because some clients like ejabberd can't assure the order
of messages.
BUG: 121507
M +8 -8 jabberaccount.cpp
--- branches/kopete/0.12/kopete/protocols/jabber/jabberaccount.cpp
#517350:517351
@ -589,14 +589,6 @
kdDebug (JABBER_DEBUG_GLOBAL) << k_funcinfo << "Requesting roster..."
<< endl;
m_jabberClient->requestRoster ();
-
- /* Since we are online now, set initial presence. Don't do this
- * before the roster request or we will receive presence
- * information before we have updated our roster with actual
- * contacts from the server! (Iris won't forward presence
- * information in that case either). */
- kdDebug (JABBER_DEBUG_GLOBAL) << k_funcinfo << "Setting initial
presence..." << endl;
- setPresence ( m_initialPresence );
}
void JabberAccount::slotRosterRequestFinished ( bool success )
@ -609,6 +601,14 @
contactPool()->cleanUp ();
}
+ /* Since we are online now, set initial presence. Don't do this
+ * before the roster request or we will receive presence
+ * information before we have updated our roster with actual
+ * contacts from the server! (Iris won't forward presence
+ * information in that case either). */
+ kdDebug (JABBER_DEBUG_GLOBAL) << k_funcinfo << "Setting initial
presence..." << endl;
+ setPresence ( m_initialPresence );
+
}
void JabberAccount::slotIncomingFileTransfer ()
_______________________________________________
kopete-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kopete-devel