Mikeprod / XIV_Databar

A databar for the World of Warcraft UI
https://mods.curse.com/addons/wow/xiv_databar
20 stars 12 forks source link

Lua error with recent gold changes #46

Open Merathilis opened 7 years ago

Merathilis commented 7 years ago

If you check the option: "Always Show Silver and Copper" you get a lua error if you mouseover the Gold Datatext:

3x XIV_Databar\modules\gold.lua:35: Usage: GetCoinText(amount, fontHeight) [C]: in functionGetCoinTextureString' XIV_Databar\modules\gold.lua:35: in function <XIV_Databar\modules\gold.lua:12> XIV_Databar\modules\gold.lua:187: in function <XIV_Databar\modules\gold.lua:174>`

Mikeprod commented 7 years ago

With the latest update I am getting the following error:

14x XIV_Databar\modules\gold.lua:35: Usage: GetCoinText(amount, fontHeight) [C]: in function `GetCoinTextureString' XIV_Databar\modules\gold.lua:35: in function <XIV_Databar\modules\gold.lua:12> XIV_Databar\modules\gold.lua:186: in function <XIV_Databar\modules\gold.lua:174>

Locals: (*temporary) = "410781.99999999997108.0000000016298"

Thanks for a great addon.

Mikeprod commented 7 years ago

I did not have any issue yesterday... Weird.

Mikeprod commented 7 years ago

Hehe I did not test with high enough amount of gold...

Mikeprod commented 7 years ago

Keeping this issue open for a reminder.

TODO : Shorten gold values in strings with texture when the option to shorten the amount is checked.