Trouble making slash commands

[lordkarthas]

This is relevant segment of the code I am using. There are no errors, and the message prints to the chat window. The slash commands just don't work (I get the type /help message).

function npcIgnore_onLoad()
	SlashCmdList["NPCIGNORE"] = npcIgnore_ChatCommand;
	SLASH_NPCIGNORE1 = "/npcignore";
	SLASH_NPCIGNORE2 = "/npci";
	ChatFrame_AddMessageEventFilter("CHAT_MSG_MONSTER_SAY", npcChatFilter);
	ChatFrame_AddMessageEventFilter("CHAT_MSG_MONSTER_WHISPER", npcChatFilter);
	ChatFrame_AddMessageEventFilter("CHAT_MSG_MONSTER_EMOTE", npcChatFilter);
	DEFAULT_CHAT_FRAME:AddMessage("NPCIgnore is loaded.");
end

Yes, the function does exist.

[Maul]

Originally Posted by lordkarthas Yes, the function does exist.

Where in relation to that function does it exist? Before or after? If it is a local function, it needs to be before that function in your post.

local function npcIgnore_ChatCommand()

end


^
^
^
function npcIgnore_onLoad()

end

[lordkarthas]

Originally Posted by Maul Where in relation to that function does it exist? Before or after? If it is a local function, it needs to be before that function in your post. local function npcIgnore_ChatCommand() end ^ ^ ^ function npcIgnore_onLoad() end

It is before. It does have the signature "function npcIgnore_chatCommand(msg)", if that makes any difference.

EDIT: the placement shouldn't matter, since the function would be added anyway before the OnLoad event fired.

[Cirk]

Do you see the "NPCIgnore is loaded" message Lordkarthas? (i.e., is the addon actually loading properly - Buggrabber + Bugsack will help there too).

The other thing you can try is inside the game, do /run ChatFrame1:AddMessage(tostring(SlashCmdList["NPCIGNORE"])) and see if it has a valid entry (should print something like "function:#####").

-- Cirk

[lordkarthas]

Originally Posted by Cirk Do you see the "NPCIgnore is loaded" message Lordkarthas? (i.e., is the addon actually loading properly - Buggrabber + Bugsack will help there too). The other thing you can try is inside the game, do /run ChatFrame1:AddMessage(tostring(SlashCmdList["NPCIGNORE"])) and see if it has a valid entry (should print something like "function:#####"). -- Cirk

The "NPCIgnore is loaded." message appears, there are no errors, and the script you provided returns nil.

[Cirk]

Originally Posted by lordkarthas The "NPCIgnore is loaded." message appears, there are no errors, and the script you provided returns nil.

Hmmm, that means that as far as the SlashCmdList is concerned, you don't have anything registered there fore "NPCIGNORE".

So, options:
1. Using the previous script, print out the values of npcIgnore_ChatCommand, SLASH_NPCIGNORE1, SLASH_NPCIGNORE2. (i.e., look and see if any of those got foobar'd).
2. Try with something like

SlashCmdList["NPCIGNORE"] = function(text) npcIgnore_ChatCommand(text) end;

Which should at least get you a valid function in the SlashCmdList, and then you can try and debug from there.

Hope that helps

[lordkarthas]

#2 helped. I was looking for npcIgnore_ChatCommand, but the function was named npcIgnore_chatCommand. Stupid case-sensitivity. Thanks for your help.

[Maul]

Originally Posted by lordkarthas EDIT: the placement shouldn't matter, since the function would be added anyway before the OnLoad event fired.

Actually, for local functions/values it does matter