[Pidgin] #216: Chats should not raise on auto-join

Pidgin trac at pidgin.im
Fri May 11 12:10:26 EDT 2007


#216: Chats should not raise on auto-join
---------------------------+------------------------------------------------
  Reporter:  rlaager       |       Owner:  elb  
      Type:  defect        |      Status:  new  
  Priority:  minor         |   Milestone:  2.0.1
 Component:  pidgin (gtk)  |     Version:  2.0  
Resolution:                |    Keywords:       
   Pending:  0             |  
---------------------------+------------------------------------------------
Comment (by elb):

 This is interesting ... this code was intentionally inserted for /join,
 /query, et al.  I'm not sure how I can cleanly work around this, without
 having bugs in other situations ... the problem is that ircds use so many
 nonstandard numerics to indicate an unsuccessful join.

 I can turn this off for all new irc conversations without too much
 difficulty, but I don't see a good way to distinguish between a user-
 requested new window and an auto-join window without heuristics.  If
 anyone has a suggestion of a good heuristic, I'm all ears.  (The first one
 that comes to mind is, put channels in a list on auto-join (or on /join
 /query, whichever is easier), then treat them appropriately when
 successfully joining, or remove them from that list on known failure --
 time out entries after some period of time, to take care of unknown
 failures.  This leaves a bizarre-behavior race in some circumstances
 (like, an auto-join fails (say) because of missing nickserv
 indentification, a user identifies, and then runs /join -- but the window
 doesn't pop to foreground, because we didn't recognize the server's
 nonstandard 499 numeric, and the timer hasn't fired yet), but I don't know
 what else to do.)  I think this might be a case of being unable to have
 your cake and eat it, too.

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


More information about the Tracker mailing list