tcgplayer: Account for condition in links

mtgban · Nov 25, 2023 · 2e2b673 · 2e2b673
1 parent 46518c4
commit 2e2b673
Show file tree

Hide file tree

Showing 7 changed files with 16 additions and 7 deletions.
diff --git a/sealedev/sealedev.go b/sealedev/sealedev.go
@@ -228,7 +228,7 @@ func (ss *SealedEVScraper) runEV(prod productChan, channelOut chan respChan, pri
 			var link string
 			tcgID, _ := strconv.Atoi(product.Identifiers["tcgplayerProductId"])
 			if tcgID != 0 {
-				link = tcgplayer.TCGPlayerProductURL(tcgID, "", ss.Affiliate, "")
+				link = tcgplayer.TCGPlayerProductURL(tcgID, "", ss.Affiliate, "", "")
 			}
 
 			channelOut <- respChan{

diff --git a/tcgplayer/generic.go b/tcgplayer/generic.go
@@ -102,7 +102,7 @@ func (tcg *TCGPlayerGeneric) processPage(channel chan<- genericChan, page int) e
 			"TCG Low", "TCG Market", "TCG Mid", "TCG Direct Low",
 		}
 
-		link := TCGPlayerProductURL(result.ProductId, result.SubTypeName, tcg.Affiliate, "")
+		link := TCGPlayerProductURL(result.ProductId, result.SubTypeName, tcg.Affiliate, "", "")
 
 		keys := []string{
 			fmt.Sprint(result.ProductId),

diff --git a/tcgplayer/index.go b/tcgplayer/index.go
@@ -97,7 +97,7 @@ func (tcg *TCGPlayerIndex) processEntry(channel chan<- responseChan, reqs []inde
 		prices := []float64{
 			result.LowPrice, result.MarketPrice, result.MidPrice, result.DirectLowPrice,
 		}
-		link := TCGPlayerProductURL(result.ProductId, result.SubTypeName, tcg.Affiliate, co.Language)
+		link := TCGPlayerProductURL(result.ProductId, result.SubTypeName, tcg.Affiliate, "", co.Language)
 
 		for i := range availableIndexNames {
 			if prices[i] == 0 {

diff --git a/tcgplayer/sealed.go b/tcgplayer/sealed.go
@@ -60,7 +60,7 @@ func (tcg *TCGPlayerSealed) processEntries(channel chan<- responseChan, reqs []i
 			}
 		}
 
-		link := TCGPlayerProductURL(result.ProductId, "", tcg.Affiliate, "")
+		link := TCGPlayerProductURL(result.ProductId, "", tcg.Affiliate, "", "")
 		out := responseChan{
 			cardId: uuid,
 			entry: mtgban.InventoryEntry{

diff --git a/tcgplayer/seller.go b/tcgplayer/seller.go
@@ -173,7 +173,7 @@ func (tcg *TCGSellerInventory) processInventory(channel chan<- responseChan, res
 				continue
 			}
 
-			link := TCGPlayerProductURL(int(result.ProductID), listing.Printing, tcg.Affiliate, listing.Language)
+			link := TCGPlayerProductURL(int(result.ProductID), listing.Printing, tcg.Affiliate, listing.Condition, listing.Language)
 			out := responseChan{
 				cardId: cardId,
 				entry: mtgban.InventoryEntry{

diff --git a/tcgplayer/tcgplayer.go b/tcgplayer/tcgplayer.go
@@ -145,7 +145,7 @@ func (tcg *TCGPlayerMarket) processEntry(channel chan<- responseChan, reqs []mar
 			if req.Printing == "FOIL" {
 				printing = "Foil"
 			}
-			link := TCGPlayerProductURL(req.ProductId, printing, tcg.Affiliate, req.Language)
+			link := TCGPlayerProductURL(req.ProductId, printing, tcg.Affiliate, cond, req.Language)
 
 			for i := range availableMarketNames {
 				if prices[i] == 0 {

diff --git a/tcgplayer/utils.go b/tcgplayer/utils.go
@@ -12,7 +12,7 @@ import (
 
 const baseProductURL = "https://www.tcgplayer.com/product/"
 
-func TCGPlayerProductURL(productId int, printing, affiliate, language string) string {
+func TCGPlayerProductURL(productId int, printing, affiliate, condition, language string) string {
 	u, err := url.Parse(baseProductURL + fmt.Sprint(productId))
 	if err != nil {
 		return ""
@@ -28,6 +28,15 @@ func TCGPlayerProductURL(productId int, printing, affiliate, language string) st
 		v.Set("utm_source", affiliate)
 		v.Set("partner", affiliate)
 	}
+	if condition != "" {
+		for full, short := range conditionMap {
+			if short == condition {
+				condition = full
+				break
+			}
+		}
+		v.Set("Condition", condition)
+	}
 	if language != "" {
 		language = mtgmatcher.Title(language)
 		switch language {