MilleXIV / XIV_Databar

19 stars 19 forks source link

Errors with Specialization Modules in 2.03 #42

Closed Vegeir closed 7 years ago

Vegeir commented 7 years ago

Using Version 2.03 as 100 Fire Mage with bar set to TOP position. 2 errors related to Spec. Module:

1. Error: Left-clicking on spec immediately caused this occur:

12x XIV_Databar\modules\talent-Talent.lua:296: attempt to index field 'color' (a nil value)
XIV_Databar\modules\talent-Talent.lua:296: in function <XIV_Databar\modules\talent.lua:295>
Locals:
(*temporary) = <function> defined =[C]:-1
(*temporary) = <unnamed> {
0 = <userdata>
}
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index field 'color' (a nil value)"
buttonText = <unnamed> {
0 = <userdata>
}
db = <table> {
general = <table> {
}
modules = <table> {
}
}

2. Error: Right-clicking on Spec button (to select loot choice) also causes errors. It looks like the same function being called at a different line number:

14x XIV_Databar\modules\talent-Talent.lua:390: attempt to index field 'color' (a nil value)
XIV_Databar\modules\talent-Talent.lua:390: in function <XIV_Databar\modules\talent.lua:389>
Locals:
(*temporary) = <function> defined =[C]:-1
(*temporary) = <unnamed> {
0 = <userdata>
}
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index field 'color' (a nil value)"
buttonText = <unnamed> {
0 = <userdata>
}
db = <table> {
general = <table> {
}
modules = <table> {
}
}
Vegeir commented 7 years ago

I tested this again on a 100 Warlock. By default, a new bar profile was created when I logged in for the firs time after installing 2.03. Here are my observations:

Hope this helps. Let me know if I can test further.

Vegeir commented 7 years ago

Here is a visual for how the bar performs when initially loaded on a toon which hasn't used it yet. This is the default profile on my 101 Paladin. It is set to BOTTOM by default and the "drop down" menu for specialization grow upwards as expected. The opacity / alpha looks what I would expect it to. If the bar is moved to TOP and then back to BOTTOM, the problem shown in my second screen capture above reoccurs.

Original settings: pally_2016-09-02_2215

After bring moved to TOP and then back to BOTTOM: pally_2016-09-02_2219

Vegeir commented 7 years ago

TL;DR: The error only occurs the first time you move the bar and is only present until after re-logging.

Was on my 100 warrior and noticed that after changing to the profile I have set (which has the bar on top) and re-logging that the error did not reoccur. I switched back to my paladin to confirm and noticed that I could switch spec / loot spec as intended. The alpha / opacity looks good again as well.

Here is a pic: pally_speclooksgood

MilleXIV commented 7 years ago

Yeah okay, I've figured out what the positioning error is. Short explanation: it was still keeping the old positioning anchors when the bar was moved. Should be fixed in c7891cfac3245104c03d1e39a19ed2ac64f28e9e.

Mikeprod commented 7 years ago

I guess this is solved. Please @Vegeir close it for more issue visibility here of what is left to do. Thanks