-
Notifications
You must be signed in to change notification settings - Fork 112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
7.2 issues #169
Comments
4 people in my guild are reporting that their Keystone is not showing up. |
Thanks for pointing that out. My keystone is also not showing up. |
I too am having an issue where Keystones do not show up in my bags. |
Items are also appearing in junk slots, as well as in the stack of empty slots. |
This also appears to be affecting visibility of artifact power items that I have received from world quests, as well as recipes. Finally got around to getting the suramar surf and turf today, and the "turf" portion was not showing up, then after combining them, the completed recipe was not showing up. Hopefully can get a fix for this one. I don't know if I can handle not having adibags, lol. |
I had peddler screwing up on account of item link changes or something, and I'm not sure I've seen any issues since fixing peddler - in getUniqueItemId, replace the local bonuses = ... line with local bonuses = ""
if split[14] then
bonuses = split[14]:match("^[^|]*")
end |
Can confirm everything said above -- keystone, artifact power, random items not showing up -- some of it is fixed by doing "Sort" then "Reset New" but sometimes that just ends up shuffling the items around without changing the icon shown in Adibags -- I just popped a Defiled Augment Rune without reading the tooltip of what looked like the LW recipe I just picked up. |
i think it's time for me to move to another bag addon, seems like this one is not been maintained for a while :( |
Would love an update for this for 7.2. This addon completes my WoW experience. |
Is there anyone willing to work on this code if the current dev is unable or unwilling? |
itemLinks for Mythic keystones have changed. They no longer use the standard item string and now use a mythic keystone string. |
I am also experiencing this issue, especially when equipping/unequipping gear. I recently got a legendary ring, and attempted to equip it (not realizing I already had 2 legendaries equipped). This caused the ring to vanish from my inventory. I disabled Adibags and it was in the vanilla bags. |
legowxelab2z8's comment fixed it for me. This is what I did. Go into AdiBags\core\Utility.lua and open in notepad++ go to line 152 and replace
with
|
Got a similar issue with food recipe which doesn't show up... |
Tpittma2, will this fix all of this issues? I appreciate your support but are there any other issues? I know people were having other items missing (Including the AK26 quest item). |
The keystone fix looks like the battlepet special case, and there are 4 places in the code where battlepets are treated specially (3 in core/Utility.lua and 1 in modules/FilterOverride.lua). Should there be similar changes in the other 3 places? |
@seathasky I would put money on their still being issues. I'm sure Tpittma2 did some on the fly debugging, so there is likely to be more issues. If the dev is not interested to maintaining this any further, we need someone else to get deeply familiar with the codebase and item API or I'm afraid this addon will not be able to continue. |
@SpareSimian Those special cases for battlepets probably also apply for Mythic Keystones as well. I updated my pull request to add those. @seathasky, @cj-zeiger There is likely still an issue that is causing some items to not be shown in adibags. |
@legowxelab2z8 Could you also look into why this item was not showing up in anywhere in the bag? You only ever have it for a short time, but its pretty important to use. I needed to disable adi bags on all my alts just to use it as it wasn't showing up anywhere in the bags. |
Someone commented that a "workaround" for Mythic Keystones not appearing was to use the item sort, and close and reopen your bags. I played around with it and found that it doesn't work 100% of the time. I discovered it only works when the Keystone changes position in your inventory. It then inherits the item information of the item it swapped spots with. Hopefully patching function containerProto:UpdateContent(bag) in widgets\ContainerFrame.lua to use GetItemInfo(itemId) when GetItemInfo(link) fails allows the slotData cache to correctly update and might resolve the issue with other items as well. |
@cj-zeiger I also had a problem with the Enchanted Elven Tome now showing up in AdiBags, but I no longer have one to do further debugging with. I don't notice anything special about the item that should be causing issues with AdiBags but further debugging is certainly needed if this item is always not shown in AdiBags. |
If you're having trouble with an item and you know how to get access to it, post the item ID. I'm betting there's a pattern to the missing items. (Is there a way to get it from Wowhead?) |
Ok, on Wowhead, there's a Links button at top right of an item page, and the link for the Enchanted Elven Tome look like this: "\124cffffffff\124Hitem:147449::::::::110:::::\124h[Enchanted Elven Tome]\124h\124r" It has the "item:" pattern so should function, but maybe the other fields are causing problems somewhere. |
Enchanted Elven Tome ID 147449 is the quest item. Empowered Elven Tome ID 147860 is the item you get for completing the quest. The Empowered Elven Tome seems to be a normal item with nothing special about it that should cause issues. |
Looks like the author is still around, his other addons are being updated, lets hope this one does too. |
Anybody who's having trouble seeing the AK26 Elven Tome, open up your artifact weapon and hover over the number in the top-left -- if the tooltip says Artifact Knowledge 26 it's already been applied. I did the quest on six different characters yesterday and on two of them I didn't get or have to use the item, it just applied automatically -- doesn't seem to be related to this addon. |
I had similar issues with a [Pile of Herbs] not showing up, as well as a stack of [Unbending Potion]. It almost seemed like they were being positioned behind other item slots. |
Vantus Rune: Krosus is also not showing in bags if Adibags is enabled. |
AdiDevs pleaseeeeeeee fix the best bag addon to ever be created! |
If you experience issues with the addon, follow tpittma2's advice from above, looks like it fixed things for now. Items displaying correctly. |
I have been on a scavenger hunt tracking down and acquiring the items listed as having issues. Empowered Elven Tome both Recipe: Surf and Recipe: Turf and also Unbending Potion. They all show up in my bags and display fine. One interesting thing I noticed was that they are all filtered as Consumable. I created a script to scan my bags and find the corresponding button in AdiBags and report any items that are missing buttons. There is a bug with the script when it tries to get buttons for full stacks of items. The function containerProto:FindExistingButton in widgets\ContainerFrame.lua that the script uses to get the AdiBags button will remove any buttons that have a full stack and then return nil, so there are some false positives, but it should report any missing items. While playing and getting all of the above items I did not have any instances of missing items. At this point I believe that there isn't a bug for specific items, but maybe a bug for consumables since all the items reported in this issue are consumables. Please continue to report if you notice item missing from your bags. TL;DR Despite my efforts I cannot reproduce this bug. Please report if this bug is still happening. |
Thanks for the update Lego, you're my hero! |
I noticed after the patch today that the quest reward http://www.wowhead.com/item=147860/empowered-elven-tome was not appearing in AdiBags. I had to turnoff the addon to consume the item.
Also, I do not get a Head equipment slot section nor are head items displayed when equipment is in one section.
The text was updated successfully, but these errors were encountered: