fix(tooltip): resolve global ID nil crash across all Tooltip functions

removed incorrect tutorial tooltip

ElvUI/Core/StaticPopups.lua

@@ -766,6 +766,8 @@ function E:StaticPopup_Resize(dialog, which)
 		height = height + 8 + editBox:GetHeight()
 	elseif info.hasMoneyFrame then
 		height = height + 16
+	elseif info.hasMoneyInputFrame then
+		height = height + 22
 	end
 
 	if height > maxHeightSoFar then
@@ -903,6 +905,13 @@ function E:StaticPopup_Show(which, text_arg1, text_arg2, data)
 		_G[name.."MoneyFrame"]:Hide()
 	end
 
+	-- Show or hide money input frame
+	if info.hasMoneyInputFrame then
+		_G[name.."MoneyInputFrame"]:Show()
+	else
+		_G[name.."MoneyInputFrame"]:Hide()
+	end
+
 -- Set the miscellaneous variables for the dialog
 	dialog.which = which
 	dialog.timeleft = info.timeout

ElvUI/Core/tutorials.lua

@@ -13,7 +13,6 @@ E.TutorialList = {
 	L["You can toggle the microbar by using your middle mouse button on the minimap you can also accomplish this by enabling the actual microbar located in the actionbar settings."],
 	L["A raid marker feature is available by pressing Escape -> Keybinds scroll to the bottom under ElvUI and setting a keybind for the raid marker."],
 	L["You can set your keybinds quickly by typing /kb."],
-	L["The focus unit can be set by typing /focus when you are targeting the unit you want to focus. It is recommended you make a macro to do this."],
 	L["ElvUI has a dual spec feature which allows you to load different profiles based on your current spec on the fly. You can enable this from the profiles tab."],
 	L["You can access copy chat and chat menu functions by mouse over the top right corner of chat panel and left/right click on the button that will appear."],
 	L["If you are experiencing issues with ElvUI try disabling all your addons except ElvUI, remember ElvUI is a full UI replacement addon, you cannot run two addons that do the same thing."],

ElvUI/Libraries/oUF/elements/assistantindicator.lua

@@ -59,7 +59,7 @@ local function Update(self, event)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: AssistantIndicator.Override(self, event, ...)
 	Used to completely override the element's update process.
 
@@ -67,7 +67,7 @@ local function Path(self, ...)
 	* event - the event triggering the update (string)
 	* ...   - the arguments accompanying the event (string)
 	--]]
-	return (self.AssistantIndicator.Override or Update) (self, unpack(arg))
+	return (self.AssistantIndicator.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/castbar.lua

@@ -368,9 +368,9 @@ local function onUpdate(self, elapsed)
 	end
 end
 
-local function Update(self, ...)
-	SPELLCAST_START(self, unpack(arg))
-	return SPELLCAST_CHANNEL_START(self, unpack(arg))
+local function Update(self, event, a, b, c, d, e)
+	SPELLCAST_START(self, event, a, b, c, d, e)
+	return SPELLCAST_CHANNEL_START(self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/combatindicator.lua

@@ -57,14 +57,14 @@ local function Update(self, event)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: CombatIndicator.Override(self, event)
 	Used to completely override the internal update function.
 
 	* self  - the parent object
 	* event - the event triggering the update (string)
 	--]]
-	return (self.CombatIndicator.Override or Update) (self, unpack(arg))
+	return (self.CombatIndicator.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/combopoints.lua

@@ -67,7 +67,7 @@ local function Update(self, event)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: ComboPoints.Override(self, event, ...)
 	Used to completely override the internal update function.
 
@@ -75,7 +75,7 @@ local function Path(self, ...)
 	* event - the event triggering the update (string)
 	* ...   - the arguments accompanying the event
 	--]]
-	return (self.ComboPoints.Override or Update) (self, unpack(arg))
+	return (self.ComboPoints.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/happinessindicator.lua

@@ -73,7 +73,7 @@ local function Update(self, event, unit)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: HappinessIndicator.Override(self, event, ...)
 	Used to completely override the internal update function.
 
@@ -81,7 +81,7 @@ local function Path(self, ...)
 	* event - the event triggering the update (string)
 	* ...   - the arguments accompanying the event
 	--]]
-	return (self.HappinessIndicator.Override or Update) (self, unpack(arg))
+	return (self.HappinessIndicator.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/health.lua

@@ -187,7 +187,7 @@ local function Update(self, event, unit)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: Health.Override(self, event, unit)
 	Used to completely override the internal update function.
 
@@ -195,7 +195,7 @@ local function Path(self, ...)
 	* event - the event triggering the update (string)
 	* unit  - the unit accompanying the event (string)
 	--]]
-	return (self.Health.Override or Update) (self, unpack(arg))
+	return (self.Health.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/leaderindicator.lua

@@ -60,7 +60,7 @@ local function Update(self, event)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: LeaderIndicator.Override(self, event, ...)
 	Used to completely override the internal update function.
 
@@ -68,7 +68,7 @@ local function Path(self, ...)
 	* event - the event triggering the update (string)
 	* ...   - the arguments accompanying the event
 	--]]
-	return (self.LeaderIndicator.Override or Update) (self, unpack(arg))
+	return (self.LeaderIndicator.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/masterlooterindicator.lua

@@ -81,7 +81,7 @@ local function Update(self, event)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: MasterLooterIndicator.Override(self, event, ...)
 	Used to completely override the internal update function.
 
@@ -89,7 +89,7 @@ local function Path(self, ...)
 	* event - the event triggering the update (string)
 	* ...   - the arguments accompanying the event
 	--]]
-	return (self.MasterLooterIndicator.Override or Update) (self, unpack(arg))
+	return (self.MasterLooterIndicator.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/portrait.lua

@@ -91,7 +91,7 @@ local function Update(self, event, unit)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: Portrait.Override(self, event, unit)
 	Used to completely override the internal update function.
 
@@ -99,7 +99,7 @@ local function Path(self, ...)
 	* event - the event triggering the update (string)
 	* unit  - the unit accompanying the event (string)
 	--]]
-	return (self.Portrait.Override or Update) (self, unpack(arg))
+	return (self.Portrait.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/power.lua

@@ -202,7 +202,7 @@ local function Update(self, event, unit)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: Power.Override(self, event, unit, ...)
 	Used to completely override the internal update function.
 
@@ -211,7 +211,7 @@ local function Path(self, ...)
 	* unit  - the unit accompanying the event (string)
 	* ...   - the arguments accompanying the event
 	--]]
-	return (self.Power.Override or Update) (self, unpack(arg))
+	return (self.Power.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/pvpindicator.lua

@@ -79,7 +79,7 @@ local function Update(self, event, unit)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[Override: PvPIndicator.Override(self, event, ...)
 	Used to completely override the internal update function.
 
@@ -87,7 +87,7 @@ local function Path(self, ...)
 	* event - the event triggering the update (string)
 	* ...   - the arguments accompanying the event
 	--]]
-	return (self.PvPIndicator.Override or Update) (self, unpack(arg))
+	return (self.PvPIndicator.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/raidtargetindicator.lua

@@ -59,14 +59,14 @@ local function Update(self, event)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: RaidTargetIndicator.Override(self, event)
 	Used to completely override the internal update function.
 
 	* self  - the parent object
 	* event - the event triggering the update (string)
 	--]]
-	return (self.RaidTargetIndicator.Override or Update) (self, unpack(arg))
+	return (self.RaidTargetIndicator.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/elements/range.lua

@@ -73,14 +73,14 @@ local function Update(self, event)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: Range.Override(self, event)
 	Used to completely override the internal update function.
 
 	* self  - the parent object
 	* event - the event triggering the update (string)
 	--]]
-	return (self.Range.Override or Update) (self, unpack(arg))
+	return (self.Range.Override or Update) (self, event, a, b, c, d, e)
 end
 
 -- Internal updating method

ElvUI/Libraries/oUF/elements/restingindicator.lua

@@ -57,14 +57,14 @@ local function Update(self, event)
 	end
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: RestingIndicator.Override(self, event)
 	Used to completely override the internal update function.
 
 	* self  - the parent object
 	* event - the event triggering the update (string)
 	--]]
-	return (self.RestingIndicator.Override or Update) (self, unpack(arg))
+	return (self.RestingIndicator.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF/events.lua

@@ -22,9 +22,9 @@ local function onEvent()
 end
 
 local event_metatable = {
-	__call = function(funcs, self, ...)
+	__call = function(funcs, self, event, a, b, c, d, e)
 		for _, func in next, funcs do
-			func(self, unpack(arg))
+			func(self, event, a, b, c, d, e)
 		end
 	end,
 }

ElvUI/Libraries/oUF_Plugins/oUF_DruidMana/LibDruidMana-1.0/LibDruidMana-1.0.lua

@@ -443,7 +443,7 @@ function lib:AddListener(tab, method)
 			error(format("Bad argument #3 to `AddListener'. Expected method, got %q.", type(tab[method])), 2)
 		end
 		func = function(...)
-			return tab[method](tab, unpack(arg))
+			return tab[method](tab, event, a, b, c, d, e)
 		end
 	elseif type(tab) == "function" then
 		func = tab

ElvUI/Libraries/oUF_Plugins/oUF_DruidMana/oUF_DruidMana.lua

@@ -53,8 +53,8 @@ local function Update(self, event, unit, currMana, maxMana)
 	end
 end
 
-local function Path(self, ...)
-	return (self.DruidAltMana.Override or Update) (self, unpack(arg))
+local function Path(self, event, a, b, c, d, e)
+	return (self.DruidAltMana.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ElementEnable(self)
@@ -101,8 +101,8 @@ local function Visibility(self, event, unit)
 	end
 end
 
-local function VisibilityPath(self, ...)
-	return (self.DruidAltMana.OverrideVisibility or Visibility) (self, unpack(arg))
+local function VisibilityPath(self, event, a, b, c, d, e)
+	return (self.DruidAltMana.OverrideVisibility or Visibility) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Libraries/oUF_Plugins/oUF_RaidDebuffs/oUF_RaidDebuffs.lua

@@ -109,13 +109,8 @@ end
 local function onUpdate(self, elapsed)
 	self.elapsed = (self.elapsed or 0) + elapsed
 	if self.elapsed >= 0.1 then
-		local _, _, _, _, _, _, timeLeft = UnitDebuff(self.__owner.unit, self.index, "HARMFUL")
-		if timeLeft and timeLeft > 0 then
-			self.time:SetText(formatTime(timeLeft))
-		else
-			self:SetScript('OnUpdate', nil)
-			self.time:Hide()
-		end
+		self:SetScript('OnUpdate', nil)
+		self.time:Hide()
 		self.elapsed = 0
 	end
 end
@@ -178,8 +173,9 @@ local function Update(self, event, unit)
 	local _stackThreshold = 0
 
 	for i = 1, 40 do
-		name, _, icon, count, debuffType, duration, expirationTime = UnitDebuff(unit, i, "HARMFUL")
-		if not (name and icon) then break end
+		local texture, stackCount, dType = UnitDebuff(unit, i, "HARMFUL")
+		name, icon, count, debuffType, duration, expirationTime = texture, texture, stackCount, dType, nil, nil
+		if not name then break end
 
 		if(addon.ShowDispellableDebuff and (element.showDispellableDebuff ~= false) and debuffType) then
 			if(addon.FilterDispellableDebuff) then
@@ -225,7 +221,7 @@ local function Update(self, event, unit)
 	}
 end
 
-local function Path(self, ...)
+local function Path(self, event, a, b, c, d, e)
 	--[[ Override: RaidDebuffs.Override(self, event, ...)
 	Used to completely override the element's update process.
 
@@ -233,7 +229,7 @@ local function Path(self, ...)
 	* event - the event triggering the update (string)
 	* ...   - the arguments accompanying the event (string)
 	--]]
-    return (self.RaidDebuffs.Override or Update) (self, unpack(arg))
+    return (self.RaidDebuffs.Override or Update) (self, event, a, b, c, d, e)
 end
 
 local function ForceUpdate(element)

ElvUI/Modules/Auras/Auras.lua

@@ -308,7 +308,7 @@ function A:ConfigureAuras(header, auraTable, weaponPosition)
 		top = max(top, button:GetTop() or -huge)
 		bottom = min(bottom, button:GetBottom() or huge)
 	end
-	local deadIndex = (getn(auraTable) + numWeapon) + 1
+	local deadIndex = getn(auraTable) + 1
 	button = _G[headerName.."AuraButton"..deadIndex]
 	while button do
 		if button:IsShown() then button:Hide() end

ElvUI/Modules/Blizzard/WatchFrame.lua

@@ -7,6 +7,40 @@ local min = math.min
 --WoW API / Variables
 local hooksecurefunc = hooksecurefunc
 local GetScreenHeight = GetScreenHeight
+local GetNumQuestWatches = GetNumQuestWatches
+local GetQuestIndexForWatch = GetQuestIndexForWatch
+local RemoveQuestWatch = RemoveQuestWatch
+local AddQuestWatch = AddQuestWatch
+local QuestWatch_Update = QuestWatch_Update
+
+local original_QuestWatch_Update = QuestWatch_Update
+function _G.QuestWatch_Update()
+	local watched = {}
+	local order = {}
+	local hasDuplicates = false
+	for i = 1, GetNumQuestWatches() do
+		local qIndex = GetQuestIndexForWatch(i)
+		if qIndex then
+			if watched[qIndex] then
+				hasDuplicates = true
+			else
+				watched[qIndex] = true
+				table.insert(order, qIndex)
+			end
+		end
+	end
+	
+	if hasDuplicates then
+		for i = 1, table.getn(order) do
+			RemoveQuestWatch(order[i])
+		end
+		for i = 1, table.getn(order) do
+			AddQuestWatch(order[i])
+		end
+	end
+	
+	original_QuestWatch_Update()
+end
 
 local WatchFrameHolder = CreateFrame("Frame", "WatchFrameHolder", E.UIParent)
 WatchFrameHolder:SetWidth(150)

ElvUI/Modules/Skins/Skins.lua

@@ -244,6 +244,7 @@ function S:HandleNextPrevButton(btn, useVertical, inverseDirection)
 end
 
 function S:HandleRotateButton(btn)
+	if not btn then return end
 	E:SetTemplate(btn, "Default")
 	E:Size(btn, btn:GetWidth() - 14, btn:GetHeight() - 14)
 

ElvUI/Modules/Tooltip/Tooltip.lua

@@ -273,7 +273,7 @@ function TT:SetItemRef(link)
 	if find(link, "^item:") then
 		if E.db.tooltip.spellID then
 			local id = tonumber(match(link, "(%d+)"))
-			ItemRefTooltip:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+			ItemRefTooltip:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 		end
 	end
 	ItemRefTooltip:Show()
@@ -312,8 +312,8 @@ function TT:SetAction(tt, buttonID)
 		self:SetPrice(tt, id, count)
 	end
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -329,8 +329,8 @@ function TT:SetAuctionItem(tt, type, index)
 		self:SetPrice(tt, id, count)
 	end
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -346,8 +346,8 @@ function TT:SetAuctionSellItem(tt)
 		self:SetPrice(tt, id, count)
 	end
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -362,8 +362,8 @@ function TT:SetBagItem(tt, bag, slot)
 		local _, count = GetContainerItemInfo(bag, slot)
 		self:SetPrice(tt, id, count)
 	end
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -382,8 +382,8 @@ function TT:SetCraftItem(tt, skill, slot)
 
 		self:SetPrice(tt, id, count)
 	end
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -394,8 +394,8 @@ function TT:SetCraftSpell(tt, id)
 
 	local id = tonumber(match(link, "enchant:(%d+)"))
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -421,8 +421,8 @@ function TT:SetHyperlink(tt, link, count)
 	end
 
 	if tt:GetName() == "GameTooltip" then
-		if E.db.tooltip.spellID then
-			tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+		if E.db.tooltip.spellID and id then
+			tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 		end
 	end
 
@@ -440,8 +440,8 @@ function TT:SetInboxItem(tt, index, attachmentIndex)
 		self:SetPrice(tt, id, count)
 	end
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -463,8 +463,8 @@ function TT:SetInventoryItem(tt, unit, slot)
 		self:SetPrice(tt, id, count)
 	end
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -479,8 +479,8 @@ function TT:SetLootItem(tt, slot)
 		local _, _, count = GetLootSlotInfo(slot)
 		self:SetPrice(tt, id, count)
 	end
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -495,8 +495,8 @@ function TT:SetLootRollItem(tt, rollID)
 		local _, _, count = GetLootRollItemInfo(rollID)
 		self:SetPrice(tt, id, count)
 	end
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -511,8 +511,8 @@ function TT:SetMerchantItem(tt, slot)
 		local _, _, _, count = GetMerchantItemInfo(slot)
 		self:SetPrice(tt, id, count)
 	end
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -527,8 +527,8 @@ function TT:SetQuestItem(tt, type, slot)
 		local _, _, count = GetQuestItemInfo(type, slot)
 		self:SetPrice(tt, id, count)
 	end
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -543,8 +543,8 @@ function TT:SetQuestLogItem(tt, type, index)
 		local _, _, count = GetQuestLogRewardInfo(index)
 		self:SetPrice(tt, id, count)
 	end
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -560,8 +560,8 @@ function TT:SetSendMailItem(tt, index)
 		self:SetPrice(tt, id, count)
 	end
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -577,8 +577,8 @@ function TT:SetTradePlayerItem(tt, index)
 		self:SetPrice(tt, id, count)
 	end
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -599,8 +599,8 @@ function TT:SetTradeSkillItem(tt, skill, slot)
 		self:SetPrice(tt, id, count)
 	end
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end
@@ -616,8 +616,8 @@ function TT:SetTradeTargetItem(tt, index)
 		self:SetPrice(tt, id, count)
 	end
 
-	if E.db.tooltip.spellID then
-		tt:AddLine(format("|cFFCA3C3C%s|r %d", ID, id))
+	if E.db.tooltip.spellID and id then
+		tt:AddLine(format("|cFFCA3C3C%s|r %d", (ID or "ID"), id))
 	end
 	tt:Show()
 end

ElvUI/Modules/UnitFrames/Elements/BuffIndicator.lua

@@ -111,7 +111,8 @@ function UF:UpdateAuraWatch(frame, petOverride, db)
 
 	for i = 1, getn(buffs) do
 		if buffs[i].id then
-			local name, _, image = UnitBuff(frame.unit, buffs[i].id)
+			local image, count = UnitBuff(frame.unit, buffs[i].id)
+			local name = image
 			if name then
 				local icon
 				if not auras.icons[buffs[i].id] then

ElvUI_Config/Libraries/AceConfig-3.0/AceConfigDialog-3.0/AceConfigDialog-3.0.lua

@@ -942,18 +942,18 @@ local function BuildTabs(group, options, path, appName)
 
 	BuildSortedOptionsTable(group, keySort, opts, options, path, appName)
 
-	for i = 1, #keySort do
+	for i = 1, getn(keySort) do
 		local k = keySort[i]
 		local v = opts[k]
 		if v.type == "group" then
-			path[#path+1] = k
+			path[getn(path)+1] = k
 			local inline = pickfirstset(v.dialogInline,v.guiInline,v.inline, false)
 			local hidden = CheckOptionHidden(v, options, path, appName)
 			if not inline and not hidden then
 				tinsert(tabs, k)
 				text[k] = GetOptionsMemberValue("name", v, options, path, appName)
 			end
-			path[#path] = nil
+			path[getn(path)] = nil
 		end
 	end
 

README.md

@@ -6,6 +6,15 @@
 # This repository is actively being fixed and maintained by Bluewhale.
 
 ## Changelog
+- **Fix:** Fixed global ID Lua 5.0 varargs crash across all Tooltip functions.
+- **Fix:** Fixed a Lua 5.0 syntax error in `AceConfigDialog-3.0.lua` where the `#` length operator was used instead of `table.getn`, preventing ElvUI_Config from loading.
+- **Fix:** Fixed nil ID formatting crash in Tooltip's SetHyperlink.
+- **Fix:** Removed implicit `arg` table memory leaks in high-frequency oUF element updates by replacing varargs (`...`) with static parameters.
+- **Fix:** Corrected bug in `Auras.lua` where unused buff icons remained visible on screen due to improper `deadIndex` calculation.
+- **Fix:** Fixed quest tracker duplicating entries by overriding `QuestWatch_Update` to deduplicate quests manually.
+- **Fix:** Hid the unwanted `MoneyInputFrame` from appearing over `StaticPopup` dialogs (like the UI Reload prompt) by updating custom popup handling.
+- **Fix:** Removed false tutorial text referencing a non-existent `/focus` command.
+- **Fix:** Corrected Vanilla WoW (1.12) `UnitBuff` and `UnitDebuff` API parsing in `BuffIndicator` and `oUF_RaidDebuffs` to prevent missing textures and improper debuff filtering.
 - **Fix:** Removed implicit `arg` table leak in `hooksecurefunc` (varargs `...` issue in Lua 5.0).
 - **Fix:** Stripped invisible UTF-8 BOM from `utf8data.lua` that caused Lua parser errors.
 - **Fix:** Resolved tooltip crash with ShaguTweaks by reusing existing `backdrop` frames in `CreateBackdrop` instead of unconditionally overwriting them.
@@ -13,9 +22,9 @@
 - **Fix:** Added type safety checks to `SetOutside` and `SetInside` to prevent crashes when other addons overwrite frame globals with dummy functions.
 - **Fix:** Added type safety check before calling `SetTexture("")` on Blizzard header frames, preventing crashes when headers are modified by other addons or the custom client.
 - **Fix:** Wrapped `SetParent` in `pcall` within `CreateBackdrop` to prevent crashes on non-UI object dummy tables.
-- **Fix:** Added graceful `nil` fallbacks to `E:Point`, `E:Size`, `E:Width`, and `E:Height` layout anchors, immunizing the UI against crashes when Turtle WoW or other addons delete elements.
+- **Fix:** Added graceful `nil` fallbacks to `E:Point`, `E:Size`, `E:Width`, and `E:Height` layout anchors, immunizing the UI against crashes when custom scripts or other addons delete elements.
 - **Fix:** Rewrote `Misc.lua` options skinning loops using `pairs()` to resolve Lua 5.0 `attempt to call a nil value` crashes caused by missing options tabs/sliders/checkboxes.
-- **Fix:** Added element existence checks in `Skins.lua` `HandleButton` to skip skinning gracefully instead of crashing on `nil`.
+- **Fix:** Added element existence checks in `Skins.lua` `HandleButton` and `HandleRotateButton` to skip skinning gracefully instead of crashing on `nil`.
 
 This is the backported version of ElvUI for World of Warcraft - Vanilla (1.12.1) 
 <br />