Subject: bug#3037 closed by Chong Yidong <[email-masked]> (Re: 23.0.92; rcirc picks incorrect connection settings)

From: [email-masked] (Emacs bug Tracking System)
Date: Sat, 18 Apr 2009 03:50:03 +0000

This is an automatic notification regarding your bug report
which was filed against the emacs package:

#3037: 23.0.92; rcirc picks incorrect connection settings

It has been closed by Chong Yidong <[email-masked]>.

Their explanation is attached below along with your original report.
If this explanation is unsatisfactory and you have not received a
better one in a separate message then please contact Chong Yidong <[email-masked]> by
replying to this email.

-- 
3037: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=3037
Emacs Bug Tracking System
Contact [email-masked] with problems

Subject: Re: 23.0.92; rcirc picks incorrect connection settings
From: Chong Yidong <[email-masked]>
To: Yann Hodique <[email-masked]>
Cc: Ryan Yeske <[email-masked]>, [email-masked]
Date: Fri, 17 Apr 2009 23:47:07 -0400

> When using `rcirc' with argument (prompt for connection parameters), the
> global default values for :port, :nick and :channels are proposed
> instead of the server-specific ones. 
>
> Reason is that current code tries to get properties 'port, 'nick
> and 'channels, which is not correct.
> Following patch fixes this issue.

I've checked in your patch.  Thanks.

Subject: 23.0.92; rcirc picks incorrect connection settings
From: Yann Hodique <[email-masked]>
To: [email-masked]
Date: Fri, 17 Apr 2009 17:48:19 +0200

When using `rcirc' with argument (prompt for connection parameters), the
global default values for :port, :nick and :channels are proposed
instead of the server-specific ones. 

Reason is that current code tries to get properties 'port, 'nick
and 'channels, which is not correct.
Following patch fixes this issue.

Yann.

diff --git a/lisp/net/rcirc.el b/lisp/net/rcirc.el
index f63237f..1f31284 100644
--- a/lisp/net/rcirc.el
+++ b/lisp/net/rcirc.el
@@ -375,16 +375,16 @@ If ARG is non-nil, instead prompt for connection parameters."
             (server-plist (cdr (assoc-string server rcirc-server-alist)))
             (port (read-string "IRC Port: "
                                (number-to-string
-                                (or (plist-get server-plist 'port)
+                                (or (plist-get server-plist :port)
                                     rcirc-default-port))))
             (nick (read-string "IRC Nick: "
-                               (or (plist-get server-plist 'nick)
+                               (or (plist-get server-plist :nick)
                                    rcirc-default-nick)))
             (channels (split-string
                        (read-string "IRC Channels: "
                                     (mapconcat 'identity
                                                (plist-get server-plist
-                                                          'channels)
+                                                          :channels)
                                                " "))
                        "[, ]+" t)))
        (rcirc-connect server port nick rcirc-default-user-name

-- 
"Once more the drama begins."

-The Emperor Paul Muad'dib on his ascension to the Lion Throne