Fix timeless jewel and rebuildurl on league and realm selectors

Author: vaisest

src/Classes/CompareBuySimilar.lua

@@ -366,35 +366,37 @@ function M.openPopup(item, slotName, primaryBuild)
 		end)
 	end
 
+	local function rebuildUrl()
+		local result = buildURL(item, slotName, controls, modEntries, defenceEntries, isUnique)
+		uri = result
+	end
 	-- Realm dropdown
 	controls.realmLabel = new("LabelControl", {"TOPLEFT", nil, "TOPLEFT"}, {leftMargin, ctrlY, 0, 16}, "^7Realm:")
 	controls.realmDrop = new("DropDownControl", {"LEFT", controls.realmLabel, "RIGHT"}, {4, 0, 80, 20}, {"PC", "PS4", "Xbox"}, function(index, value)
 		local realmApiId = REALM_API_IDS[value] or "pc"
 		fetchLeaguesForRealm(realmApiId)
+		rebuildUrl()
 	end)
 
 	-- League dropdown
 	controls.leagueLabel = new("LabelControl", {"LEFT", controls.realmDrop, "RIGHT"}, {12, 0, 0, 16}, "^7League:")
 	controls.leagueDrop = new("DropDownControl", {"LEFT", controls.leagueLabel, "RIGHT"}, {4, 0, 160, 20}, {"Loading..."}, function(index, value)
 		-- League selection stored in the dropdown itself
+		rebuildUrl()
 	end)
 	controls.leagueDrop.enabled = function() return #controls.leagueDrop.list > 0 and controls.leagueDrop.list[1] ~= "Loading..." end
 
 	-- Listed status dropdown
 	controls.listedDrop = new("DropDownControl", {"TOPRIGHT", nil, "TOPRIGHT"}, {-leftMargin, ctrlY, 242, 20}, LISTED_STATUS_LABELS, function(index, value)
 		-- Listed status selection stored in the dropdown itself
+		rebuildUrl()
 	end)
 	controls.listedLabel = new("LabelControl", {"RIGHT", controls.listedDrop, "LEFT"}, {-4, 0, 0, 16}, "^7Listed:")
 
 	-- Fetch initial leagues for default realm
 	fetchLeaguesForRealm("pc")
 	ctrlY = ctrlY + rowHeight + 4
 
-	local function rebuildUrl()
-		local result = buildURL(item, slotName, controls, modEntries, defenceEntries, isUnique)
-		uri = result
-	end
-
 
 	if isUnique then
 		-- Unique item name label

src/Classes/TradeHelpers.lua

@@ -144,6 +144,20 @@ end
 function M.findTradeIdOption(modLine, modType)
 	-- match stringify() behaviour and ignore casing
 	modLine = modLine:gsub("\n", " "):lower()
+	-- exception: timeless jewels
+	-- these have special pseudo trade ids and are not options in poe1
+	local timelessPatterns = { "bathed in the blood of (%d+) sacrificed in the name of (.+)",
+		"carved to glorify (%d+) new faithful converted by high templar (.+)",
+		"commanded leadership over (%d+) warriors under (.+)",
+		"commissioned (%d+) coins to commemorate (.+)",
+		"denoted service of (%d+) dekhara in the akhara of (.+)",
+		"remembrancing (%d+) songworthy deeds by the line of (.+)" }
+	for _, pat in ipairs(timelessPatterns) do
+		local value, conqueror = modLine:match(pat)
+		if conqueror then
+			return "explicit.pseudo_timeless_jewel_" .. conqueror, tonumber(value)
+		end
+	end
 	local tradeStats = M.getTradeStats()
 	local optionTradeStatMap = getOptionTradeStatMap(tradeStats)
 	if not tradeStats or not optionTradeStatMap then return end