[Quality Border] For Immersion addon’s quesr choice currency tooltip border.
hobulian opened this issue · 5 comments
Quest reward(currency) preview without addon immersion seems to work fine now.
But when with Immersion, if there is currency at quest reward, it's border and icon function seems to be not working now.
Can be fixed by adding
hooksecurefunc(tip,"SetQuestCurrency",SetQuestCurrency_Hook);
-- HOOK: SetQuestCurrency_Hook
local function ##SetQuestCurrency_Hook(self,type,id)
if (cfg.if_enable) and (not tipDataAdded[self]) then
local currencyId = GetQuestCurrencyID(type, id);
if (currencyId) then
tipDataAdded[self] = "currency";
LinkTypeFuncs.currency(self,nil,"currency",currencyId);
end
end
end
I have noticed, that some currency rewards of my own quests in quest log had a wrong border color. I added a fix to the function "ttItemRef.lua"->LinkTypeFuncs:currency(), so that currency containers are considered for border colors. So Renown does not need to be adjusted manually. Looks fine now. 🙂
I committed my changes to main-branch. Can you test with this version, if your issue is resolved already with this modification?
I have noticed, that some currency rewards of my own quests in quest log had a wrong border color. I added a fix to the function "ttItemRef.lua"->LinkTypeFuncs:currency(), so that currency containers are considered for border colors. So Renown does not need to be adjusted manually. Looks fine now. 🙂
I committed my changes to main-branch. Can you test with this version, if your issue is resolved already with this modification?
It's fine with other stuffs but immersion rewards seems still not working ..
I think "SetQuestCurrency" related stuff have to be added also.
-- HOOK: SetQuestCurrency_Hook
local function SetQuestCurrency_Hook(self,type,id)
if (cfg.if_enable) and (not tipDataAdded[self]) then
local currencyId = GetQuestCurrencyID(type, id);
if (currencyId) then
local quantity = select(3, GetQuestCurrencyInfo(type, id))
tipDataAdded[self] = "currency";
LinkTypeFuncs.currency(self,nil,"currency",currencyId, quantity);
end
end
end
I changed the hooking of quest(log) items and currencies to add compatibility to other addons (e.g. Immersion). I added the following hooks:
- GameTooltip:SetQuestItem()
- GameTooltip:SetQuestCurrency()
- GameTooltip:SetQuestLogItem()
- GameTooltip:SetQuestLogCurrency()
I temporarily installed addon Immersion and it looks fine now. I committed my changes to main-branch.
I changed the hooking of quest(log) items and currencies to add compatibility to other addons (e.g. Immersion). I added the following hooks:
- GameTooltip:SetQuestItem()
- GameTooltip:SetQuestCurrency()
- GameTooltip:SetQuestLogItem()
- GameTooltip:SetQuestLogCurrency()
I temporarily installed addon Immersion and it looks fine now. I committed my changes to main-branch.
Thanks for your hardworking. I know it's not easy to download and check addons that you are not using. I've checked your fix and it works fine now.