arkayenro / arkinventory

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

[BUG] reputations from headers not getting displayed #1990

Open amethystnw opened 2 months ago

amethystnw commented 2 months ago

What version number you are using? ArkInventory-3.11.00-alpha-6 August 20, 2024

What game client are you playing? Retail, Wrath, Classic / Live, PTR, Beta Retail Versioin 11.0.2 - 8/21/2024

What language is the game client set to? English

Describe the bug On logon to the game on initial load, Bug Sack & Bug Grabber reports an error as follows:

6x ArkInventory/Core/ArkInventoryLDB.lua:396: bad argument #2 to 'format' (string expected, got nil) string "=[C]": in function format' [string "@ArkInventory/Core/ArkInventoryLDB.lua"]:396: in functionUpdate' [string "@ArkInventory/Core/ArkInventoryLDB.lua"]:365: in function <ArkInventory/Core/ArkInventoryLDB.lua:361>

[string "@Ace3/AceBucket-3.0-4/AceBucket-3.0.lua"]:69: in function <Ace3/AceBucket-3.0/AceBucket-3.0.lua:67> [string "@Ace3/AceBucket-3.0-4/AceBucket-3.0.lua"]:83: in function `func' [string "@Ace3/AceTimer-3.0-17/AceTimer-3.0.lua"]:57: in function <Ace3/AceTimer-3.0/AceTimer-3.0.lua:50>

Locals: (temporary) = "|T%s:0|t %s" (temporary) = nil (temporary) = "Dragonscale Expedition: Unknown" (temporary) = "string expected, got nil"

To Reproduce Steps to reproduce the behaviour, if possible:
Logon to game.

Screenshots If applicable, add screenshots to help explain your problem.
Application running bag inventory otherwise appears to run correctly. This appears to be related to reputation specifically for Dragonscale Expedition. When I open the listing for reputation and do a search for Dragonscale Expedition it is not found, nor is it listed when I scroll through the list (see first image below).

Ark Inventory Error - Reputation

The image below shows the menu listing for LBD, and Dragonscale Expedition does list.

Ark Inventory Error - Reputation - LBD Screen

Additional context Add any other context about the problem here.

arkayenro commented 2 months ago

its a header with rep now so i'll need to update the code to support that

arkayenro commented 2 months ago

will be fixed in the next alpha