TFS 0.X Check CAP + free space slots in backpack

[fyalhed]

I can check if player got cap to get the items by just check getPlayerFreeCap

        -- dont have cap
        if getPlayerFreeCap(cid) < TILE.needcap then
            doTeleportThing(cid, fromPosition, false)
            doSendMagicEffect(position, CONST_ME_MAGIC_BLUE)
            doPlayerPopupFYI(cid, 'You need '.. TILE.needcap ..' OZ CAP to recive the items!')
            return false
        end
        for i = 1, #TILE.extraItems do
            doPlayerAddItem(cid, TILE.extraItems[i].itemID, TILE.extraItems[i].count)
        end
        setPlayerStorageValue(cid, item.actionid, 1)

But sometimes players has the cap, but don't have space, slots in backpack to recive the items

How to check if player has the free slots enough?

 

[fyalhed]

There is any function to check free slots?

 

[fyalhed]

when i say free slots its empty slots on backpack

 

[Roddet]

I don't know if there's an existing function to check that but since i was bored i created a new one

It will search for containers on your equipment slots and then it will check all the free slots on those containers (including containers inside the containers, until there is nothing to check)

It will also check your equipment slots as 'free space' if there is no container on them (RIGHT-LEFT HAND SLOT, AMMO SLOT)

Add this function to your libs/050-functions.lua

function getPlayerFreeSlots(cid) -- by Musztang
    local slots = 0
    local containers = {}
    for s = 3, 10 do -- CONST_SLOT_BACKPACK to CONST_SLOT_AMMO
        local item = getPlayerSlotItem(cid, s)
        if item.itemid > 0 then
            if isContainer(item.uid) then
                table.insert(containers, item.uid)
            end
        else
            if s == 5 or s == 6 or s == 10 then
                slots = slots + 1
            end
        end
    end
    while #containers > 0 do
        for i = (getContainerSize(containers[1]) - 1), 0, -1 do
            local it = getContainerItem(containers[1], i)
            if isContainer(it.uid) then
                table.insert(containers, it.uid)
            end
        end
        slots = slots + (getContainerCap(containers[1]) - getContainerSize(containers[1]))
        table.remove(containers, 1)
    end
    return slots
end

Now on your script, you can simply need to do a check before giving the items:

if getPlayerFreeSlots(cid) < #TILE.extraitems then
    return doPlayerPopupFYI(cid, 'You dont have enough space to carry this items.') and false
end

@TOPIC ASIDE
Also if someone is interested in a function to separately check containers, you can use this function.

function getFreeSlotsFromContainer(item) -- by Musztang
    local slots = 0
    local containers = {}
    if isContainer(item) then
        table.insert(containers, item)
    end
    while #containers > 0 do
        for i = (getContainerSize(containers[1]) - 1), 0, -1 do
            local it = getContainerItem(containers[1], i)
            if isContainer(it.uid) then
                table.insert(containers, it.uid)
            end
        end
        slots = slots + (getContainerCap(containers[1]) - getContainerSize(containers[1]))
        table.remove(containers, 1)
    end
    return slots
end

Use: getFreeSlotsFromContainer(container.uid)

 

[Niebieski]

Why dont you just use simple if doPlayerAddItem... ?

 

[Xikini]

Why dont you just use simple if doPlayerAddItem... ?

Depending on the situation, it's not a good idea to allow items to drop onto the floor.

Kill boss -> get special loot.. no guarantee that the player has cap or space.
Cash shop -> again, can't let that drop on the floor

Most one-time quests from chests it's fine, but some are too 'important' to let drop.
Or if an npc is giving a quest item, et cetera.

but the most important part..
Is having the functionality to do either or.

 

[Niebieski]

Depending on the situation, it's not a good idea to allow items to drop onto the floor.

Kill boss -> get special loot.. no guarantee that the player has cap or space.
Cash shop -> again, can't let that drop on the floor

Most one-time quests from chests it's fine, but some are too 'important' to let drop.
Or if an npc is giving a quest item, et cetera.

but the most important part..
Is having the functionality to do either or.

doPlayerAddItem(cid, itemid[, count/subtype = 1[, canDropOnMap = true[, slot = 0]]])

if doPlayerAddItem(cid, itemid, count, false) == RETURNVALUE_NOERROR then
    -- do something
else
    -- no space or cap
    return true
end

 

[Xikini]

doPlayerAddItem(cid, itemid[, count/subtype = 1[, canDropOnMap = true[, slot = 0]]])

if doPlayerAddItem(cid, itemid, count, false) == RETURNVALUE_NOERROR then
    -- do something
else
    -- no space or cap
    return true
end

Okay, now do that with 23 items.

 

[Roddet]

doPlayerAddItem(cid, itemid[, count/subtype = 1[, canDropOnMap = true[, slot = 0]]])

if doPlayerAddItem(cid, itemid, count, false) == RETURNVALUE_NOERROR then
    -- do something
else
    -- no space or cap
    return true
end

With that if you have 2 slots free and the array has 5 items, you will only get 2 items and u wont get the rest. Thats why i made the function to prevent that.

 

[Niebieski]

With that if you have 2 slots free and the array has 5 items, you will only get 2 items and u wont get the rest. Thats why i made the function to prevent that.

Im sorry, havent actually realized he wanted to do that in loop still you may do the same with adding virtual items to some backpack and gives player backpack with items inside

 

[Roddet]

Im sorry, havent actually realized he wanted to do that in loop still you may do the same with adding virtual items to some backpack and gives player backpack with items inside

Yeah, i though about that aswell but i was bored Still, i believe the function can be useful.

 

[fyalhed]

I don't know if there's an existing function to check that but since i was bored i created a new one

It will search for containers on your equipment slots and then it will check all the free slots on those containers (including containers inside the containers, until there is nothing to check)

It will also check your equipment slots as 'free space' if there is no container on them (RIGHT-LEFT HAND SLOT, AMMO SLOT)

Add this function to your libs/050-functions.lua

function getPlayerFreeSlots(cid) -- by Musztang
    local slots = 0
    local containers = {}
    for s = 3, 10 do -- CONST_SLOT_BACKPACK to CONST_SLOT_AMMO
        local item = getPlayerSlotItem(cid, s)
        if item.itemid > 0 then
            if isContainer(item.uid) then
                table.insert(containers, item.uid)
            end
        else
            if s == 5 or s == 6 or s == 10 then
                slots = slots + 1
            end
        end
    end
    while #containers > 0 do
        for i = (getContainerSize(containers[1]) - 1), 0, -1 do
            local it = getContainerItem(containers[1], i)
            if isContainer(it.uid) then
                table.insert(containers, it.uid)
            end
        end
        slots = slots + (getContainerCap(containers[1]) - getContainerSize(containers[1]))
        table.remove(containers, 1)
    end
    return slots
end

Now on your script, you can simply need to do a check before giving the items:

if getPlayerFreeSlots(cid) < #TILE.extraitems then
    return doPlayerPopupFYI(cid, 'You dont have enough space to carry this items.') and false
end

@TOPIC ASIDE
Also if someone is interested in a function to separately check containers, you can use this function.

function getFreeSlotsFromContainer(item) -- by Musztang
    local slots = 0
    local containers = {}
    if isContainer(item) then
        table.insert(containers, item)
    end
    while #containers > 0 do
        for i = (getContainerSize(containers[1]) - 1), 0, -1 do
            local it = getContainerItem(containers[1], i)
            if isContainer(it.uid) then
                table.insert(containers, it.uid)
            end
        end
        slots = slots + (getContainerCap(containers[1]) - getContainerSize(containers[1]))
        table.remove(containers, 1)
    end
    return slots
end

Use: getFreeSlotsFromContainer(container.uid)

thank you!!!

 

[Xikini]

Im sorry, havent actually realized he wanted to do that in loop still you may do the same with adding virtual items to some backpack and gives player backpack with items inside

Like this?
Give player infinite items from table, with 1 small function. | OTLand