Attempting to get player spec for addon if condition

[Joker119]

So I have an UI addon that creates it's own Paladin Holy Power bar.

Since Only Ret pallies have it, logic would dictate that I should only display it to Ret pallies!!
Here's my problem.

I determine the player's class (this part works)

cfg.playerclass = select(2,UnitClass("player"))

in the holypower portion of the addon...

if cfg.playerclass == "PALADIN" and self.cfg.holypower.show then
bars.createHolyPowerBar(self)
end

that part works.
however, when I determine the players spec:

cfg.playerspec = GetSpecialization()

and try to work that into the Holy Power creation conditions:

if cfg.playerclass == "PALADIN" and cfg.playerspec == "3" then
bars.createHolyPowerBar(self)
end

then it breaks.
No errors.
Just breaks.

If I type the the following into the games chat:

/script print(GetSpecialization())

I get a result of 1 for Holy, 2 for Prot, 3 for Ret.

I've tried the code with all 3 specs and with the "cfg.playerspec == "3" condition, the holy power bar will not display in any pally spec.

[syncrow]

-- Preset you Variable
if GetSpecialization() then
    cfg.playerspec = select(1, GetSpecializationInfo(GetSpecialization()))
end

-- Query
if cfg.playerclass == "PALADIN" and cfg.playerspec == "70" then
    bars.createHolyPowerBar(self)
end

Note:

• Be aware to call the preset every time, you change your spec.
• Here is a list of specIDs

[Joker119]

Originally Posted by syncrow -- Preset you Variable if GetSpecialization() then     cfg.playerspec = select(1, GetSpecializationInfo(GetSpecialization())) end -- Query if cfg.playerclass == "PALADIN" and cfg.playerspec == "70" then     bars.createHolyPowerBar(self) end Note: Be aware to call the preset every time, you change your spec. Here is a list of specIDs

Thank you, I will try this method aswell, though I was able to solve this by forcing it to check

if (GetSpecialization() == 3) then
 bar:Show()

in the visibility check (that also checks for if the player is in a vehicle).

[Seerah]

The problem is that you were comparing to "3", not to 3. String ~= number

[myrroddin]

Also, you do not need to use select if the first return is all that interests you.

Both of these are identical in function, by way of example. The second adds an unnecessary function/table lookup.

local player_name = GetFullName("player")
local player_name = select(1, GetFullName("player"))