arkayenro / arkinventory

A World of Warcraft Inventory mod for Retail, Burning Crusade, and Classic
108 stars 14 forks source link

[BUG] 3.10.33-alpha-4 induces error in Titan Panel #1902

Closed hollo6 closed 3 months ago

hollo6 commented 3 months ago

What version number you are using? 3.10.33-alpha-4

What game client are you playing? Retail, Wrath, Classic / Live, PTR, Beta Retail

What language is the game client set to? enGB

Describe the bug Since updating to this version, getting the following error on login/reload: [09:33:48] Titan: Error: 'Failed_to_Register' 'ArkInventory_Tracking_Bronze' '' 'Interface/AddOns/Titan/TitanLDB.lua:600: Unsupported LDB type 'hidden''

To Reproduce

  1. Install ArkInventory.
  2. Install Titan Panel.
  3. log in / reload interface

Screenshots N/A

Additional context I don't know if this would still happen with both addons' all default settings, but I can send mine once I know which ones are relevant to this issue.

arkayenro commented 3 months ago

the LDB API states that if the data type cannot be handled it is to be ignored. i set my LDB objects to "hidden" if they cant be displayed in that game variant, eg the currency object is hidden in classic. the bronze object is hidden in retail.

does everything still work even with that error? if it doesnt then titan panel needs to be fixed so that it doesnt generate a hard error.

chocolate bar (which is what i use) does not have any issues with not displaying "hidden" data types, it properly ignores them.

hollo6 commented 3 months ago

It seems to still work, but I have no clue what problems it can cause or what I can't see at a glance. Posted a comment on their page, hoping for a reply.

hollo6 commented 3 months ago

They removed the printout and it seems to not cause any issues.