Well what im trying to do is... i have a chunk of code that deals with moving frames. It used to just build a table when the game loaded and those were your moveable frames. well... im trying to make things so that i can add/remove frames to that table from in game and have it save my additions/removels per session. I added another savevar to the .toc called GMoveableFrames. I tried to switch everything from that table to the saved variables but the big problem is on log out it only writes half of the frames it should to the saved variable file so on the next load up it does not load all the frames it should to the frame mover. I know this code might be a little confusing with out posting some of the other lua's in the addon but im pretty sure all important parts are there.
Code:
local addonName, addon = ...
addon:RegisterDefaultSetting("lockFrames", false)
--[[-----------------------------------------------------------------------------
Frames to allow moving and saving
-------------------------------------------------------------------------------]]
--["FrameName"] = false - move the frame
--true - move the frame's parent instead
--string - move the named frame instead
function addon:DefaultMoveableFrames()
GMoveableFrames = {
['MiniMapLFGFrame'] = false,
['MiniMapBattlefieldFrame'] = false,
['ShardBarFrame'] = false,
['BNToastFrame'] = false,
['RuneFrame'] = false,
['SpellBookFrame'] = false,
['QuestLogFrame'] = false,
['FriendsFrame'] = false,
['LFGParentFrame'] = false,
['KnowledgeBaseFrame'] = true,
['MerchantFrame'] = false,
['MailFrame'] = false,
['DressUpFrame'] = false,
['TaxiFrame'] = false,
['QuestLogFrame'] = false,
['PaperDollFrame'] = true,
['PVPFrame'] = false,
['WatchFrameHeader'] = true,
['VehicleMenuBar'] = false,
['InspectFrame'] = false,
['PlayerTalentFrame'] = false,
['AchievementFrame'] = false,
['AchievementFrameHeader'] = true,
['AchievementFrameCategoriesContainer'] = 'AchievementFrame',
['GlyphFrame'] = 'PlayerTalentFrame',
['CalendarFrame'] = false,
['GrimUIPlayerFrame'] = false,
['GrimUIPartyFrame1'] = false,
['GrimUIPartyFrame2'] = false,
['GrimUIPartyFrame3'] = false,
['GrimUIPartyFrame4'] = false,
['GrimExpBar'] = false,
['GrimRepBar'] = false,
['GDevBar'] = false,
}
for id = 1, NUM_CONTAINER_FRAMES do
GMoveableFrames['ContainerFrame' .. id] = false
end
end
--[[-----------------------------------------------------------------------------
Frame hooking
-------------------------------------------------------------------------------]]
local hookedFrame, parentFrame, movedFrames, oldPoint, oldX, oldY = { }, { }
local function GetRelativePosition(frame)
local _, _, uiWidth, uiHeight = UIParent:GetRect()
local left, bottom, width, height = frame:GetRect()
if bottom + height / 2 >= uiHeight / 2 then
if left + width / 2 >= uiWidth / 2 then
return 'TOPRIGHT', left + width - uiWidth - 1, bottom + height - uiHeight - 1
else
return 'TOPLEFT', left, bottom + height - uiHeight - 1
end
elseif left + width / 2 >= uiWidth / 2 then
return 'BOTTOMRIGHT', left + width - uiWidth - 1, bottom
end
return 'BOTTOMLEFT', left, bottom
end
local function OnShow(self)
local frame = parentFrame[self] or self
local position = movedFrames[frame:GetName()]
if position then
addon:UnlockFrame(frame)
frame:ClearAllPoints()
frame:SetPoint(unpack(position))
addon:LockFrame(frame)
end
end
local function OnMouseDown(self, button)
if button ~= 'LeftButton' or addon.settings.lockFrames then return end
if IsControlKeyDown() and button == 'LeftButton' then
local frame = parentFrame[self] or self
oldPoint, oldX, oldY = GetRelativePosition(frame)
addon:UnlockFrame(frame)
frame:StartMoving()
end
end
local function OnMouseUp(self, button)
if button ~= 'LeftButton' or not oldPoint then return end
local frame = parentFrame[self] or self
frame:StopMovingOrSizing()
addon:LockFrame(frame)
local point, x, y = GetRelativePosition(frame)
if point ~= oldPoint or x ~= oldX or y ~= oldY then
movedFrames[frame:GetName()] = { point, 'UIParent', x, y }
end
oldPoint, oldX, oldY = nil, nil, nil
end
local HookFrame
function HookFrame(name, parent)
if hookedFrame[name] then return true end
local frame = _G[name]
if not frame then return end
if parent then
if type(parent) == 'string' then
parent = _G[parent]
else
parent = frame:GetParent()
end
if not parent then return end
HookFrame(parent:GetName())
while(parentFrame[parent]) do
parent = parentFrame[parent]
end
parentFrame[frame] = parent
elseif not name:match("^" .. addonName) then
frame:HookScript('OnShow', OnShow)
end
frame:EnableMouse(true)
frame:EnableKeyboard(true)
frame:SetMovable(true)
frame:SetClampedToScreen(false)
frame:HookScript('OnMouseDown', OnMouseDown)
frame:HookScript('OnMouseUp', OnMouseUp)
hookedFrame[name] = true
if movedFrames[name] and frame:IsShown() then
OnShow(frame)
end
return true
end
--[[-----------------------------------------------------------------------------
Initialize
-------------------------------------------------------------------------------]]
addon.RegisterEvent("MoveFrames-Initialize", 'PLAYER_LOGIN', function(self, event)
addon.UnregisterEvent(self, event)
movedFrames = addon.settings.movedFrames
if type(movedFrames) ~= 'table' then
movedFrames = { }
addon.settings.movedFrames = movedFrames
end
addon:RegisterDefaultSetting("movedFrames", true) -- Prevent it from being removed on PLAYER_LOGOUT
local function HookFrames(self, event)
if not GMoveableFrames then
addon:DefaultMoveableFrames()
end
for name, parent in pairs(GMoveableFrames) do
if HookFrame(name, parent) then
GMoveableFrames[name] = nil
end
end
if next(GMoveableFrames) then return end
addon.UnregisterEvent(self, event)
end
addon.RegisterEvent("MoveFrames-Hook", 'ADDON_LOADED', HookFrames)
HookFrames("MoveFrames-Hook", 'ADDON_LOADED')
end)
--[[-----------------------------------------------------------------------------
in game add remove functions
-------------------------------------------------------------------------------]]
function addon:AddMoveableFrame()
end
function addon:RemMoveableFrame()
end
what em i doing wrong?
Hybrid Mode
