[Pidgin] #12016: XMPP message error stanzas only displayed if they have a <body/>

Pidgin trac at pidgin.im
Tue May 25 10:45:55 EDT 2010


#12016: XMPP message error stanzas only displayed if they have a <body/>
-----------------------------+----------------------------------------------
 Reporter:  Kev (from XMPP)  |        Owner:  deryni
     Type:  defect           |       Status:  new   
Milestone:                   |    Component:  XMPP  
  Version:  2.7.0            |   Resolution:        
 Keywords:                   |  
-----------------------------+----------------------------------------------
Description changed by darkrain42:

Old description:

> XMPP message errors are only displayed if they have a `<body/>`
>
> {{{
> static void handle_error(JabberMessage *jm)
> {
>         char *buf;
>
>         if(!jm->body)
>                 return;
> ...
> }}}
>
> Someone reported to Kev that Pidgin wasn't displaying message errors in
> this case.  My guess is that this is a by-product of the day when all
> errors had the full stanza contents echoed back along with an <error/>
> sub-element.

New description:

 XMPP message errors are only displayed if they have a `<body/>`

 {{{
 static void handle_error(JabberMessage *jm)
 {
         char *buf;

         if(!jm->body)
                 return;
 ...
 }}}

 Someone reported to Kev that Pidgin wasn't displaying message errors
 without the `<body/>`.  My guess is that this is a by-product of the day
 when all errors had the full stanza contents echoed back along with an
 `<error/>` sub-element.

--

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


More information about the Tracker mailing list