Fix bug in regex which removed newlines

command-manager.go

@@ -9,9 +9,10 @@ package disgoman
 import (
 	"errors"
 	"fmt"
-	"github.com/bwmarrin/discordgo"
+	"regexp"
-	"github.com/kballard/go-shellquote"
 	"strings"
+
+	"github.com/bwmarrin/discordgo"
 )
 
 // AddCommand adds the Command at the address passed in to the Commands array on the CommandManager.
@@ -90,13 +91,6 @@ func (c *CommandManager) OnMessage(session *discordgo.Session, m *discordgo.Mess
 
 	guild, _ := session.Guild(m.GuildID)
 
-	var cmd []string
-	// If we found our prefix then remove it and split the command into pieces
-	cmd, err = shellquote.Split(strings.TrimPrefix(content, prefix))
-	if err != nil {
-		if err.Error() == "Unterminated double-quoted string" || err.Error() == "Unterminated single-quoted string" {
-			cmd = strings.Split(strings.TrimPrefix(content, prefix), " ")
-		} else {
 	ctx := Context{
 		Session:      session,
 		Channel:      channel,
@@ -107,18 +101,23 @@ func (c *CommandManager) OnMessage(session *discordgo.Session, m *discordgo.Mess
 		Invoked:      "",
 		ErrorChannel: c.ErrorChannel,
 	}
-			c.ErrorChannel <- CommandError{
+
-				Context: ctx,
+	var cmd []string
-				Message: "",
+	// If we found our prefix then remove it and split the command into pieces
-				Error:   err,
+	content = strings.TrimPrefix(content, prefix)
+	r := regexp.MustCompile(`[^ "]+|"([^"]*)"`)
+	cmd = r.FindAllString(content, -1)
+	for i, val := range cmd {
+		cmd[i] = strings.Trim(val, "\"")
 	}
+
+	if len(cmd) < 1 {
 		return
 	}
-	}
 
 	var command *Command
-	invoked := cmd[0]
+	ctx.Invoked = cmd[0]
-	if cmnd, ok := c.Commands[invoked]; ok {
+	if cmnd, ok := c.Commands[ctx.Invoked]; ok {
 		command = cmnd
 	} else {
 		fmt.Println("Command Not Found")
@@ -126,23 +125,13 @@ func (c *CommandManager) OnMessage(session *discordgo.Session, m *discordgo.Mess
 	}
 
 	if command.SanitizeEveryone {
-		for i := 1; i < len(cmd[1:]); i++ {
+		for i := 1; i < len(cmd); i++ {
 			cmd[i] = strings.ReplaceAll(cmd[i], "@everyone", "@\ufff0everyone")
 			cmd[i] = strings.ReplaceAll(cmd[i], "@here", "@\ufff0here")
 		}
 	}
 
 	if !CheckPermissions(session, m.Author.ID, *channel, command.RequiredPermissions) {
-		ctx := Context{
-			Session:      session,
-			Channel:      channel,
-			Message:      m.Message,
-			User:         m.Author,
-			Guild:        guild,
-			Member:       m.Member,
-			Invoked:      cmd[0],
-			ErrorChannel: c.ErrorChannel,
-		}
 		c.ErrorChannel <- CommandError{
 			Context: ctx,
 			Message: "You don't have the correct permissions to run this command.",
@@ -152,16 +141,6 @@ func (c *CommandManager) OnMessage(session *discordgo.Session, m *discordgo.Mess
 	}
 
 	if !CheckPermissions(session, session.State.User.ID, *channel, command.RequiredPermissions) {
-		ctx := Context{
-			Session:      session,
-			Channel:      channel,
-			Message:      m.Message,
-			User:         m.Author,
-			Guild:        guild,
-			Member:       m.Member,
-			Invoked:      cmd[0],
-			ErrorChannel: c.ErrorChannel,
-		}
 		c.ErrorChannel <- CommandError{
 			Context: ctx,
 			Message: "I don't have the correct permissions to run this command.",
@@ -172,16 +151,6 @@ func (c *CommandManager) OnMessage(session *discordgo.Session, m *discordgo.Mess
 	}
 
 	if command.OwnerOnly && !c.IsOwner(m.Author.ID) {
-		ctx := Context{
-			Session:      session,
-			Channel:      channel,
-			Message:      m.Message,
-			User:         m.Author,
-			Guild:        guild,
-			Member:       m.Member,
-			Invoked:      cmd[0],
-			ErrorChannel: c.ErrorChannel,
-		}
 		c.ErrorChannel <- CommandError{
 			Context: ctx,
 			Message: "Sorry, only the bot owner(s) can run that command!",
@@ -191,16 +160,5 @@ func (c *CommandManager) OnMessage(session *discordgo.Session, m *discordgo.Mess
 
 	}
 
-	context := Context{
+	go command.Invoke(ctx, cmd[1:])
-		Session:      session,
-		Channel:      channel,
-		Message:      m.Message,
-		User:         m.Author,
-		Guild:        guild,
-		Member:       m.Member,
-		Invoked:      invoked,
-		ErrorChannel: c.ErrorChannel,
-	}
-
-	go command.Invoke(context, cmd[1:])
 }

utils.go

@@ -20,7 +20,7 @@ func GetDefaultStatusManager() StatusManager {
 			"Golang!",
 			"DiscordGo!",
 			"Disgoman!",
-		}, "30s"}
+		}, "300s"}
 }
 
 // HasHigherRole checks if the caller has a higher role than the target in the given guild