arkayenro / arkinventory

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

[BUG] Loading UI while mounted causes search results to display currencies as broken item links at the top #1869

Open elasota opened 4 months ago

elasota commented 4 months ago

What version number you are using? v31024

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

What language is the game client set to? English

Describe the bug Loading the UI while mounted causes the search results to display all currencies as a bunch of broken item links at the start of the search results. The results display even if the broken item links don't match the search text at all.

To Reproduce

  1. Go to Argus (not sure if this is necessary)
  2. Get on a mount
  3. Enter /reload in the console
  4. Open inventory
  5. Click the magnifying class at the top and wait for search results to populate

Screenshots https://i.imgur.com/04U0G7S.png

Additional context I have tested this with all add-ons other than ArkInventory disabled.

I am not sure that mounting causes this issue, but I've been able to reproduce it 100% of the time of I reload the UI while mounted, and it never happens if I do it while not mounted.

I believe this also happens if you log into a character that is already mounted but haven't completely tested that.

arkayenro commented 4 months ago

try 3.10.24 alpha 2

elasota commented 4 months ago

Correction: This still happens in alpha 2, but instead of the items appearing at the top of the search results, they appear in the middle of the search results as "Item is not ready yet"

https://i.imgur.com/Tv5V10N.png

If I reload UI while not mounted, then the "Item is not ready yet" items appear briefly but then turn into currency results. If I reload while mounted, the items don't resolve, and the text left of the search bar stays at "Loading..."

The problem fixes itself in alpha 2 if I close the search window, get off of my mount, and open the search window again.

arkayenro commented 4 months ago

sorry, yes, just realised that its deliberate. dismount and it should update.

3.10.19 (25-NOV-2023)