[Pidgin] #12519: Patch for "pounce when my status is..."

Pidgin trac at pidgin.im
Thu Aug 26 23:14:18 EDT 2010


#12519: Patch for "pounce when my status is..."
--------------------------------------------+-------------------------------
 Reporter:  grigoryj                        |        Owner:  darkrain42
     Type:  patch                           |       Status:  new       
Milestone:  Patches Needing Improvement     |    Component:  libpurple 
  Version:  2.7.3                           |   Resolution:            
 Keywords:  pidgin libpurple pounce status  |  
--------------------------------------------+-------------------------------

Comment(by darkrain42):

 * Typo (the backtick):
 +               xmlnode_set_attrib(child, "value",
 `purple_primitive_get_id_from_type(purple_pounce_get_status(pounce)));

   * With regard to the generic fields, I honestly just don't see the
 utility in it currently (I'm not quite sure what my complaint about
 genericness was.  Why not make the XML representation of the status
 something like:
 {{{
 <only-when>
     <status name='away'/>
     <status name='available'/>
 </only-when>
 }}}
 Which I find clearer (even if we don't end up supporting multiple
 statuses).

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


More information about the Tracker mailing list