BUG: Drawer unavailable items not vanishing

[MacOSX4ever]

I have it set to not show items, such as potions, food, or other consumables not show up, when you're out.
This is working for many items, but it i not working for:

211879, Algari Healing Potion
224442, Weavercloth Bandage
224112, Oil of Deep Toxins

again, many other food and potion items, as well as permanent buff items, that I have on one toon, but not another, all seem to work.
When I log in, on another toon, they do not display.

But the above particular items show, even when I don't have them. They show as grayed out, not available, but they are not hidden.

[Peterodox]

Hi! Did you enable "Update Frequently"? Otherwise, consumable items will not automatically become hidden when they run out.

[MacOSX4ever]

Yes, that is not the issue. It hides some things and not others.
I think it may be an oversight by design, for consumables.

The problem is, it hides things I don't have on this toon, like a buff crystal, but it doesn't hide food and potion items I don't have.
This may have been oversight on design, thinking people could see the items they ran out of, but what it really does is show a lot of junk items you will never have. For example a healer, or different levels will use totally different potions, food, bandages.

What it probably needs for a permanent fix, is to add a preference flag, like "hideUsable" does, but for missing items "hideMissing".
for now, I fixed it, and tied it to the same "hideUsable" flag, until this functionality is added.

This works fine for now:

Stop showing items I do not have:
Look for: “if id and DoesItemReallyExist(id) then” right around line 1148 in the file: Macros.lua

if id and DoesItemReallyExist(id) then name = GetItemNameByID(id); icon = GetItemIconByID(id); macroText = format("/use \"item:%d\"", id); if GetItemCount(id) == 0 then if hideUnusable then -- Use the local parameter passed to the function usable = false; end else usable = true; end craftingQuality = GetItemCraftingQuality(id); if name and craftingQuality then name = format("%s T%s", name, craftingQuality); --name = format("%s|A:Professions-ChatIcon-Quality-Tier%s:0:0|a", name, craftingQuality); --local markup = CreateAtlasMarkupWithAtlasSize("Professions-ChatIcon-Quality-Tier"..craftingQuality, 0, 0, nil, nil, nil, 0.5) --name = name..markup end end

I added that if... then... block in the middle.

Anyone doing this will have to disable auto-updates to prevent it from being overwritten, until it's fixed in the master addon.

[MacOSX4ever]

Idk why it looks messed up when I posted that, maybe it's normal, or a bug in the forum?
So, here it is again, just in case. Trying without the code block formatting.

if id and DoesItemReallyExist(id) then
name = GetItemNameByID(id);
icon = GetItemIconByID(id);
macroText = format("/use "item:%d"", id);
if GetItemCount(id) == 0 then
if hideUnusable then -- Use the local parameter passed to the function
usable = false;
end
else
usable = true;
end
craftingQuality = GetItemCraftingQuality(id);
if name and craftingQuality then
name = format("%s T%s", name, craftingQuality);
--name = format("%s|A:Professions-ChatIcon-Quality-Tier%s:0:0|a", name, craftingQuality);
--local markup = CreateAtlasMarkupWithAtlasSize("Professions-ChatIcon-Quality-Tier"..craftingQuality, 0, 0, nil, nil, nil, 0.5)
--name = name..markup
end
end

[Peterodox]

Hi. Could you give me an itemID for the buff crystal?