Merge pull request #92 from pawndev/feat/issue-78/add-knulli-game-metadata-to-gamelist-if-not-exist

feat(knulli): add grout to gamelist if not exist

Author: BrandonKowalski

app/setup.go

@@ -4,6 +4,7 @@ import (
 	"errors"
 	"grout/cache"
 	"grout/cfw"
+	"grout/cfw/knulli"
 	"grout/cfw/muos"
 	"grout/internal"
 	"grout/internal/environment"
@@ -82,12 +83,15 @@ func setup() SetupResult {
 		log.Fatalf("Failed to initialize i18n: %v", err)
 	}
 
+	if cfw.GetCFW() == cfw.Knulli {
+		knulli.AddToToolsGameList()
+	}
+
 	config, err := internal.LoadConfig()
 	isFirstLaunch := err != nil || (len(config.Hosts) == 0 && config.Language == "")
 
 	if isFirstLaunch {
 		logger.Debug("First launch detected, showing language selection")
-		cfw.FirstLaunchSetup()
 		languageScreen := ui.NewLanguageSelectionScreen()
 		selectedLanguage, langErr := languageScreen.Draw()
 		if langErr != nil {

cfw/knulli/startup.go cfw/knulli/setup.gocfw/knulli/startup.go renamed to cfw/knulli/setup.go

@@ -15,8 +15,8 @@ const (
 	flagPath               = "./knulli_restart_request"
 )
 
-func FirstLaunchSetup(romDir string) {
-	path := filepath.Join(romDir, "tools", "gamelist.xml")
+func AddToToolsGameList() {
+	path := filepath.Join(GetRomDirectory(), "tools", "gamelist.xml")
 	gaba.GetLogger().Debug("using filepath for knulli gamelist.xml", "path", path)
 	gl := gamelist.New()
 
@@ -37,6 +37,15 @@ func FirstLaunchSetup(romDir string) {
 		}
 	}
 
+	if gl.GameContainsElements(GroutEntryGameListName, []string{
+		gamelist.PathElement, gamelist.DescElement,
+		gamelist.ImageElement, gamelist.DeveloperElement,
+		gamelist.PlayersElement, gamelist.GenreElement,
+	}) {
+		gaba.GetLogger().Debug("gamelist.xml already contains Grout entry, skipping addition", "path", path)
+		return
+	}
+
 	gl.AdddOrUpdateEntry(GroutEntryGameListName, map[string]string{
 		gamelist.NameElement:      GroutEntryGameListName,
 		gamelist.DescElement:      "Download games wirelessly from your RomM instance",

cfw/setup.go

@@ -59,6 +59,31 @@ func (gl *GameList) Contains(element, value string) bool {
 	return false
 }
 
+func (gl *GameList) GetGameElementByName(name string) *etree.Element {
+	root := gl.document.SelectElement(GameListElement)
+	games := root.SelectElements(GameElement)
+	for _, game := range games {
+		nameElement := game.FindElement(NameElement)
+		if nameElement != nil && nameElement.Text() == name {
+			return game
+		}
+	}
+	return nil
+}
+
+func (gl *GameList) GameContainsElements(name string, elements []string) bool {
+	e := gl.GetGameElementByName(name)
+	if e == nil {
+		return false
+	}
+	for _, element := range elements {
+		if e.FindElement(element) == nil {
+			return false
+		}
+	}
+	return true
+}
+
 func (gl *GameList) Save(filepath string) error {
 	gl.document.Indent(4)
 	if err := gl.document.WriteToFile(filepath); err != nil {
@@ -77,22 +102,18 @@ func (gl *GameList) AddGameEntry(info map[string]string) {
 }
 
 func (gl *GameList) AdddOrUpdateEntry(name string, info map[string]string) {
-	root := gl.document.SelectElement(GameListElement)
-	games := root.SelectElements(GameElement)
-	for _, game := range games {
-		nameElement := game.FindElement(NameElement)
-		if nameElement != nil && nameElement.Text() == name {
-			// Update existing entry
-			for key, value := range info {
-				if element := game.FindElement(key); element != nil {
-					element.SetText(value)
-				} else {
-					game.CreateElement(key).SetText(value)
-				}
-			}
-			return
+	game := gl.GetGameElementByName(name)
+	if game == nil {
+		gl.AddGameEntry(info)
+		return
+	}
+
+	for key, value := range info {
+		if element := game.FindElement(key); element != nil {
+			element.SetText(value)
+		} else {
+			game.CreateElement(key).SetText(value)
 		}
 	}
-	// Add new entry
-	gl.AddGameEntry(info)
+
 }