FluidFrames bug

[Imithat]

Hey folks,

Ok, i tried MoveAnything (updated to 6.0.2) and its very nice. But MoveAnything find not all Frames. This makes me sick! So i need FluidFrames. But this was bugged. No chance to that was updated by the owner. Got this error:

38x FluidFrames\FrameReparent.lua:75: Attempt to access forbidden object from code tainted by an AddOn
[C]: in function `GetNumPoints'
FluidFrames\FrameReparent.lua:75: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
...
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:55: in function `GetPositionallyDependantOn'
FluidFrames\FrameHide.lua:67: in function `HideAssociatedFrames'
FluidFrames\FrameHide.lua:37: in function `LockHiddenFrame'
FluidFrames\FluidFrames-2.4g.lua:164: in function `InitMovedFrames'
FluidFrames\FluidFrames-2.4g.lua:202: in function <FluidFrames\FluidFrames.lua:173>
 
Locals:
frameSet = <table> {
 (null) = true
}
first = ServicesLogoutPopup {
 0 = <userdata>
 Background = <unnamed> {
 }
}

I cant hide frames. This function doesnt work at this time

Can anybody help me?

(You find FluidFrames here)

[Phanx]

This should be an easy fix, but I'm lazy... can you post the contents of FluidFrames\FrameReparent.lua around line 75 where the error is occuring? Either attach the whole file or just post the ~10 lines above and below in a [code] tag.

[Imithat]

Sure
thanks phanx

Line 73 to 88

--[[-- From a list of frames/regions add the ones that are anchored once only to one of the frames in the frameSet to the frameSet. -- ]]-- 
function FluidFrames.GetAnchoredTo(frameSet, first, ...)
    if first and not frameSet[first] and first:GetNumPoints() == 1 then
        -- TODO: check if it's anchored to a child of the frame
        local relativeFrame = select(2, first:GetPoint(1))
        if frameSet[relativeFrame] then
            frameSet[first] = true
            if select("#", ...) > 0 then
                FluidFrames.GetAnchoredTo(frameSet, ...)
            end
        end
    end
    if select("#", ...) > 0 then
        FluidFrames.GetAnchoredTo(frameSet, ...)
    end
end

Or on Git:
https://github.com/Imithat/FluidFram...meReparent.lua

[Phanx]

This should probably fix it:

if first and not first:IsProtected() and not frameSet[first] and first:GetNumPoints() == 1 then

[Imithat]

get following

1x FluidFrames\FrameReparent.lua:75: Attempt to access forbidden object from code tainted by an AddOn
[C]: in function `IsProtected'
FluidFrames\FrameReparent.lua:75: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
...
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:86: in function `GetAnchoredTo'
FluidFrames\FrameReparent.lua:55: in function `GetPositionallyDependantOn'
FluidFrames\FrameHide.lua:67: in function `HideAssociatedFrames'
FluidFrames\FrameHide.lua:37: in function `LockHiddenFrame'
FluidFrames\FluidFrames-2.4g.lua:164: in function `InitMovedFrames'
FluidFrames\FluidFrames-2.4g.lua:202: in function <FluidFrames\FluidFrames.lua:173>
 
Locals:
frameSet = <table> {
 (null) = true
}
first = ServicesLogoutPopup {
 0 = <userdata>
 Background = <unnamed> {
 }
}

[Phanx]

Alright, you probably need to move the :IsProtected() check further up the food chain. Find where the .GetAnchoredTo function is called, and trace it back to the first place it tries to do anything with a frame, and stop it from doing that if the frame is protected. You can't really break anything by adding more :IsProtected() checks in more places.

Or, just use MoveAnything... I can't imagine there can be that many frames in the default UI it can't handle, and any addon that provides a GUI should also provide some function for moving it.