cloudbells / fieldguide Goto Github PK

I get the following Lua error when I scroll to the bottom of the priest pane:

This only happens when "Talents" is selected. Perhaps a missing texture for one of the talented abilities? Though all of the abilities are there so maybe something else is trying to show but can't?

This also happens at the end (with talents selected) for Paladins but no other pane seems to bring up this Lua error.

Message: Interface\AddOns\FieldGuide\FieldGuide.lua:185: bad argument #1 to 'ipairs' (table expected, got nil)
Time: Sat Jun 22 07:19:11 2019
Count: 1
Stack: Interface\AddOns\FieldGuide\FieldGuide.lua:185: bad argument #1 to 'ipairs' (table expected, got nil)
[C]: in function ipairs' Interface\AddOns\FieldGuide\FieldGuide.lua:185: in function <Interface\AddOns\FieldGuide\FieldGuide.lua:171> Interface\AddOns\FieldGuide\FieldGuide.lua:410: in function <Interface\AddOns\FieldGuide\FieldGuide.lua:402> Interface\AddOns\FieldGuide\FieldGuide.lua:543: in function FieldGuide_OnEvent'
[string ":OnEvent"]:1: in function <[string ":OnEvent"]:1>

Locals: (*temporary) = nil
(*temporary) = "table expected, got nil"
= defined =[C]:-1

Not working with bcc

Not sure what is causing this bug, but it only occurs when a spell row is hidden (i.e. all spells for e.g. level 32 are learned the row will be skipped).

This may be related to issue #2 but I get the following Lua error:

This happens when I toggle the "known spells" on the weapons pane. As an aside, I also know those weapons on my Priest but are shown when the known spells are unchecked. Probably related to said Lua error.

Please set up the project on curseforge.com so it can be installed via Twitch. It must be possible to use this GitHub repository as the source and to set up automated builds of new versions.

Right now you can toggle the horde abilities on priest but as a dwarf priest I still see the human / night elf abilities.

This is because they are simply not tracked in any way in the game. I have yet to find a function for knowing whether or not the player has learned Fist Weapons (the skill simply shows up as Unarmed in the skills tab and when using Fist Weapons Unarmed will be leveled up).

This will have to be tracked some other way.

Mail and Plate spells are technically not spells, but rather skills in the character pane.

Proper usage of GetNumSkillLines() and GetSkillLineInfo() should suffice. Perhaps rename

This is because this is a skill and when learning them from the trainer, they are treated as profession skills.

The above is not the correct behaviour. The correct behaviour would be if "Level 60" did not show and simply the "Level 40" along with the Mail skill (ignoring the fact that I have actually learned it already).

This is because pet skills and demon spells are not learned as spells by either the pet or the player (with Growl being the only exception). They are part of the Craft UI.

Is it possible to hide empty level brackets? It's a bit annoying to scroll past a bunch of empty rows. They can come back when Known Spells is available of course but when it's unchecked id like to see just the abilities I have left to learn.

Such as First Aid (when do I learn Heavy Mageweave Bandage and where do I get it?) or Enchanting (which recipes have I learned and which do I still have to get?) etc.

Bear Form will show up as learnable again once Dire Bear Form is learned.