[Pidgin] #1061: Authorization signals

Pidgin trac at pidgin.im
Sun Sep 16 06:06:30 EDT 2007


#1061: Authorization signals
------------------------+---------------------------------------------------
  Reporter:  cockroach  |       Owner:  sadrul       
      Type:  patch      |      Status:  new          
  Priority:  minor      |   Milestone:  2.3.0        
 Component:  libpurple  |     Version:  2.0          
Resolution:             |    Keywords:  authorization
   Pending:  0          |  
------------------------+---------------------------------------------------
Changes (by sadrul):

  * owner:  seanegan => sadrul
  * status:  assigned => new
  * milestone:  => 2.3.0

Comment:

 Looking at the latest patch (Aug 21), I have the following comments:

  * I think 'account-authorization-requested' is better namespaced than
 'account-request-authorization'.
  * Core signal should be emitted from the core.
   * The '-requested' signal should be emitted from
 purple_account_request_authorization.
   * PurpleAccountRequestInfo should remember the 'auth_cb' and 'deny_cb'.
 uiops->request_authorization will pass two other functions in account.c
 which will call info->auth_cb/deny_cb and emit the signal.

 It'd be grand if you could update the patch accordingly.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/1061#comment:4>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list