[Pidgin] #547: jabber roster item exchange (XEP-0093, XEP-0144) (was: jabber rosterintems)

Pidgin trac at pidgin.im
Sat Apr 9 17:15:08 EDT 2011


#547: jabber roster item exchange (XEP-0093, XEP-0144)
-----------------------------------------+----------------------------------
 Reporter:  Emil                         |        Owner:  deryni
     Type:  patch                        |       Status:  new   
Milestone:  Patches Needing Improvement  |    Component:  XMPP  
  Version:  2.0                          |   Resolution:        
 Keywords:  jabber rosteritems           |  
-----------------------------------------+----------------------------------
Changes (by darkrain42):

  * milestone:  Patches Needing Review => Patches Needing Improvement


Comment:

 Beyond the fact this wouldn't really apply cleanly anymore, it should
 probably:
    * Detect advertised support in the receiving client
    * Utilize either of the Roster Item specifications, as supported by the
 remote client
    * Optionally allow picking a specific resource to send to (maybe?)
    * Use `purple_find_buddies(account, NULL)` instead of iterating the
 list looking for buddies whose protocol pretty name is 'XMPP' (which, as
 an aside, breaks i18n, as that string is translated in
 `purple_account_get_protocol_name`)
    * Use C-style comments (/* */) only
    * Use a better idiom than g_list_append() (wjt had an interesting one
 on his blog, but that post appears to have been taken down -- which is
 totally unhelpful for me to write here)
    * "Name" may be too vague to be translated properly (not sure)

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


More information about the Tracker mailing list