Lua Help with getItemHoldingCount()

[Fortera Global]

I want to prevent players from placing items when the backpack reaches 5 full slots, this script works to some extent, to cheat is just the player to open the backpack that is inside the other and move the item there, like getTopParent() no working.

tried onMove;

local containerIdDP = toPosition.y - 64
        local containerDP = self:getContainerById(containerIdDP)
        if containerDP then
        local ItemDEE = containerDP:getTopParent()
        if ItemDEE and containerDP:getItemHoldingCount() >= 5 then
            self:sendCancelMessage("NOT POSSIBLE")
            self:getPosition():sendMagicEffect(CONST_ME_POFF)
            return false
        end
        end

TFS 1.2

 

[narko]

What does this produce? lag, kicks?

 

[Itutorial]

Why not change how many slots are in the backpack?

 

[Infernum]

function Player:onMoveItem(item, count, fromPosition, toPosition, fromCylinder, toCylinder)
    if toPosition.x == CONTAINER_POSITION then
        local container = self:getContainerById(toPosition.y - 64)
        if not container then
            if toPosition.y == CONST_SLOT_BACKPACK then
                container = self:getSlotItem(CONST_SLOT_BACKPACK)
            end
        end
        if container then
            local parent = toCylinder:getParent()
            local finalContainer = container
            if parent and parent:isItem() then
                local nextParent = parent:getParent()
                while nextParent and nextParent:isItem() do
                    parent = nextParent
                    nextParent = parent:getParent()
                end
                finalContainer = parent
            end
            if finalContainer:getItemHoldingCount() >= 5 then
                self:sendCancelMessage("NOT POSSIBLE")
                self:getPosition():sendMagicEffect(CONST_ME_POFF)
                return false
            end
        end
    end
    return true
end

 

[Fortera Global]

function Player:onMoveItem(item, count, fromPosition, toPosition, fromCylinder, toCylinder)
    if toPosition.x == CONTAINER_POSITION then
        local container = self:getContainerById(toPosition.y - 64)
        if not container then
            if toPosition.y == CONST_SLOT_BACKPACK then
                container = self:getSlotItem(CONST_SLOT_BACKPACK)
            end
        end
        if container then
            local parent = toCylinder:getParent()
            local finalContainer = container
            if parent and parent:isItem() then
                local nextParent = parent:getParent()
                while nextParent and nextParent:isItem() do
                    parent = nextParent
                    nextParent = parent:getParent()
                end
                finalContainer = parent
            end
            if finalContainer:getItemHoldingCount() >= 5 then
                self:sendCancelMessage("NOT POSSIBLE")
                self:getPosition():sendMagicEffect(CONST_ME_POFF)
                return false
            end
        end
    end
    return true
end

Thanks buddy, but I needed to check containers only on the floor, not on the player. And this method could not cause lags for some reason?

Oh my bad, this script holding the player right? I'll test <3

 

[Infernum]

Thanks buddy, but I needed to check containers only on the floor, not on the player. And this method could not cause lags for some reason?

Oh my bad, this script holding the player right? I'll test <3

if you're only checking for tiles then use this
also you rarely have to worry about performance since lua is so fast

function Player:onMoveItem(item, count, fromPosition, toPosition, fromCylinder, toCylinder)
    if toPosition.x == CONTAINER_POSITION then
        local container = self:getContainerById(toPosition.y - 64)
        if container then
            local topParent = container:getTopParent()
            if topParent and topParent:isItem() and topParent:getItemHoldingCount() >= 5 then
                self:sendCancelMessage("NOT POSSIBLE")
                self:getPosition():sendMagicEffect(CONST_ME_POFF)
                return false
            end
        end
    end
    return true
end

 

[Fortera Global]

if you're only checking for tiles then use this
also you rarely have to worry about performance since lua is so fast

function Player:onMoveItem(item, count, fromPosition, toPosition, fromCylinder, toCylinder)
    if toPosition.x == CONTAINER_POSITION then
        local container = self:getContainerById(toPosition.y - 64)
        if container then
            local topParent = container:getTopParent()
            if topParent and topParent:isItem() and topParent:getItemHoldingCount() >= 5 then
                self:sendCancelMessage("NOT POSSIBLE")
                self:getPosition():sendMagicEffect(CONST_ME_POFF)
                return false
            end
        end
    end
    return true
end

Worked 100%, thanks!
Can you give your contact discord?