Number of used bag slots?

[Haleth]

Simple question. Is there any way to get the number of used bag slots (the number of bags you are actually using)? I've been looking for this for ages but I can't find it. Currently I have to change this number manually for every char I have and it's a bit annoying.

[xConStruct]

Hmm, maybe checking against bag size, like:

local function getBagCount()
    for i=1, 4 do
        if GetContainerNumSlots(i) == 0 then
            return i-1
        end
    end
    return i
end

Just thinking if it is possible to have a free slot between two bags - then this method wouldn't work, but then you'd need to fetch their actual id's and not just the count.

Where do you need it?

[Seerah]

If you want to include the backpack, you'll have to do

for i = 0, 4 do

Then for each bag, use GetContainerNumSlots(i) - GetContainerFreeSlots(i)

[Haleth]

Well I'm trying to define 'NUMBAGS' as the number of bag slots, meaning if NUMBAGS = 1 then I only have the default bagpack, if NUMBAGS = 5 then I have the maximum amount of bags possible.

I've tried what you both said and I came up with this.

if GetContainerNumSlots(4) == 0 then
	if GetContainerNumSlots(3) == 0 then
		if GetContainerNumSlots(2) == 0 then
			if GetContainerNumSlots(1) == 0 then
				NUMBAGS = 1
			else
				NUMBAGS = 2
			end
		else
			NUMBAGS = 3
		end
	else
		NUMBAGS = 4
	end
else
	NUMBAGS = 5
end

Would this work out properly? It seems to work on one of my chars with the maximum amount of bags, as well as on a char with just the default bag.

Edit: It doesn't seem to work all the time. Strange. When I use it after login, it'll always return 1, when I use it after a reload it works fine. Maybe I need to delay it.

Edit 2: Added a login check now and it seems to work! .. most of the time.

Edit 3: It doesn't work when I log out, switch chars and log back in. It does work when I log out and log back in on the same char... This is so annoying.

[hairy_palms]

just one question. are you firing it from a PLAYER_ENTERING_WORLD or PLAYER_LOGIN event?

[Haleth]

Both actually, to make sure.

function CheckSlots()
if GetContainerNumSlots(4) == 0 then
	if GetContainerNumSlots(3) == 0 then
		if GetContainerNumSlots(2) == 0 then
			if GetContainerNumSlots(1) == 0 then
				NUMBAGS = 1
			else
				NUMBAGS = 2
			end
		else
			NUMBAGS = 3
		end
	else
		NUMBAGS = 4
	end
else
	NUMBAGS = 5
end
end

local f = CreateFrame("Frame")
f:SetScript("OnEvent", CheckSlots)
f:RegisterEvent("PLAYER_LOGIN")
f:RegisterEvent("PLAYER_ENTERING_WORLD")

CheckSlots()

[xConStruct]

On login (I don't know in which order), the bags will be loaded separately. They fire a BAG_UPDATE with their bagID as arg1. You probably want to register for this one.

My for-loop should do the same thing you posted but without the nested if's.

local count = getBagCount() -- returns 0 for only backpack.
local count = getBagCount() -- returns 4 for all bags.

If you want to include the backpack in the count, just add a + 1 for each id. Since the backpack is available every time, we don't need to check for it.

[Haleth]

Registering the BAG_UPDATE did indeed do the trick.

Thanks a lot.

Edit: No, it still messes up sometime... I really don't get it now.

[Vrul]

This is the function I use for that info:

--[[-----------------------------------------------------------------------------
Find out what the current bag situation is.

ignoreBank	t/f		- Ignore bank bags if true

Returns:
numSlots	#		- Number of bag slots
numEquiped	#		- Number of bag slots that have bags in them
numOpen		#		- Number of bags currently open
-------------------------------------------------------------------------------]]
local function GetNumBags(ignoreBank)
	local numEquiped, numSlots = 0, NUM_BAG_SLOTS + (not ignoreBank and BankFrame:IsShown() and NUM_BANKBAGSLOTS or 0)
	for i = 0, numSlots do
		if GetContainerNumSlots(i) > 0 then
			numEquiped = numEquiped + 1
		end
	end
	return numSlots + 1, numEquiped, #ContainerFrame1.bags - (IsBagOpen(KEYRING_CONTAINER) and 1 or 0)
end