WoWInterface - Frame error from Artifact

WoWInterface (https://www.wowinterface.com/forums/index.php)

- Lua/XML Help (https://www.wowinterface.com/forums/forumdisplay.php?f=16)

- - Frame error from Artifact (https://www.wowinterface.com/forums/showthread.php?t=55720)

Frame error from Artifact

1x FrameXML\MainMenuBar.lua:125: Usage: local cost = C_ArtifactUI.GetCostForPointAtRank(rank, tier)
[C]: in function `GetCostForPointAtRank'
FrameXML\MainMenuBar.lua:125: in function `MainMenuBar_GetNumArtifactTraitsPurchasableFromXP'
_Deranjata\modules\xpbar\artxp.lua:34: in function <_Deranjata\modules\xpbar\artxp.lua:29>

Locals:
(*temporary) = 38
(*temporary) = nil

Here is the function in its entirety, im really struggling to figure out what is broken i can find no documentation that any of these were changed or discontinued

Lua Code:

derArtifactBar:SetScript("OnEvent", function(self, event, ...)
    if not HasArtifactEquipped() then self:Hide() return end
    self:Show()
 
    local itemID, altItemID, name, icon, totalXP, pointsSpent = C_ArtifactUI.GetEquippedArtifactInfo()
    local numPoints, artifactXP, xpForNextPoint = MainMenuBar_GetNumArtifactTraitsPurchasableFromXP(pointsSpent, totalXP)
    self:SetMinMaxValues(0,xpForNextPoint)
    self:SetValue(artifactXP)
 
    self.Text:SetFormattedText("Artifact XP: %d / %d       +%d Points",artifactXP,xpForNextPoint,numPoints)
end)

Your best bet is checking the most recent Blizzard Interface files.
Also, pasting to what events the frame derArtifactBar is registered, would be beneficial.

I don't see C_ArtifactUI.GetCostForPointAtRank in your posted code. Perhaps posting all of your code would shed some light.

Because I don't see it, I am thinking you have tainted the global table with a leaking variable. In which case I highly suggest running your Lua files through this tool.

i think i may have found a way around this

Lua Code:

--Thank you for your help 
--SDPhantom
--Phanx
--Fizzlemizz
local AXP_COLOR = { r = 0.6, g = 0.2, b = 0.1 }
--Lets set up the Frame
local derArtifactBar = CreateFrame("Statusbar", "derArtifactBar",artifactFrame)
derArtifactBar:SetPoint("CENTER", 0, 0)
derArtifactBar:SetHeight(18)
derArtifactBar:SetWidth(artifactFrame:GetWidth() * .968)
derArtifactBar:SetStatusBarTexture("Interface\\AddOns\\_Deranjata\\media\\cast\\Waterline")
derArtifactBar:SetStatusBarColor(AXP_COLOR.r, AXP_COLOR.g, AXP_COLOR.b, 0.5)
derArtifactBar:SetBackdrop({
    bgFile =  "Interface\\AddOns\\_Deranjata\\media\\cast\\Lines",
    insets = { left = 1, right = 1, top = 1, bottom = 1 }
})
derArtifactBar:SetBackdropColor(0.1, 0.1, 0.1)
derArtifactBar:SetBackdropBorderColor(0.6, 0.6, 0.6)
--lets clear any text then set out centerpoint
derArtifactBar.Text=derArtifactBar:CreateFontString(nil, "OVERLAY", "GameFontNormal")
derArtifactBar.Text:SetPoint("CENTER")      
 
--Registering out watched events
derArtifactBar:RegisterEvent("PLAYER_ENTERING_WORLD")
derArtifactBar:RegisterEvent("ARTIFACT_XP_UPDATE")
derArtifactBar:RegisterEvent("UNIT_INVENTORY_CHANGED")
 
-- now lets update the position and text on the bar so we can always have accurate info
derArtifactBar:SetScript("OnEvent", function(self, event, ...)
    if not HasArtifactEquipped() then self:Hide() return end
    self:Show()
 
    --local itemID, altItemID, name, icon, xp, pointsSpent = C_ArtifactUI.GetEquippedArtifactInfo()
    local itemID, altItemID, name, icon, xp, pointsSpent, quality, artifactAppearanceID, appearanceModID, itemAppearanceID, altItemAppearanceID, altOnTop, artifactTier = C_ArtifactUI.GetEquippedArtifactInfo()
    local numPoints, artifactXP, xpForNextPoint = MainMenuBar_GetNumArtifactTraitsPurchasableFromXP(pointsSpent, xp, artifactTier) --(pointsSpent, totalXP)
    self:SetMinMaxValues(0,xpForNextPoint)
    self:SetValue(artifactXP)
 
    self.Text:SetFormattedText("Artifact XP: %d / %d       +%d Points",artifactXP,xpForNextPoint,numPoints)
end)

According to gamepedia, C_ArtifactUI.GetCostForPointAtRank is WoW API function.
Edit. Did you take a break, for, like 6 months from WoW?

Quote:

Originally Posted by Kakjens (Post 325050)

According to gamepedia, C_ArtifactUI.GetCostForPointAtRank is WoW API function.
Edit. Did you take a break, for, like 6 months from WoW?

six months.... maybe more

Lua Code:

local GetCostForPointAtRank = C_ArtifactUI.GetCostForPointAtRank
local itemID, altItemID, name, icon, artifactXP, pointsSpent, quality, artifactAppearanceID, appearanceModID, itemAppearanceID, altItemAppearanceID, altOnTop, artifactTier = C_ArtifactUI.GetEquippedArtifactInfo()
local pointsAvailable = 0
local xpForNextPoint = GetCostForPointAtRank(pointsSpent + pointsAvailable, artifactTier) or 0
while artifactXP >= xpForNextPoint  do
    artifactXP = artifactXP - xpForNextPoint
    pointsAvailable = pointsAvailable + 1
    xpForNextPoint = GetCostForPointAtRank(pointsSpent + pointsAvailable, artifactTier) or 0
end
local text = floor((artifactXP/xpForNextPoint)*100).."%"

You might want to add a check for artifactXP/xpForNextPoint being less than 0 or greater than 1.

Quote:

Originally Posted by myrroddin (Post 325047)

According to the stack trace, it is called by MainMenuBar_GetNumArtifactTraitsPurchasableFromXP, which the OP is calling.

Quote:

Originally Posted by Fizzlemizz (Post 325053)

Lua Code:

local GetCostForPointAtRank = C_ArtifactUI.GetCostForPointAtRank
local itemID, altItemID, name, icon, artifactXP, pointsSpent, quality, artifactAppearanceID, appearanceModID, itemAppearanceID, altItemAppearanceID, altOnTop, artifactTier = C_ArtifactUI.GetEquippedArtifactInfo()
local pointsAvailable = 0
local xpForNextPoint = GetCostForPointAtRank(pointsSpent + pointsAvailable, artifactTier) or 0
while artifactXP >= xpForNextPoint  do
    artifactXP = artifactXP - xpForNextPoint
    pointsAvailable = pointsAvailable + 1
    xpForNextPoint = GetCostForPointAtRank(pointsSpent + pointsAvailable, artifactTier) or 0
end
local text = floor((artifactXP/xpForNextPoint)*100).."%"

You might want to add a check for artifactXP/xpForNextPoint being less than 0 or greater than 1.

in a wierd little way i do that here

Lua Code:

if not HasArtifactEquipped() then self:Hide() return end

Quote:

Originally Posted by Uitat (Post 325058)

in a wierd little way i do that here

Lua Code:

if not HasArtifactEquipped() then self:Hide() return end

I actually have that before the code I quoted which is just getting the current AP level plus pointsAvailable if you have "filled the bar" multiple times before using the forge, which is quite easy with new 110s (or atl least was pre 7.3)