Restore support for multiple channels + single user

irc/handlers.go

@@ -1273,7 +1273,9 @@ func sajoinHandler(server *Server, client *Client, msg ircmsg.Message, rb *Respo
 }
 
 // KICK <channel>{,<channel>} <user>{,<user>} [<comment>]
-// The number of channels must be either 1 or equal to the number of users
+// RFC 2812 requires the number of channels to be either 1 or equal to
+// the number of users.
+// Addditionally, we support multiple channels and a single user.
 func kickHandler(server *Server, client *Client, msg ircmsg.Message, rb *ResponseBuffer) bool {
 	hasPrivs := client.HasRoleCapabs("samode")
 	channels := strings.Split(msg.Params[0], ",")
@@ -1287,16 +1289,31 @@ func kickHandler(server *Server, client *Client, msg ircmsg.Message, rb *Respons
 		channel string
 		nick    string
 	}
-	kicks := make([]kickCmd, 0, len(users))
+	var kicks []kickCmd
-	channel := channels[0]
+	if len(users) == 1 {
-	for index, user := range users {
+		kicks = make([]kickCmd, 0, len(channels))
-		if len(channels) > 1 {
+		// Single user, possibly multiple channels
-			channel = channels[index]
+		user := users[0]
+		for _, channel := range channels {
+			if channel == "" {
+				continue // #679
+			}
+			kicks = append(kicks, kickCmd{channel, user})
 		}
-		if channel == "" {
+	} else {
-			continue // #679
+		// Multiple users, either a single channel or as many channels
+		// as users.
+		kicks = make([]kickCmd, 0, len(users))
+		channel := channels[0]
+		for index, user := range users {
+			if len(channels) > 1 {
+				channel = channels[index]
+			}
+			if channel == "" {
+				continue // #679
+			}
+			kicks = append(kicks, kickCmd{channel, user})
 		}
-		kicks = append(kicks, kickCmd{channel, user})
 	}
 
 	var comment string