Centias / BankItems

Keep track of you inventory, bank, reagant bank, void storage and more across all your characters. Fork of the original BankItems on Curse/WowAce. Credit to XinHuan/Burstroc and anyone else who worked on the original.
26 stars 2 forks source link

Ranks Showing Now, But Nothing Else Is #15

Closed Lixialitixa closed 1 year ago

Lixialitixa commented 1 year ago

I was looking forward to the most recent update for the different ranks of items to show. However, that's the ONLY thing that's showing in the mouseovers. If it's not an item that has a rank, it's not showing who else has it, or even if you have more than one.

I love this addon and I'm addicted to it! Yes, I made an account just so I could submit this issue.

Centias commented 1 year ago

So you aren't seeing the breakdown like this for basic items like old potions? image

Lixialitixa commented 1 year ago

That's correct. I can mouse over any item with a rank 1, 2, or 3 denoted by the copper diamond, 2 silver diamonds, or 3 gold diamonds in the icon's corner, and it will show how many are on each of my characters except the character I am currently on (ex. LWing reagent Adamant Scales or consumable Refreshing Healing Potions.) That's kind of annoying also, as I would like to know if I have some in the bank or in the mail for a total amount on a given character. It shows how many I have on my alts, but not the character currently logged in.

And nothing else I mouse over has that information. That includes equipment, reagents, consumables, anything: (ex. rousing earth or Trailblazer's Scale Bracers or the potions you show above) and it's the same in my mailbox, bank, inventory, and the auction house.

EDIT: I found one item in my inventory displaying the tooltip correctly that doesn't have a rank: the Sturdy Expedition Shovel. It shows 1 in bags, 5 in mail on the char I'm logged in on, and how many other chars have too.

Please let me know if you need screenshots. Also I should add, all of this is true even if BankItems is the only addon enabled so I don't think it's an addon conflict.

Centias commented 1 year ago

So it's both A ) not showing for items with no quality/rank and B ) not showing the current character in the list at all?

I'm not really sure where to start troubleshooting this. If it's generating the tooltips for items with different qualities, it has to at least be getting the item ID. If it has that, it should be getting the rest too. But it sounds like for some reason something is going wrong in getting the current character's saved info for anything else, then just shows nothing.

Are you getting any kind of error messages? I would basically expect some kind of error about ' BankItems_SelfCache '

Lixialitixa commented 1 year ago

I read your comment and the cache part got me thinking. I removed all the BankItems files I could find and reinstalled BankItems and have been repopulating my database of characters. It helped a ton! I don't seem to be having those particular issues anymore.

However, I am having another issue, dang it. On every character, things seem to be working fine, except for the first bank bag (the farthest most left one). It doesn't show up, in any character's BankItems, and I can't search for anything that's in that bag.

Here are my bank bag slots:

Screen Shot 2023-04-27 at 5 47 24 PM

And here is the BankItems version:

Screen Shot 2023-04-27 at 5 48 04 PM

Edit: I was trying to avoid having to repopulate my data for BankItems as it's a pain with so many characters, which is why I didn't try the reinstall in the first place. But if you weren't sure where to start with that, I figured I should run with it.

Centias commented 1 year ago

Oh hey, what do you know. I guess I've had data saved the whole time and didn't realize it wouldn't "create" that slot when saving character data for the first time.

Looks like I had +2 where I needed a +1. Changing this line for bagNum = ITEM_INVENTORY_BANK_BAG_OFFSET+2, ITEM_INVENTORY_BANK_BAG_OFFSET+1+NUM_BANKBAGSLOTS do

to this for bagNum = ITEM_INVENTORY_BANK_BAG_OFFSET+1, ITEM_INVENTORY_BANK_BAG_OFFSET+1+NUM_BANKBAGSLOTS do

will fix the issue. I'll put out an actual update when I'm done with dungeons for the night.

Lixialitixa commented 1 year ago

Great, thank you very much! The only 2 things I've noticed that are still not showing up are hearthstones (and whatever, don't really care about those) and any caged pets.