[Pidgin] #8675: Changed Buddy State Notification to reflect Mobile status
Pidgin
trac at pidgin.im
Fri May 22 02:55:25 EDT 2009
#8675: Changed Buddy State Notification to reflect Mobile status
---------------------------------------------+------------------------------
Reporter: jlintern | Owner:
Type: patch | Status: new
Milestone: 3.0.0 | Component: plugins
Version: 2.5.5 | Resolution:
Keywords: buddy state notification mobile |
---------------------------------------------+------------------------------
Changes (by darkrain42):
* milestone: Patches Needing Improvement => 3.0.0
Comment:
Oh, I see now. `purple_blist_update_buddy_status` is only passed the old
status (in this case, PURPLE_STATUS_AVAILABLE or something) and then that
function picks the current active status (which is still
PURPLE_STATUS_AVAILABLE, because mobile is independent) and emits a "hey,
the status has changed" signal.
I think the fix is to change purple_blist_update_buddy_status() to take
both the old and new status (or just the new status, possibly). That's a
3.0.0 change.
--
Ticket URL: <http://developer.pidgin.im/ticket/8675#comment:4>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list