tukui-org / ElvUI

User Interface replacement AddOn for World of Warcraft.
https://tukui.org
Other
396 stars 136 forks source link

Enchants don't show in character info or inspect #1231

Closed collinstevens closed 2 weeks ago

collinstevens commented 2 weeks ago

Troubleshooting Steps

image image

Describe the expected behavior and what actually happened?

The item level, gems, and rarity of each item in Character Info and Inspect are displaying, but the enchants aren't displaying.

The tooltip for Display Inspect Info is Shows item level of each item, enchants, and gems when inspecting another player. The tooltip for Display Character Info is Shows item level of each item, enchants, and gems on the character page..

Reproducing the issue.

  1. Enable Display Inspect Info and Display Character Info in General > BlizzUI Improvements > Item Level.
  2. Open Character Info or Inspect on another player.

Lua Errors

N/A

Verification Steps

Repooc commented 2 weeks ago

Correct, they do not show due to how wow displays them in the tooltip. The only thing you can get regarding enchants on an item, is an enchant id from the item link, but wow has no api to use that id to get the actual enchant text. In retail we scan the tooltip to get the enchant as the enchants are prefixed in the tooltip with Enchanted:, that is not the case in cata... so no enchant will show... since we do not plan on maintaining a database with enchants and their enchantid which is the only reliable way to display the enchant string in cata.

I have a plugin that I am in middle of reworking but has a working alpha version on curseforge and wago addons named cataarmory that can display the enchant info since I have a list of enchants and the enchant ids...

collinstevens commented 2 weeks ago

Correct, they do not show due to how wow displays them in the tooltip. The only thing you can get regarding enchants on an item, is an enchant id from the item link, but wow has no api to use that id to get the actual enchant text. In retail we scan the tooltip to get the enchant as the enchants are prefixed in the tooltip with Enchanted:, that is not the case in cata... so no enchant will show... since we do not plan on maintaining a database with enchants and their enchantid which is the only reliable way to display the enchant string in cata.

I have a plugin that I am in middle of reworking but has a working alpha version on curseforge and wago addons named cataarmory that can display the enchant info since I have a list of enchants and the enchant ids...

Ok, I'll check out your addon.

I was using BetterCharacterPanel as a substitute and it looks like they are just doing some checking on the enchant text for how they display it.