Clean UP

xbapps · Nov 29, 2024 · 5457327 · 5457327
1 parent 5897b52
commit 5457327
Showing 1 changed file with 17 additions and 21 deletions.
diff --git a/pkg/config/scraper_list.go b/pkg/config/scraper_list.go
@@ -191,8 +191,6 @@ func MigrateFromOfficalToCustom(id string, url string, name string, company stri
 		// Data taken from offical scraper list
 		scraper := ScraperConfig{URL: url, Name: name, Company: company, AvatarUrl: avatarUrl}
 
-		exists := false
-
 		// Update any alt sites that is using the old id to the new id
 		updateMasterSite := func(sites []ScraperConfig) {
 			for idx, site := range sites {
@@ -207,25 +205,23 @@ func MigrateFromOfficalToCustom(id string, url string, name string, company stri
 		updateMasterSite(scraperConfig.CustomScrapers.VrpornScrapers)
 		updateMasterSite(scraperConfig.CustomScrapers.VrphubScrapers)
 
-		if exists == false {
-			// Append our scraper to the the Custom Scraper list unless its new id already exists
-			switch customId {
-			case "slr":
-				if CheckMatchingSite(scraper, scraperConfig.CustomScrapers.SlrScrapers) == false {
-					scraperConfig.CustomScrapers.SlrScrapers = append(scraperConfig.CustomScrapers.SlrScrapers, scraper)
-				}
-			case "povr":
-				if CheckMatchingSite(scraper, scraperConfig.CustomScrapers.PovrScrapers) == false {
-					scraperConfig.CustomScrapers.PovrScrapers = append(scraperConfig.CustomScrapers.PovrScrapers, scraper)
-				}
-			case "vrporn":
-				if CheckMatchingSite(scraper, scraperConfig.CustomScrapers.VrpornScrapers) == false {
-					scraperConfig.CustomScrapers.VrpornScrapers = append(scraperConfig.CustomScrapers.VrpornScrapers, scraper)
-				}
-			case "vrphub":
-				if CheckMatchingSite(scraper, scraperConfig.CustomScrapers.VrphubScrapers) == false {
-					scraperConfig.CustomScrapers.VrphubScrapers = append(scraperConfig.CustomScrapers.VrphubScrapers, scraper)
-				}
+		// Append our scraper to the the Custom Scraper list unless its new id already exists
+		switch customId {
+		case "slr":
+			if CheckMatchingSite(scraper, scraperConfig.CustomScrapers.SlrScrapers) == false {
+				scraperConfig.CustomScrapers.SlrScrapers = append(scraperConfig.CustomScrapers.SlrScrapers, scraper)
+			}
+		case "povr":
+			if CheckMatchingSite(scraper, scraperConfig.CustomScrapers.PovrScrapers) == false {
+				scraperConfig.CustomScrapers.PovrScrapers = append(scraperConfig.CustomScrapers.PovrScrapers, scraper)
+			}
+		case "vrporn":
+			if CheckMatchingSite(scraper, scraperConfig.CustomScrapers.VrpornScrapers) == false {
+				scraperConfig.CustomScrapers.VrpornScrapers = append(scraperConfig.CustomScrapers.VrpornScrapers, scraper)
+			}
+		case "vrphub":
+			if CheckMatchingSite(scraper, scraperConfig.CustomScrapers.VrphubScrapers) == false {
+				scraperConfig.CustomScrapers.VrphubScrapers = append(scraperConfig.CustomScrapers.VrphubScrapers, scraper)
 			}
 		}