[Pidgin] #10633: ICQ Invisible mode set custom status text to "Invisible"

Pidgin trac at pidgin.im
Wed Jun 23 18:54:22 EDT 2010


#10633: ICQ Invisible mode set custom status text to "Invisible"
-------------------------------------+--------------------------------------
 Reporter:  marq                     |        Owner:  MarkDoliner
     Type:  defect                   |       Status:  closed     
Milestone:                           |    Component:  ICQ        
  Version:  2.6.3                    |   Resolution:  fixed      
 Keywords:  invisible custom status  |  
-------------------------------------+--------------------------------------
Changes (by ivan.komarov at soc.pidgin.im):

  * status:  new => closed
  * resolution:  => fixed


Comment:

 (In [021e6a7b4ba46230e93812c39268259f8c6bc7fe]):[[BR]]
 "Invisibility mode" improvements.

   1. Removed the code for setting the status message to the status name
      (like "Available" or "Invisible") when no message is set by user.
      This should fix #10633, since now no one will see our status
      change to "Invisible" if we go invisible with no message set.
   2. Got rid of sending superfluous packets on server.
      * There's no need to call oscar_set_extended_status() from
        oscar_set_status_icq(), because in all cases it has already
        been called before that. The only thing oscar_set_status_icq()
        does except setting the extended status is setting the
        permit/deny setting, so I renamed it accordingly.
      * No need to call oscar_set_icq_permdeny() in
        purple_ssi_parseerr() (it will be called in purple_ssi_parselist()
 anyway).
      * Combined setting the extended status and the status message
        (as per TODO).
      This should fix the problem when client like QIP see us as Invisible
      for some (quite long) time before we appear as Offline.

 Also, I removed the code for setting the "itmsurl" status attribute,
 as there was no way to set this iTunes Store URL from the UI.

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


More information about the Tracker mailing list