Item Categories sometimes don't load, all items "Misc"
Stefen74 opened this issue · 23 comments
Hi all,
When I open my bank, I'm often met by this ...
The bank window is out the top of the screen, it can't be moved or reset using the 'N' button. If I press ESC to close it, and then re-open it, this can sometimes solve it, but it can also make it vanish altogether, when it does this I just presume it's now outside the screen somewhere. From here the only way to get it back is to reset the window positions and sizes from the settings.
My only thought is that by bank might be a bit big slot number wise? Other then that I have no idea.
Anyone?
Thank you in advance.
What version of AdiBags and WoW are you on? (Looks like Wrath PTR, but can you confirm?)
Just an idea, but could it be the sheer number of items in the bank? The addon cannot cache them all quick enough to display them correctly, and so it looks like the first picture? And by time you've closed and re-opened it, it the addon has had the time to sort everything and so it can display it correctly?
You'll notice that the first picture only has a few items under a "Armor" heading, the rest (other than a few "Junk" items) are in "Miscellaneous", that's what makes me think the addon cannot handle that many items in such a short time. I'd test it by emptying my bank a bit, but of course, my bags are also nearly full :(
Adibags is version 1.9.26-bcc. So yea, it is the TBC version, I've not seen a wrath version? Or have I not been looking hard enough? :(
It is the Wrath PTR, which is showing version 3.4.0
So, Curse isn't up to date, but there have been a ton of updates, and there's no more "branches." I'm working on getting Curse up to date for non-retail, but if you take the release from GitHub, v1.9.34 (click here to go to it) and install this, you'll have the newest version.
Can you confirm this newest version works?
Perfect! The new version fixes it. Thanks a ton for this, facing the realisation of the default bags UI was giving me nightmares :( ... thank you again :)
Awesome! We'll work on getting Curse updated so those addon clients auto-update. You can also use wowup, etc, to manage updates until we get this done.
Thanks!
Shoot, okay. Unfortunately, the PTR is down for me right now and I can't test. I don't have a high level character to test with on the classic game versions, but I will level something up really fast and get multiple items to test as soon as I can. This will be a priority, as it is breaking for Wrath.
I'll keep playing with it and get back to you, I was convinced it was the number of armor items in the bank, but although I could get it to fail, I have loaded my bank up with over 70 items of armor, and of course, now it won't fall over :( ... It might be a certain item, but more testing, maybe not tonight, but I will keep on it and see if I can get it to fall over on demand :)
Appears related to #252. My characters' banks and bags have always been quite filled up when observing such behavior.
This is happening because item data is pulled asynchronously from the game. This will require us to implement RequestLoadItemData, register for the event, and track what has been loaded. It shouldn't be too hard, but it does add complexity as we need fan out and join pattern to determine when all requests are done.
We'll get this in place after the other Layout bits are done.