[Pidgin] #3319: MSN group name display support

Pidgin trac at pidgin.im
Tue Dec 2 04:24:55 EST 2008


#3319: MSN group name display support
-----------------------+----------------------------------------------------
 Reporter:  manphiz    |        Owner:  khc
     Type:  patch      |       Status:  new
Milestone:             |    Component:  MSN
  Version:  2.2.0      |   Resolution:     
 Keywords:  msn group  |  
-----------------------+----------------------------------------------------

Comment(by SuperMMX):

 Replying to [comment:38 felipec]:
 > Temporary hacks that mess with libpurple core are bad. If I was a
 libpurple dev I wouldn't merge your patch until it's done properly.

 Does my patch mess libpurple ? I don't think so. It can be removed if
 there is a better solution available. (There is, but it will change a lot
 of interfaces, so it does not apply here).

 > Two group chats, with different people, the same buddy is in both. Or
 you have a group chat with your buddy there, and also IM.

 This doesn't matter at all. The group member can set different nick in
 different group with some command (/nn in xiaoi).

 In all my comments, the *buddy* means the bot that forwards messages
 between group members. While the group member or message sender stands for
 the real person that actually sends messages, who may not in the buddy
 list.

 > I wrote the code to display that debugging message, I didn't think
 adding the passport was necessary, so what?
 >
 > Look at what's sent over the wire:
 > http://www.hypothetic.org/docs/msn/switchboard/messages.php

 I still didn't see the passport of the *original message sender* in the
 group.

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


More information about the Tracker mailing list