Removed old cache function, fixed bug

On download and play, we were always breaking out of the loop

Author: TheTipo01

download.go

@@ -5,7 +5,6 @@ import (
 	"fmt"
 	"github.com/bwmarrin/discordgo"
 	"github.com/zmb3/spotify"
-	"io/ioutil"
 	"os"
 	"os/exec"
 	"runtime"
@@ -16,28 +15,6 @@ import (
 func downloadAndPlay(s *discordgo.Session, guildID, channelID, link, user, txtChannel string) {
 	go sendAndDeleteEmbed(s, NewEmbed().SetTitle(s.State.User.Username).AddField("Enqueued", link).SetColor(0x7289DA).MessageEmbed, txtChannel)
 
-	files, _ := ioutil.ReadDir("./audio_cache")
-
-	//We check if the song is already downloaded
-	for _, f := range files {
-		//This one is how we handle id
-		realId := strings.TrimSuffix(f.Name(), ".dca")
-		//This one is for caching purposes
-		id := strings.Split(realId, "-")[0]
-		//If for reasons, the id is empty we just skip
-		if id == "" {
-			break
-		}
-
-		if strings.Contains(link, id) && f.Size() != 0 {
-			el := Queue{"", "", realId, link, user, nil, 0, ""}
-			queue[guildID] = append(queue[guildID], el)
-			addInfo(id, guildID)
-			playSound(s, guildID, channelID, f.Name(), txtChannel, findQueuePointer(guildID, realId))
-			return
-		}
-	}
-
 	//Gets info about songs
 	out, _ := exec.Command("youtube-dl", "--ignore-errors", "-q", "--no-warnings", "-j", link).Output()
 
@@ -75,7 +52,6 @@ func downloadAndPlay(s *discordgo.Session, guildID, channelID, link, user, txtCh
 		queue[guildID] = append(queue[guildID], el)
 		go playSound(s, guildID, channelID, fileName+".dca", txtChannel, findQueuePointer(guildID, fileName))
 
-		break
 	}
 
 }

utilities.go

@@ -5,8 +5,6 @@ import (
 	"github.com/bwmarrin/discordgo"
 	"log"
 	"net/url"
-	"os/exec"
-	"strings"
 	"time"
 )
 
@@ -66,24 +64,6 @@ func isValidUrl(toTest string) bool {
 	return err == nil
 }
 
-//Gets title and duration for a given video
-func addInfo(id string, guild string) {
-	for i, el := range queue[guild] {
-		if el.id == id {
-			out, _ := exec.Command("youtube-dl", "-e", "--get-duration", el.link).Output()
-			output := strings.Split(strings.TrimSuffix(string(out), "\n"), "\n")
-
-			if len(output) == 2 {
-				queue[guild][i].title = output[0]
-				queue[guild][i].duration = output[1]
-			} else {
-				removeFromQueue(id, guild)
-			}
-			return
-		}
-	}
-}
-
 //Removes element from the queue
 func removeFromQueue(id string, guild string) {
 	for i, q := range queue[guild] {