fix: priciest tooltip showing base item, not actual looted item

will0w7 · Oct 6, 2024 · 06cbf5a · 06cbf5a
1 parent 62d5a4d
commit 06cbf5a
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/core/Core.lua b/core/Core.lua
@@ -207,7 +207,7 @@ function Core.LootEventHandler(_, event, ...)
         Data.AddItemsMoney(totalPrice)
         Data.AddTotalMoney(totalPrice)
         -- only price of individual items, not groups (1xBismuth not 5xBismuth)
-        Data.SetPriciest(price, itemID)
+        Data.SetPriciest(price, itemLink)
         UpdateLoot()
     elseif event == Constants.Events.ChatMsgMoney or event == Constants.Events.QuestTurnedIn then
         -- here we dont stop interaction, if we turn in a quest with a profession

diff --git a/data/Data.lua b/data/Data.lua
@@ -32,7 +32,7 @@ Data.DB.prototype = {
     ItemsMoney = 0,
     TotalMoney = 0,
     Priciest = 0,
-    PriciestID = 0,
+    PriciestLink = "",
     ListLootedItems = CircularBuffer,
     ----------------------------------------------
     Timer = 0,
@@ -153,18 +153,18 @@ function Data.GetPriciest()
     return MoneyLooterDB.Priciest
 end
 
----@param id integer
+---@param itemLink string
 ---@param money integer
-function Data.SetPriciest(money, id)
+function Data.SetPriciest(money, itemLink)
     if money ~= nil and money > MoneyLooterDB.Priciest then
         MoneyLooterDB.Priciest = money
-        MoneyLooterDB.PriciestID = id
+        MoneyLooterDB.PriciestLink = itemLink
     end
 end
 
----@return integer
-function Data.GetPriciestID()
-    return MoneyLooterDB.PriciestID
+---@return string
+function Data.GetPriciestLink()
+    return MoneyLooterDB.PriciestLink
 end
 
 ---@return table

diff --git a/ui/UIEvents.lua b/ui/UIEvents.lua
@@ -133,10 +133,10 @@ UI.MLMainFrame.MinimizeCheck:SetScript(Constants.Events.OnClick, function()
 end)
 
 UI.MLMainFrame.PriciestFS:SetScript(Constants.Events.OnEnter, function()
-    local priciestID = Data.GetPriciestID()
-    if priciestID == nil or priciestID == 0 then return end
+    local priciestLink = Data.GetPriciestLink()
+    if priciestLink == nil or priciestLink == "" then return end
     GameTooltip:SetOwner(UI.MLMainFrame.PriciestFS, "ANCHOR_BOTTOMRIGHT")
-    GameTooltip:SetItemByID(Data.GetPriciestID())
+    GameTooltip:SetHyperlink(priciestLink)
     GameTooltip:Show()
 end)