Old method of frame placement no longer working.

[suicidalkatt]

So, previous in 4.3 I was using a simple hooksecurefunc to "SetPoint" for the DurabilityFrame to allow me to set a custom point for the frame.

For some reason unknown to me this method no longer works and there has been 0 changes to the durability frame.

My Code:

local function UpdatePoint()
    DurabilityFrame:ClearAllPoints()
    DurabilityFrame:SetPoint("TOPRIGHT",Minimap,"BOTTOMLEFT",-20,-20)
end
 
hooksecurefunc(DurabilityFrame,"SetPoint",function(self,_,parent)
    if (parent == "MinimapCluster") or (parent == _G["MinimapCluster"]) then
        UpdatePoint()
    end
end)
 
tinsert(TUSUI.Updaters, UpdatePoint)

Can anyone point me as to what is preventing me from updating the point correctly?

Edit: Just to make this clear, the function UpdatePoint() is getting called and my global table insert is just to make sure it gets called again after my addon is loaded.

[zork]

If you add print() debugs. Does the function get called twice on loadup like you say?
Is the parent element still the MinimapCluster?

If I do

/run print(DurabilityFrame:GetParent():GetName())

I get UIParent. (Which is correct because the parent in FrameXML\DurablityFrame.xml is UIParent)

If I had to do sth like this the first thing I do is to get the latest FrameXML files.
I then use TotalCommander (Windows) and to a text-search (Alt+F7) in the FrameXML folder for the string "DurabilityFrame".

The only occurence of DurabilityFrame:SetPoint is in UIParent.lua.

[suicidalkatt]

Originally Posted by zork If you add print() debugs. Does the function get called twice on loadup like you say? Is the parent element still the MinimapCluster? If I do /run print(DurabilityFrame:GetParent():GetName()) I get UIParent. (Which is correct because the parent in FrameXML\DurablityFrame.xml is UIParent) If I had to do sth like this the first thing I do is to get the latest FrameXML files. I then use TotalCommander (Windows) and to a text-search (Alt+F7) in the FrameXML folder for the string "DurabilityFrame". The only occurence of DurabilityFrame:SetPoint is in UIParent.lua.

Yes it does get called twice on load up with print("DurabilityFrame SetPoint hooked / updated"). Regardless once SetPoint is hooked should it not always update from that point? I don't see how it's getting reparented, I even added SetParent(Minimap) still doesn't matter.

Edit: DurabilityFrame.lua also has a few lines of SetPoint without a parent frame labeled, also Minimap is the parent for the frame in the DurabilityFrame.xml which still seems strange that UIParent is being set.

Edit 2: I have 010 Editor (total boss text / hex editor)

[zork]

In my FrameXML\DurabilityFrame.xml the parent is UIParent.
https://github.com/Ketho/wow-ui-sour...tyFrame.xml#L4

Have you tried:

DurabilityFrame.ignoreFramePositionManager = true

In the XML at the bottom the frame position manager is called "OnShow". Setting the ignore value may help.
Additionally enabling SetUserPlaced(true), SetMovable(true) can help sometimes too.

[suicidalkatt]

Originally Posted by zork In my FrameXML\DurabilityFrame.xml the parent is UIParent. https://github.com/Ketho/wow-ui-sour...tyFrame.xml#L4 Have you tried: DurabilityFrame.ignoreFramePositionManager = true In the XML at the bottom the frame position manager is called "OnShow". Setting the ignore value may help. Additionally enabling SetUserPlaced(true), SetMovable(true) can help sometimes too.

I had no knowledge of the UIParent_ManageFramePosition funciton. TIL.

Still doesn't let me change its point. I'm really clueless at this point as to what is preventing it's placement from changing.

local function UpdatePoint()
    DurabilityFrame.ignoreFramePositionManager = true
    DurabilityFrame:SetMovable(true)
    DurabilityFrame:SetUserPlaced(true)
    DurabilityFrame:ClearAllPoints()
    DurabilityFrame:SetParent(Minimap)
    DurabilityFrame:SetPoint("LEFT",Minimap,"LEFT",-20,-20)
    print("DurabilityFrame Point Updated")
end
 
hooksecurefunc(DurabilityFrame,"SetPoint",function(self,_,parent)
    if (parent == "MinimapCluster") or (parent == _G["MinimapCluster"]) then
        UpdatePoint()
    end
end)
 
tinsert(TUSUI.Updaters, UpdatePoint)

[zork]

This works for me:

local frame = CreateFrame("Frame", "rMM_DurabilityDragFrame", UIParent)
  frame:SetSize(DurabilityFrame:GetWidth(),DurabilityFrame:GetHeight())
  frame:SetPoint("CENTER",0,0)
  --frame:SetPoint(DurabilityFrame:GetPoint())
  --rCreateDragFrame(frame, dragFrameList, -2 , true) --frame, dragFrameList, inset, clamp
  DurabilityFrame:SetParent(frame)
  DurabilityFrame:SetAllPoints(frame)
  hooksecurefunc(DurabilityFrame, "SetPoint", function(...)
    DurabilityFrame:SetAllPoints(frame)
  end)

[suicidalkatt]

Originally Posted by zork This works for me: (snip)

That seems to work for me as well.

Interesting that you'd have to go that route to be able to place it in a different location.

Is there some handling of the Minimap children I'm not aware of?

[zork]

No your problem is the condition inside the hook function. Just print all the ... variables and you will find out why. You don't need another surrounding frame. You just have to fix the condition. Make sure not to get into a loophole.

--hook
hooksecurefunc(DurabilityFrame, "SetPoint", function(...)
  local a, b, c, d, e = ...
  print(a)
  print(b)
  print(c)
  print(d)
  print(e)
  --DurabilityFrame:SetPoint()--make sure to only call this if needed
end)

[suicidalkatt]

Originally Posted by zork No your problem is the condition inside the hook function. Just print all the ... variables and you will find out why. You don't need another surrounding frame. You just have to fix the condition. Make sure not to get into a loophole. -snip

I see now... Makes sense. Thank you for your time Zork I appreciate it

[Phanx]

There seem to be an awful lot of threads popping up recently about moving the DurabilityFrame... why don't you guys just look at addons that already do it successfully (SmoothDurability, for one), instead of trying to reinvent the wheel? It took me about 90 seconds (at most) of looking at its code to figure out how to move the frame...

[suicidalkatt]

Originally Posted by Phanx There seem to be an awful lot of threads popping up recently about moving the DurabilityFrame... why don't you guys just look at addons that already do it successfully (SmoothDurability, for one), instead of trying to reinvent the wheel? It took me about 90 seconds (at most) of looking at its code to figure out how to move the frame...

I wanted to see the reasoning behind my own old method and what was causing the issue. Perhaps reinventing the wheel may not always be the best / fastest solution, but it certainly has helped me learn a few things rather than say "oh that works".

[zork]

Can you post your final code?

[suicidalkatt]

Originally Posted by zork Can you post your final code?

local function UpdatePoint()
    DurabilityFrame.ignoreFramePositionManager = true
    DurabilityFrame:ClearAllPoints()
    DurabilityFrame:SetParent(Minimap)
    DurabilityFrame:SetPoint("TOPRIGHT",Minimap,"BOTTOMLEFT",-20,-20)
end
 
hooksecurefunc(DurabilityFrame,"SetPoint",function(self,_,parent)
    if parent ~= _G["Minimap"] then
        UpdatePoint()
    end
end)
 
tinsert(TUSUI.Updaters, UpdatePoint)

Edit: I had to wrap this into a frame and run the hook after "PLAYER_ENTERING_WORLD" to prevent some strange WoW client crash.