[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