Fix megalomaniac trader crash (#10181)

* Fix #10180

* Add test

* Fix test

---------

Co-authored-by: LocalIdentity <localidentity2@gmail.com>

Author: vaisest

spec/System/TestTradeQuery_spec.lua

@@ -61,6 +61,45 @@ describe("TradeQuery", function()
 			assert.are.equal(0, #tooltip.lines)
 		end)
 	end)
+	describe("GetResultEvaluation", function()
+		it("evaluates a socketed Megalomaniac by node combination", function()
+			local slotTbl = {
+				slotName = "Megalomaniac", unique = true, alreadyCorrupted = true, selectedJewelNodeId = 12345,
+			}
+			local tq = new("TradeQuery"):TradeQuery({ itemsTab = {} })
+			tq.statSortSelectionList = {}
+			tq.tradeQueryGenerator = new("TradeQueryGenerator"):TradeQueryGenerator({ itemsTab = {} })
+			tq.itemsTab.build = {
+				spec = {
+					tree = {
+						clusterNodeMap = {
+							["Node One"] = { dn = "Node One" },
+							["Node Two"] = { dn = "Node Two" },
+							["Node Three"] = { dn = "Node Three" },
+						}
+					}
+				}
+			}
+			tq.slotTables[1] = slotTbl
+			tq.resultTbl[1] = {
+				[1] = {
+					item_string = table.concat({
+						"1 Added Passive Skill is Node One",
+						"1 Added Passive Skill is Node Two",
+						"1 Added Passive Skill is Node Three",
+					}, "\n")
+				}
+			}
+
+			local evaluation = tq:GetResultEvaluation(1, 1, function() return {} end, {})
+
+			assert.are.equal(4, #evaluation)
+			for _, entry in ipairs(evaluation) do
+				assert.is_table(entry.DNs)
+				assert.is_true(#entry.DNs >= 2)
+			end
+		end)
+	end)
 	describe("ReduceOutput", function()
 		it("preserves lower-is-better values for weighted result comparison", function()
 			local weights = {

src/Classes/TradeQuery.lua

@@ -834,8 +834,7 @@ function TradeQueryClass:GetResultEvaluation(row_idx, result_index, calcFunc, ba
 	end
 	local slotTbl = self.slotTables[row_idx]
 	local jewelNodeId = slotTbl.nodeId or slotTbl.selectedJewelNodeId
-	local slotName = jewelNodeId and "Jewel " .. tostring(jewelNodeId) or slotTbl.slotName
-	if slotName == "Megalomaniac" then
+	if slotTbl.slotName == "Megalomaniac" then
 		local addedNodes = {}
 		for nodeName in (result.item_string.."\r\n"):gmatch("1 Added Passive Skill is (.-)\r?\n") do
 			t_insert(addedNodes, self.itemsTab.build.spec.tree.clusterNodeMap[nodeName])
@@ -857,6 +856,7 @@ function TradeQueryClass:GetResultEvaluation(row_idx, result_index, calcFunc, ba
 		}
 		table.sort(result.evaluation, function(a, b) return a.weight > b.weight end)
 	else
+		local slotName = jewelNodeId and "Jewel " .. tostring(jewelNodeId) or slotTbl.slotName
 		local item = new("Item"):Item(result.item_string)
 
 		local output = self:ReduceOutput(calcFunc({ repSlotName = slotName, repItem = item }))