mtgban: Make sure to keep market shorthands unique for sealed scrapers

mtgban · Oct 5, 2023 · 261edee · 261edee
1 parent 9e37deb
commit 261edee
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/mtgban/market.go b/mtgban/market.go
@@ -2,6 +2,7 @@ package mtgban
 
 import (
 	"sort"
+	"strings"
 )
 
 // Separate a Market into multiple Seller objects
@@ -25,6 +26,10 @@ func Seller2Sellers(market Market) ([]Seller, error) {
 		seller.info = market.Info()
 		seller.info.Name = sellerName
 		seller.info.Shorthand = sellerName
+		// Shorthand must be unique and there may be stores that share the same identifier
+		if market.Info().SealedMode && !strings.Contains(seller.info.Shorthand, "Sealed") {
+			seller.info.Shorthand += " Sealed"
+		}
 		sellers = append(sellers, seller)
 	}
 	return sellers, nil