Help with my own addon, need some LUA guidance.

[Kraiven]

I'm attempting to write my first addon while trying to learn LUA. I'm having trouble finding the exact API data to move forward with my code. I'd like to write it myself, but I'm wondering if someone can point me in the right direction with some better examples. Currently I'm trying to make a very simple code that when you enter a LFG, it compares your active spec(tank, healer, damage) to your Queued selected role(tank, healer, damage), and if they don't match, prints a message in chat saying you are in the wrong spec. Some of the "get(functions)" I've been looking at in the WoW API are incomplete for data and explanations. Any pointers?

Here's my current codewhich I'm sorry if it's noobish, but I'm currently learning Python)

-- Spec Check load on Instance
local f = CreateFrame("Frame")
local role = UnitGroupRolesAssigned("player")
local isAssigned = GetPartyAssignment("assignment", "player")

f:RegisterEvent(PLAYER_ROLES_ASSIGNED);
--f:RegisterEvent(ending event);


-- Check Active Role Vs. Assigned Role
f:SetScript("OnEvent", function(self, event, ...)
	if event == "PLAYER_ROLES_ASSIGNED" and role ~= isassigned then
		print("Your current Talent Spec is set Wrong, switch specs!")
	end

end)

Thank you for any tips, I'm just doing this for fun.

[Mazzop]

you comparing 2 different things
http://wowprogramming.com/docs/api/U...pRolesAssigned
http://wowprogramming.com/docs/api/GetPartyAssignment

try with http://wow.gamepedia.com/API_GetSpecializationRoleByID

[Phanx]

Also, you're only checking roles once when your addon first loads during login. At that point, role information is probably not available yet, even if you're reloading the UI and already had a role assigned in your group. Calling a function and assigning its return value to a variable sets the value of that variable to whatever was returned by the function at that time. If the function would return a different value at some other time, it doesn't matter -- your variable still contains the original value. If you want to get the new value, you have to call the function again.

[SDPhantom]

Here's applying the fixes suggested by the other posters. I'm not sure if PLAYER_ROLES_ASSIGNED is the event you're looking for. I don't know enough about your intentions to suggest one.

local f = CreateFrame("Frame")
f:RegisterEvent("PLAYER_ROLES_ASSIGNED");
 
f:SetScript("OnEvent", function(self, event, ...)
    if event == "PLAYER_ROLES_ASSIGNED" then
        local assigned = UnitGroupRolesAssigned("player")-- Role selected in LFD or assigned in raid
        local current = GetSpecializationRoleByID(GetSpecialization())--    Role from specialization
        if assigned ~= current then
            print("Your current Talent Spec is set Wrong, switch specs!")
        end
    end
end)

[Kraiven]

Thanks for the help. After an all day attempt to get the functions to work right I was finally able to get my results. One more question, is there a way to get the "currentSpecName" to show up in a pop up message window? I got it to work under print(blah blah), but not in the message(blah blah).

if role ~= specRole then
	playSound()
	message("|cffFF4500You are in the wrong Specialization!|r \n \n |cffffff00Change your Current Specialization before Combat!")
	print("|cff00ccffYour current Specialization:|r", currentSpecName)
end
if role == specRole then
	print("|cff00ccffYour current Specialization:|r", currentSpecName)
end

Edit: I added a little comment to my addon thanking all of you. If you would like me to leave you out just let me know. Thank you all again.

[Phanx]

print automatically concatenates multiple arguments together with a space. message (and most other functions) do not -- you have to do the concatenation yourself:

local name = UnitName("player")
print("Hello", name) <-- shows "Hello Kraiven"
message("Hello", name) <-- probably shows just "Hello"
message("Hello " .. name) <-- now it shows "Hello Kraiven"

[Kraiven]

Originally Posted by Phanx message("Hello " .. name) <-- now it shows "Hello Kraiven"

Worked perfectly, Seems I need to do some more tutorials. Thanks for the quick tips.