[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