Attempt to call global a nil value?

[AeroMaxxD]

Would anyone happen to know why self would be a nil value?

This has been working perfectly fine for a few days, but I must have broken it somehow, I stripped the code right back to try and hung down what I broke, but I still get the error.

1x [string "*OnloadTest.xml:25_OnLoad"]:1: attempt to call global 'AMD_OnLoad' (a nil value)
[string "*OnloadTest.xml:25_OnLoad"]:1: in function <[string "*OnloadTest.xml:25_OnLoad"]:1>

Locals:
self = AMD_TabOnCharacterFrame {
 0 = <userdata>
}
(*temporary) = nil
(*temporary) = AMD_TabOnCharacterFrame {
 0 = <userdata>
}
(*temporary) = "attempt to call global 'DGT_OnLoad' (a nil value)"

local TabName="AMD";
local TabID=CharacterFrame.numTabs+1;
local Tab=CreateFrame("Button", "$parentTab"..TabID, CharacterFrame, "CharacterFrameTabTemplate", TabID);
PanelTemplates_SetNumTabs(CharacterFrame, TabID);
Tab:SetPoint("LEFT", "$parentTab"..(TabID-1), "RIGHT", -16, 0);
Tab:SetText(TabName);
-- Tab:SetID(TabID);
 
tinsert(CHARACTERFRAME_SUBFRAMES, "AMD_TabOnCharacterFrame");
 
hooksecurefunc("CharacterFrameTab_OnClick", function(self, button)
    if self:GetID() == TabID then
        ToggleCharacter("AMD_TabOnCharacterFrame")
    end
end)
 
function AMD_OnLoad(self)
    print("AddOn loaded")
end

<Ui xmlns="http://www.blizzard.com/wow/ui/" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.blizzard.com/wow/ui/ http://wowprogramming.com/FrameXML/UI.xsd">
	<Frame name="AMD_TabOnCharacterFrame" parent="CharacterFrame" frameStrata="HIGH" enableMouse="true" movable="true" setallpoints="true" id="4">
		<Size x="206" y="580"/>
		<Frames>
			<Frame name="$parent_History" parent="AMD_TabOnCharacterFrame">
				<Size x="300" y="325"/>
				<Anchors>
					<Anchor point="CENTER" relativePoint="CENTER" relativeTo="CharacterFrame"/>
				</Anchors>
				<Layers>
					<Layer level="OVERLAY">
						<FontString name="History_Title" parentKey="History_title" inherits="GameFontNormal">
							<Color r="1" g="1" b="1" a="1"/>
							<Anchors>
								<Anchor point="TOP">
									<Offset x="0" y="20"/>
								</Anchor>
							</Anchors>
						</FontString>
					</Layer>
				</Layers>
			</Frame>
		</Frames>
		<Scripts>
			<OnLoad>
				AMD_OnLoad(self)
			</OnLoad>
		</Scripts>
	</Frame>
</Ui>

[Fizzlemizz]

It has to do with the way the files are ordered in the .toc (again)

We changed it to XML first so the .lua code could "see" AMD_TabOnCharacterFrame

Now, when it's created, you're asking AMD_TabOnCharacterFrame to call a function in your .lua (which hasn't been loaded yet)

Instead of using an OnLoad script in the XML just do whatever you were going to do to AMD_TabOnCharacterFrame in the .lua.

hooksecurefunc("CharacterFrameTab_OnClick", function(self, button)
    if self:GetID() == TabID then
        ToggleCharacter("AMD_TabOnCharacterFrame")
    end
end)
--[[ 
function AMD_OnLoad(self)
    print("AddOn loaded")
end
]]--
 
-- Instead of the AMD_OnLoad function:
AMD_TabOnCharacterFrame_NextButton:SetText("OnLoad Change")

Depending on what the end goal is there might be a better alternative but this should work for now.

[AeroMaxxD]

Actually, it's ok I remember what I had changed, I put the XML file before the LUA file in the TOC file.

[AeroMaxxD]

I'm not sure I understood that.

What is "OnLoad Change" is that the name of a function?

I'm getting the error "attempt to call method 'SetText' (a nil value)".

I'm using this code however, slightly different from yours.

AMD_TabOnCharacterFrame:SetText("OnLoad Change")

Originally Posted by Fizzlemizz AMD_TabOnCharacterFrame_NextButton:SetText("OnLoad Change")

[Fizzlemizz]

Originally Posted by AeroMaxxD I'm not sure I understood that. What is "OnLoad Change" is that the name of a function?

Just changing the text of the NextButton to show something happend after the frame was created (what you "might" have previously done in your OnLoad function)

Originally Posted by AeroMaxxD I'm getting the error "attempt to call method 'SetText' (a nil value)". I'm using this code however, slightly different from yours. AMD_TabOnCharacterFrame:SetText("OnLoad Change")

Because AMD_TabOnCharacterFrame doesn't have/isn't a FontString which is why I used the AMD_TabOnCharacterFrame_NextButton but I might have the name wrong or maybe you've removed the button...

I don't know what your code looks once you've made changes so I'm just guessing what might work.

[AeroMaxxD]

Ah ok sure I understand now, yeah I removed the button and pretty much everything as I was trying to figure out what I had broken.

If what was in my Onload function was a script that prints out a table into the frame can I not reference an onload function in lua?

[Fizzlemizz]

Your OnLoad function was global so it can be called pretty much anywhere. The only proviso being that the file containing the function has to be loaded before the function is used (same goes for within in the same .lua file, a function (variable etc.) must be defined before it is used.)

MyFunction() -- error
function MyFunction()
    print("This is MY function!")
end

function MyFunction()
    print("This is MY function!")
end
MyFunction() -- no error

[AeroMaxxD]

Oh sorry what I meant was how to change this:

<Scripts>
	<OnLoad>
		AMD_OnLoad(self)
		self:Hide()
	</OnLoad>
</Scripts>

into lua code?

Like maybe

AMD_TabOnCharacterFrame:OnLoad(AMD_OnLoad(self));

[Fizzlemizz]

You don't need

AMD_TabOnCharacterFrame:OnLoad(AMD_OnLoad(self));

but if you wanted to add a method to the frame you could use a Mixin or

function AMD_TabOnCharacterFrame:OnLoad()
    print(self:GetName(), "I'm loaded now!")
end
 
AMD_TabOnCharacterFrame:OnLoad()

Adding an OnLoad function in lua to AMD_TabOnCharacterFrame doesn't really make sense in your case as it's already been loaded so, just using:

print(AMD_TabOnCharacterFrame:GetName(), "I'm loaded now!")

instead of adding the function and making the call would do the same.

[AeroMaxxD]

Oh but my onload function updates some stuff.

function AMD_OnLoad(self)
    print("AMD: AddOn loaded")
 
    AMD_TabOnCharacterFrame:RegisterEvent("PLAYER_ENTERING_WORLD");
    AMD_TabOnCharacterFrame:RegisterEvent("VARIABLES_LOADED");
 
    AMD_TabOnCharacterFrame:SetScript(
        "OnEvent", function(self, event, ...)
            print("DEBUG: Event:", event); -- For debugging
 
            if (event == "VARIABLES_LOADED") then
                if AMD_CharacterSavedVariables == nil then
                    print("AMD_CharacterSavedVariables : ", AMD_CharacterSavedVariables );
 
                    -- No Saves for this character found
                    AMD_CharacterSavedVariables = {};
                end
 
                if AMD_SavedVariables == nil then
                    print("AMD_SavedVariables: ", AMD_SavedVariables);
 
                    -- No Saves found
                    AMD_SavedVariables = {};
 
                    AMD_OpagueBackgroundCheckbox:SetChecked(true);
                    AMD_DisplayFormatIconRadioButton:SetChecked(true);
                else
                    print("AMD_SavedVariables: ", AMD_SavedVariables);
 
                    AMD_OpaqueBackgroundCheckbox:SetChecked(AMD_SavedVariables["OpaqueBackground"]);
                    if (AMD_SavedVariables["DisplayFormat"] == true) then
                        AMD_DisplayFormatIconRadioButton:SetChecked(true);
                        AMD_DisplayFormatTextRadioButton:SetChecked(false);
                    else
                        AMD_DisplayFormatIconRadioButton:SetChecked(false);
                        AMD_DisplayFormatTextRadioButton:SetChecked(true);
                    end
                end
 
                if (AMD_OpaqueBackgroundCheckbox:GetChecked() == true) then
                    AMD_TabOnCharacterFrame:EnableDrawLayer("BORDER");
                else
                    AMD_TabOnCharacterFrame:DisableDrawLayer("BORDER");
                end
            end
 
            if (event == "PLAYER_ENTERING_WORLD") or (event == "PLAYER_MONEY") then
                AMD_UpdateScreen(event);
            end
        end
    )
end