Skillet-Classic

Skillet-Classic

445k Downloads

Merchant Alternate Currency error

matthewhively opened this issue ยท 4 comments

commented

Version 1.37.alpha9

This happened when I opened the <keeper of the sha'tari heirlooms>
Not sure what else would cause it.

Message: ...terface\AddOns\Skillet-Classic\UI\MerchantWindow.lua:75: attempt to concatenate a nil value
Time: Sat Oct  9 23:07:04 2021
Count: 1
Stack: ...terface\AddOns\Skillet-Classic\UI\MerchantWindow.lua:75: attempt to concatenate a nil value
[string "=[C]"]: ?
[string "@Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua"]:29: in function <...las\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:25>
[string "@Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua"]:64: in function `Fire'
[string "@Interface\AddOns\Attune\Libs\AceEvent-3.0\AceEvent-3.0.lua"]:120: in function <...ace\AddOns\Attune\Libs\AceEvent-3.0\AceEvent-3.0.lua:119>

Locals: <none>
commented

Aldor or Scryer?

The error is on a debug line and it doesn't happen on the equivalent Scryer vendor. Is it repeatable?

I've changed line 75 in `...\Skillet-Classic\UI\MerchantWindow.lua from:

DA.DEBUG(2,"itemTexture= "..tostring(itemTexture)..", itemValue= "..tostring(itemValue)..", itemLink= "..DA.PLINK(itemLink)..", currencyName= "..tostring(currencyName))

to:

DA.DEBUG(2,"itemTexture= "..tostring(itemTexture)..", itemValue= "..tostring(itemValue)..", currencyName= "..tostring(currencyName))

Currently, there are no crafting recipes that use materials bought with alternate currency in Burning Crusade Classic so this section of code is future proofing.

commented

Doesn't seem like it happens with other vendors

commented

This was Aldor, but I suppose it would happen with any alternative currency vendor.
I suppose this will become more relevant when nethers are purchasable with badges later on.

commented

would happen with any alternative currency vendor.
"Could" is a better choice of words and yes, it would become relevant when nethers are purchasable. At that point, we'll get to test the rest of the code that is in place.