Grr, I hate it when I miss stuff...

[Sythalin]

Getting an error that I don't know how to resolve:

-- SET CITIES --
function MageTaxi:SetFaction()
    if UnitFactionGroup("player") == "Alliance" then
        cityList = {
            [1] = "Stormwind",
            [2] = "Ironforge",
            [3] = "Darnassus",
            [4] = "Exodar",
            [5] = "Theramore",
            [6] = "Shattrath",
            [7] = "Dalaran",
        }
    else
        cityList = {
            [1] = "Undercity",
            [2] = "Orgrimmar",
            [3] = "Thunder Bluff",
            [4] = "Silvermoon",
            [5] = "Stonard",
            [6] = "Shattrath",
            [7] = "Dalaran"
        }
    end;
        dest = cityList[1]
end
 
-- SET DESTINATION --
local function setLoc(i)
    for num= 1,#cityList do
        _G["MageTaxi_City".. num]:SetChecked(false)  -- line 91
    end
    dest = cityList[i]
    _G["MageTaxi_City".. i]:SetChecked(true)
end
 
function MageTaxi_createGUI()
    -- other code
    for i = 1,#cityList do
        cb = CreateFrame("CHECKBUTTON", "MageTaxi_City"..i, f, "UIRadioButtonTemplate")
            if i == 1 then
                cb:SetPoint("TOPLEFT", f, "TOPLEFT", 40, -45)
            else
                cb:SetPoint("TOP", "MageTaxi_City".. (i-1), "BOTTOM", 0, -10)
            end
            cb:SetScript("OnClick", setLoc(i)) -- line 149
        end
end

Message: Interface\AddOns\MageTaxi\magetaxi.lua:91: attempt to index field '?' (a nil value)
Time: 02/09/10 21:08:49
Count: 1
Stack: Interface\AddOns\MageTaxi\magetaxi.lua:91: in function <Interface\AddOns\MageTaxi\magetaxi.lua:89>
Interface\AddOns\MageTaxi\magetaxi.lua:149: in function `MageTaxi_createGUI'
[string "MageTaxi_createGUI()"]:1: in main chunk
[C]: in function `RunScript'

Locals: i = 1
(for index) = 2
(for limit) = 7
(for step) = 1
num = 2
(*temporary) = nil
(*temporary) = nil
(*temporary) = "2"
(*temporary) = "attempt to index field '?' (a nil value)"

As far as the checkbuttons, it's only creating one and stopping (leading to the error, figured that part out). I'm looking for WHY it's not building all the buttons.

[Xrystal]

If you haven't already I would convert the _G["MageTaxi_City".. num] part out to a variable and see what that says each time and if the object of that name exists both there and in CreateGUI.

Apart from that I cannot initially see anything wrong there. Especially with the error reporting values between 1 and 7 with it being stuck at 2. So the question is to find out why MageTax_City2 isn't being created, or thinks it hasn't been created.

Ah, I think I know the problem, at least based on the code there. I had the same problem in my own code recently. When you call a function at creation time that is also used later on after all items are created the chances are it is trying to access something that doesn't exist yet.

Try changing the setloc function to something like this. This way if _G["MageTaxi_City2"] doesn't exist when working on _G["MageTaxi_City1"] it won't kick up a fuss. Of course this assumes you haven't pre-created the objects themselves. But then again #citylist is already of size 7 but does that mean _G["MageTaxi_City2"] exists or that there are blank entries in citylist ?

local function setLoc(i)
  for num= 1,#cityList do
    if _G["MageTaxi_City".. num] then _G["MageTaxi_City".. num]:SetChecked(false) end
  end
  dest = cityList[i]
  _G["MageTaxi_City".. i]:SetChecked(true)
end

[Akryn]

Edit: Oh I should've noticed that too,

[Vrul]

Originally Posted by ChaosInc As far as the checkbuttons, it's only creating one and stopping (leading to the error, figured that part out). I'm looking for WHY it's not building all the buttons.

cb:SetScript("OnClick", setLoc(i))

Was that line meant to be:

cb:SetScript("OnClick", function() setLoc(i) end)

As it is now you are trying to set the "OnClick" script for each button to the return value of setLoc, but in calling setLoc you are attempting to reference frames that haven't been created yet.

Edit:

Instead of creating a separate function for each frame's "OnClick" handler why not just do:

local function setLoc(self)
	for i = 1, #cityList do
		_G["MageTaxi_City" .. i]:SetChecked(false)
	end
	dest = cityList[self.id]
	self:SetChecked(true)
end
 
function MageTaxi_createGUI()
	-- other code
	for i = 1, #cityList do
		cb = CreateFrame("CHECKBUTTON", "MageTaxi_City" .. i, f, "UIRadioButtonTemplate")
		if i == 1 then
			cb:SetPoint("TOPLEFT", f, "TOPLEFT", 40, -45)
		else
			cb:SetPoint("TOP", "MageTaxi_City" .. (i - 1), "BOTTOM", 0, -10)
		end
		cb.id = i
		cb:SetScript("OnClick", setLoc) -- line 149
	end
end

[nightcracker]

The error is in the SetScript on line 149. SetScript accepts 2 parameters, the script and the function that should be called.

cb:SetScript("OnClick", setLoc(i))

This will generate an error This will happen if the onclick handler is called:

setLoc(i)(...)

Errors. This IS correct:

cb:SetScript("OnClick", function(i) setLoc(i) end)

Which will be called like this(function:2FJ39 is the link to the local function created by function(i) setLoc(i) end):

function:2FJ39(...)

And the most neat way is this:

cb:SetScript("OnClick", setLoc)

Which will get called like this:

setLoc(...)

Which is what we want.

Final thing, you are concatenating the i variable in your setLoc function, don't you mean this?

i = i:GetName()

[Sythalin]

Originally Posted by Vrul [code] Edit: Instead of creating a separate function for each frame's "OnClick" handler why not just do: local function setLoc(self) for i = 1, #cityList do _G["MageTaxi_City" .. i]:SetChecked(false) end dest = cityList[self.id] self:SetChecked(true) end function MageTaxi_createGUI() -- other code for i = 1, #cityList do cb = CreateFrame("CHECKBUTTON", "MageTaxi_City" .. i, f, "UIRadioButtonTemplate") if i == 1 then cb:SetPoint("TOPLEFT", f, "TOPLEFT", 40, -45) else cb:SetPoint("TOP", "MageTaxi_City" .. (i - 1), "BOTTOM", 0, -10) end cb.id = i cb:SetScript("OnClick", setLoc) -- line 149 end end

Isn't just passing i doing the same thing, or am I not following this logic correctly?

Final thing, you are concatenating the i variable in your setLoc function, don't you mean this? i = i:GetName()

Why would I want the name of a number?

And the most neat way is this: cb:SetScript("OnClick", setLoc) Which will get called like this: setLoc(...) Which is what we want.

... is still somewhat confusing to me. I understand that it's all arguments that are passed, but cb:SetScript("OnClick", setLoc) isn't passing anything, so how would I get the i value?

Still learning as I go along.

[nightcracker]

Originally Posted by ChaosInc Isn't just passing i doing the same thing, or am I not following this logic correctly? Why would I want the name of a number? ... is still somewhat confusing to me. I understand that it's all arguments that are passed, but cb:SetScript("OnClick", setLoc) isn't passing anything, so how would I get the i value? Still learning as I go along.

NOW I see what you want to do.... Here it comes:

function MageTaxi:SetFaction()
    if UnitFactionGroup("player") == "Alliance" then
        cityList = {
            [1] = "Stormwind",
            [2] = "Ironforge",
            [3] = "Darnassus",
            [4] = "Exodar",
            [5] = "Theramore",
            [6] = "Shattrath",
            [7] = "Dalaran",
        }
    else
        cityList = {
            [1] = "Undercity",
            [2] = "Orgrimmar",
            [3] = "Thunder Bluff",
            [4] = "Silvermoon",
            [5] = "Stonard",
            [6] = "Shattrath",
            [7] = "Dalaran"
        }
    end;
        dest = cityList[1]
end

local function setLoc(i)
    for num= 1,#cityList do
        _G["MageTaxi_City".. num]:SetChecked(false)  -- line 91
    end
    dest = cityList[i]
    _G["MageTaxi_City".. i]:SetChecked(true)
end
 
function MageTaxi_createGUI()
    for i = 1,#cityList do
        cb = CreateFrame("CHECKBUTTON", "MageTaxi_City"..i, f, "UIRadioButtonTemplate")
		if i == 1 then
			cb:SetPoint("TOPLEFT", f, "TOPLEFT", 40, -45)
		else
			cb:SetPoint("TOP", "MageTaxi_City".. (i-1), "BOTTOM", 0, -10)
		end
		cb:SetScript("OnClick", function() setLoc(i) end) -- line 149
    end
end

[Sythalin]

Thanks, I realized the fix last night already, but good to see it verified (and it works btw).

One thing though that I've run into that maybe someone can help me with. I have ZERO understanding of string manipulation and it has me painted into a corner atm.

cityList = {
	[3] = "ThunderBluff",
}

b = CreateFrame("BUTTON", "MageTaxi_Port"..i, f, "SecureActionButtonTemplate")
	b:SetPoint("RIGHT", "MageTaxi_City"..i, "LEFT", 0, 0)
	b:SetHeight(18)
	b:SetWidth(18)
	b.texture = b:CreateTexture("MageTaxi_Spell"..i)
	b.texture:SetAllPoints(b)
	b.texture:SetTexture("Interface\\Icons\\Spell_Arcane_Portal".. cityList[i])
	b:SetAttribute("type", "spell")
	b:SetAttribute("spell", "Portal: ".. cityList[i])
fs = b:CreateFontString("MageTaxi_Text"..i)
	fs:SetFontObject("GameFontGreenSmall")
	fs:SetJustifyH("LEFT")
	s:SetText(cityList[i])
	fs:SetPoint("LEFT", "MageTaxi_City"..i, "RIGHT", 0,0)

Only things that really matter are the orange spots. The issue is with Thunder Bluff. The icon name puts it as "ThunderBluff" (no space), but I still want it to appear as "Thunder Bluff" in the font string. Is there a way to do this without having to assign a temp var for difference?

[nightcracker]

Originally Posted by ChaosInc Thanks, I realized the fix last night already, but good to see it verified (and it works btw). One thing though that I've run into that maybe someone can help me with. I have ZERO understanding of string manipulation and it has me painted into a corner atm. cityList = { [3] = "ThunderBluff", } b = CreateFrame("BUTTON", "MageTaxi_Port"..i, f, "SecureActionButtonTemplate") b:SetPoint("RIGHT", "MageTaxi_City"..i, "LEFT", 0, 0) b:SetHeight(18) b:SetWidth(18) b.texture = b:CreateTexture("MageTaxi_Spell"..i) b.texture:SetAllPoints(b) b.texture:SetTexture("Interface\\Icons\\Spell_Arcane_Portal".. cityList[i]) b:SetAttribute("type", "spell") b:SetAttribute("spell", "Portal: ".. cityList[i]) fs = b:CreateFontString("MageTaxi_Text"..i) fs:SetFontObject("GameFontGreenSmall") fs:SetJustifyH("LEFT") s:SetText(cityList[i]) fs:SetPoint("LEFT", "MageTaxi_City"..i, "RIGHT", 0,0) Only things that really matter are the orange spots. The issue is with Thunder Bluff. The icon name puts it as "ThunderBluff" (no space), but I still want it to appear as "Thunder Bluff" in the font string. Is there a way to do this without having to assign a temp var for difference?

s:SetText(gsub(cityList[i], "([a-z]+)([A-Z][a-z]+)", "%1 %2"))

I really like that I've done my regex homework so I can solve problems like this

[Sythalin]

Originally Posted by nightcracker s:SetText(gsub(cityList[i], "([a-z]+)([A-Z][a-z]+)", "%1 %2")) I really like that I've done my regex homework so I can solve problems like this

Worked like a charm. Now mind explaining it to me? LOL

[ArrchDK]

Originally Posted by ChaosInc Worked like a charm. Now mind explaining it to me? LOL

Info on gsub

The second argument ("([a-z]+)([A-Z][a-z]+)") looks for a lower case letter ([a-z]) followed by an uppercase letter ([A-Z]) followed by a lower case letter ([a-z]). So in the case of ThunderBluff, it would find "rBl" as the pattern. Something like LookingForGroup would find "gFo" and "rGr". It then substitutes it with the third argument ("%1 %2"). In this case, it is substituting the first letter (%1) and the second letter (%2), combined would be (%1%2) with (%1 %2); essentially adding a space inbetween them. So "rBl" becomes "r Bl" in the string, making "ThunderBluff" change to "Thunder Bluff"

I hope that's easier to understand than it is to explain =/

[Akryn]

Originally Posted by ArrchDK Info on gsub The second argument ("([a-z]+)([A-Z][a-z]+)") looks for a lower case letter ([a-z]) followed by an uppercase letter ([A-Z]) followed by a lower case letter ([a-z]). So in the case of ThunderBluff, it would find "rBl" as the pattern. Something like LookingForGroup would find "gFo" and "rGr". It then substitutes it with the third argument ("%1 %2"). In this case, it is substituting the first letter (%1) and the second letter (%2), combined would be (%1%2) with (%1 %2); essentially adding a space inbetween them. So "rBl" becomes "r Bl" in the string, making "ThunderBluff" change to "Thunder Bluff" I hope that's easier to understand than it is to explain =/

That's a better explanation than I could have given.

To be completely accurate, it's searching for a string of lowercase letters of length >=1 both before and after the single capital letter; but it's true that you don't really need that. Removing the + signs would make it behave exactly as described.

As it is, "ThunderBluff" yields one match: "hunderBluff", which in turn matches the subpatterns as "(hunder)(Bluff)" and therefore %1 == "hunder" and %2 == "Bluff", resulting in gsub replacing all instances of "hunderBluff" with "hunder Bluff", with a final result of "Thunder Bluff" since there's only one instance of that.

In any case, I think: "(%l)(%u)" would work just as well; but it doesn't really matter much.

[Vrul]

Originally Posted by ChaosInc Isn't just passing i doing the same thing, or am I not following this logic correctly?

Passing i requires the function setLoc and an additional separate function for each city button. Saving i into the frame allows you to just have the one function shared by all the buttons. It is just an efficiency thing, why create 8 functions to accomplish the same thing you can do with just 1.

[nightcracker]

Originally Posted by Akryn To be completely accurate, it's searching for a string of lowercase letters of length >=1 both before and after the single capital letter; but it's true that you don't really need that. Removing the + signs would make it behave exactly as described.

It's efficienter with the +, because else it's constantly look-aheading while the greedyness of [a-z]+ is getting exactly to the point we want.

[Xrystal]

Originally Posted by nightcracker It's efficienter with the +, because else it's constantly look-aheading while the greedyness of [a-z]+ is getting exactly to the point we want.

Checks dictionary, hmm efficienter isn't in there :P Sorry, couldn't resist

[Torhal]

Bah. You missed "greedyness" in your dictionary check. :P

[Xrystal]

Originally Posted by Torhal Bah. You missed "greedyness" in your dictionary check. :P

d'oh, didn't read that far rofl.

[nightcracker]

:P I'm sorry for my malformed english, only a 15 year old dutch kiddo who just learns english by browsing forums like this one. "efficienter" is the comparative of efficient(which IS a dutch word) in dutch. So I was confused. And greedyness is well..... greedyness! :P

[Sythalin]

Great, glad the thread has become a spelling bee.

[Xrystal]

Originally Posted by nightcracker :P I'm sorry for my malformed english, only a 15 year old dutch kiddo who just learns english by browsing forums like this one. "efficienter" is the comparative of efficient(which IS a dutch word) in dutch. So I was confused. And greedyness is well..... greedyness! :P

ah then this English gal forgives you English has been known to be the worst language to learn

And sorry Chaos, didn't mean to derail your thread somewhat

So, hows your coding going ? ( trying to steer back on topic - rofl ).