Looping through bags + reagent bag

[Walkerbo]

Hi all

I need to loop through all equipped bags including the reagents bag.

After 10.2 changed the way we reach bags and the new reagents bag my original loop was;

for bag = 0, 4 do -- check each bag
    for slot = 0, GetContainerNumSlots(bag) do -- check each bag slot
        local itemID = GetContainerItemID(bag, slot)

This used to loop through all my equipped bags, starting with the backpack(0) and each following bag(1-4)

From reading the new changes it shows that the new reagents bag(-3)
So to test I used;

name = C_Container.GetBagName(-3)
print(name)

This returns nothing, no errors, not even a nil;

So then I tried using the bag constants;

for bag = BACKPACK_CONTAINER, NUM_BAG_SLOTS,BACKPACK_CONTAINER+NUM_BAG_SLOTS+1 do
    for slot = 1, C_Container.GetContainerNumSlots(bag) do
        local itemLink = C_Container.GetContainerItemLink(bag, slot)
        if itemLink then
            print(bag, slot, itemLink)
        end
    end
end

This looped through the backpack only.

If I remove the backpack;

for bag = NUM_BAG_SLOTS,BACKPACK_CONTAINER+NUM_BAG_SLOTS+1 do
   for slot = 1, C_Container.GetContainerNumSlots(bag) do
      local itemLink = C_Container.GetContainerItemLink(bag, slot)
      if itemLink then
         print(bag, slot, itemLink)
      end
   end
end

Which loops through all the bags excluding the backpack.

At this point I am lost, I still have not found a fool, (me being the fool), proof way to loop through all the bags in one sweep.

What am I missing here?
How do I loop through all equipped bags, backpack, bags 1-4 and the reagent bag in one go?

[Kanegasi]

The "numeric for" iteration in Lua has three expressions:

for index = a, b, c do

a is the start
b is the end
c is optional and is how much to increment each iteration (+1 if omitted)

The following are some examples and what index will be:

for index = 1, 9 do
1, 2, 3, 4, 5, 6, 7, 8, 9

for index = 9, 1, -2 do
9, 7, 5, 3, 1

for index = 1, 2, 3 do
1

Notice how the third one only has 1, which is because the increment of 3 tried to go to 4 but that's beyond the max of 2.

BACKPACK_CONTAINER is a constant of 0
NUM_BAG_SLOTS is a constant of 4

Both, along with the new reagent bag of 5 (which doesn't have a direct constant), are defined in FrameXML\Constants.lua. However, there is also the Enum.BagIndex table and a new constant NUM_TOTAL_EQUIPPED_BAG_SLOTS.

So your first attempt is "for i = 0, 4, 5 do", which will only do 0. Removing the 0 in your second attempt ends up with "for i = 4, 5 do" which will go through 4 and 5, 4 being the leftmost "real" bag and 5 is the new reagent bag.

Try the following:

for bag = BACKPACK_CONTAINER, NUM_TOTAL_EQUIPPED_BAG_SLOTS do
    for slot = 1, GetContainerNumSlots(bag) do

And finally, -3 is the reagent bank.

[Xruptor]

Yep they changed the bags with the latest patch. You can find the changes here.

https://wowpedia.fandom.com/wiki/BagID

It should give you constants you need to loop through the bags you want