master 9fc1fdc: Restore the previous order of ERC's '/whois' arguments

[Amin Bandali]

branch: master
commit 9fc1fdcbf330b0a85cd019bb75afcb8d36243524
Author: Amin Bandali <bandali@gnu.org>
Commit: Amin Bandali <bandali@gnu.org>

    Restore the previous order of ERC's '/whois' arguments
    
    * etc/NEWS: Remove unneeded entry.
    * lisp/erc/erc.el (erc-cmd-WHOIS): Restore the previous order of
    arguments sent to the server, so that there's no change in the
    function's behavior.  Instead, rename the arguments to be more
    accurate, and expand upon them in the doc string.
---
 etc/NEWS        |  5 -----
 lisp/erc/erc.el | 29 ++++++++++++++++++-----------
 2 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/etc/NEWS b/etc/NEWS
index 25164d5..d57f67e 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -2881,11 +2881,6 @@ The new '/opme' convenience command asks ChanServ to set 
the operator
 status for the current nick in the current channel, and '/deopme'
 unsets it.
 
----
-*** Fix the order of '/whois' arguments sent to the server.
-Per RFC 1459 and RFC 2812, when given, the optional 'server' argument
-for the WHOIS command must come before the 'user' argument, not after.
-
 ** xwidget-webkit mode
 
 ---
diff --git a/lisp/erc/erc.el b/lisp/erc/erc.el
index 7fa2d37..ccb1f63 100644
--- a/lisp/erc/erc.el
+++ b/lisp/erc/erc.el
@@ -3298,17 +3298,24 @@ a script after exceeding the flood threshold."
     t)
    (t nil)))
 
-(defun erc-cmd-WHOIS (user &optional server)
-  "Display whois information for USER.
-
-If SERVER is non-nil, use that, rather than the current server.
-This is useful for getting the time USER has been idle for, if
-USER is on a different server of the network than the current
-user, since only the server the user is connected to knows this
-information."
-  (let ((send (if server
-                  (format "WHOIS %s %s" server user)
-                (format "WHOIS %s" user))))
+(defun erc-cmd-WHOIS (nick-or-server &optional nick-if-server)
+  "Display whois information for the given user.
+
+If NICK-IF-SERVER is nil, NICK-OR-SERVER should be the nick of
+the user about whom the whois information is to be requested.
+Otherwise, if NICK-IF-SERVER is non-nil, NICK-OR-SERVER should be
+the server to which the user with the nick NICK-IF-USER is
+connected to.
+
+Specifying the server NICK-OR-SERVER that the nick NICK-IF-SERVER
+is connected to is useful for getting the time the NICK-IF-SERVER
+user has been idle for, when the user NICK-IF-SERVER is connected
+to a different server of the network than the one current user is
+connected to, since only the server a user is connected to knows
+the idle time of that user."
+  (let ((send (if nick-if-server
+                  (format "WHOIS %s %s" nick-or-server nick-if-server)
+                (format "WHOIS %s" nick-or-server))))
     (erc-log (format "cmd: %s" send))
     (erc-server-send send)
     t))