WowAce

to show up the path i must open the artifact window twice. i am using the latest version 1.4.2

The following classes have 1 or more traits that each rank (1-6) have different spell IDs:

DeathKnight:
Blood (Vampiric Fangs)
Unholy (Runic Tattoos)
Monk:
Brewmaster (Hot Blooded, Dark Side of the Moon)
Paladin:
Protection (Faith's Armor, Righteous Crusader)
Rogue:
Subtlety (Catlike Reflexes)
Shaman:
Restoration (Pull of the Sea)

To correct this issue, I have modified ArtifactPathFinder.lua to support the same artifact trait having multiple (alternate) spell IDs (found here: https://pastebin.com/WHBsUBV0 - lines 87-96).
I then had a look through all artifacts and 3-point traits, and placed all the different spell IDs into PathData.lua (found here: https://pastebin.com/Hd93CUpr ).
I may or may not have missed some, so if you use it, please continue to post bug reports.

It's my first time working with lua, so hopefully it works right for you all.

To the developer:
This modification works by having a lua table (array) as an element in the artifact's path for the affected traits, instead of having a spell ID.
Then I added a section in CreateIndexesForPowers() to check if the element is a table, and if so, loop through the contents of that table and pick the spell ID that exists. That spellID will then be the one used for that index.

Feel free to incorporate these fixes into your addon :)

The following classes have 1 or more traits that each rank (1-6) have different spell IDs:

DeathKnight:

Blood (Vampiric Fangs)

Unholy (Runic Tattoos)

Monk:

Brewmaster (Hot Blooded, Dark Side of the Moon)

Paladin:

Protection (Faith's Armor, Righteous Crusader)

Rogue:

Subtlety (Catlike Reflexes)

Shaman:

Restoration (Pull of the Sea)

To correct this issue, I have modified ArtifactPathFinder.lua to support the same artifact trait having multiple (alternate) spell IDs (found here: https://pastebin.com/WHBsUBV0 - lines 87-96).

I then had a look through all artifacts and 3-point traits, and placed all the different spell IDs into PathData.lua (found here: https://pastebin.com/Hd93CUpr ).

I may or may not have missed some, so if you use it, please continue to post bug reports.

It's my first time working with lua, so hopefully it works right for you all. I have only tested it on my protection paladin, but should work for the other classes.

To the developer:

This modification works by having a lua table (array) as an element in the artifact's path for the affected traits, instead of having a spell ID.
Then I added a section in CreateIndexesForPowers() to check if the element is a table, and if so, loop through the contents of that table and pick the spell ID that exists. That spellID will then be the one used for that index.

Feel free to incorporate these fixes into your addon :)

Featurerequest

It was very nice, to see the totally obtained artifactpower right to the current power.