Lua If have slots free in backpack

warriorfrog · Jun 11, 2016

I have this script to auto cath gold from dead monsters and another to transform golds...
But when i kill a monster and no have slots countainers free all money going to ground...
And it can lag my ot cause so much trash

Someone could help me to fix it?

Auto Get Gold SCRIPT:

Code:

<event type="kill" name="GP" event="script" value="gp.lua"/>

Code:

local function doPlayerAddContainerMoney(cid, container, gold)
local extrac = 0
for slot = (getContainerSize(container) - 1), 0, -1 do
local item = getContainerItem(container, slot)
local amount = (item.itemid == 2148 and item.type or (item.itemid == 2152 and item.type * 100 or (item.itemid == 2160 and item.type * 10000)) or 0)
if item.uid > 0 then
if amount > 0 then
gold = gold + amount
doRemoveItem(item.uid)
end
end
if isContainer(item.uid) then
extrac = item.uid
end
end
if extrac > 0 then
doPlayerAddContainerMoney(cid, extrac, gold)
else
doPlayerAddMoney(cid, gold)
end
end

local function autoloot(cid, position, corpseID)
if not isPlayer(cid) then
return
end

local corpse = getTileItemById(position, corpseID)
if corpse and isContainer(corpse.uid) then
doPlayerAddContainerMoney(cid, corpse.uid, 0)
end
end

function onKill(cid, target)
if isPlayer(target) then
return true
end
local position = getCreaturePosition(target)
local info = getMonsterInfo(getCreatureName(target))
addEvent(autoloot, 100, cid, position, info.lookCorpse)
return true
end

Nekiro · Jun 11, 2016

Code:

local function doPlayerAddContainerMoney(cid, container, gold)
local extrac = 0
for slot = (getContainerSize(container) - 1), 0, -1 do
local item = getContainerItem(container, slot)
local amount = (item.itemid == 2148 and item.type or (item.itemid == 2152 and item.type * 100 or (item.itemid == 2160 and item.type * 10000)) or 0)
if item.uid > 0 then
if amount > 0 then
gold = gold + amount
doRemoveItem(item.uid)
end
end
if isContainer(item.uid) then
extrac = item.uid
end
end
if extrac > 0 then
doPlayerAddContainerMoney(cid, extrac, gold)
else
doPlayerAddMoney(cid, gold)
end
end

local function autoloot(cid, position, corpseID)
if not isPlayer(cid) then
return
end

local corpse = getTileItemById(position, corpseID)
if corpse and isContainer(corpse.uid) then
doPlayerAddContainerMoney(cid, corpse.uid, 0)
end
end

function onKill(cid, target)
local bp = getPlayerSlotItem(cid, CONST_SLOT_BACKPACK).uid
local freeSlotsInBp = math.max(0, getContainerCap(bp) - getContainerSize(bp))

if isPlayer(target) then
return true
end
local position = getCreaturePosition(target)
local info = getMonsterInfo(getCreatureName(target))
if freeSlotsInBp > 0 then
addEvent(autoloot, 100, cid, position, info.lookCorpse)
else
return false
end
return true
end

Try this.

warriorfrog · Jun 11, 2016

Thexamx said:

Code:

local function doPlayerAddContainerMoney(cid, container, gold)
local extrac = 0
for slot = (getContainerSize(container) - 1), 0, -1 do
local item = getContainerItem(container, slot)
local amount = (item.itemid == 2148 and item.type or (item.itemid == 2152 and item.type * 100 or (item.itemid == 2160 and item.type * 10000)) or 0)
if item.uid > 0 then
if amount > 0 then
gold = gold + amount
doRemoveItem(item.uid)
end
end
if isContainer(item.uid) then
extrac = item.uid
end
end
if extrac > 0 then
doPlayerAddContainerMoney(cid, extrac, gold)
else
doPlayerAddMoney(cid, gold)
end
end

local function autoloot(cid, position, corpseID)
if not isPlayer(cid) then
return
end

local corpse = getTileItemById(position, corpseID)
if corpse and isContainer(corpse.uid) then
doPlayerAddContainerMoney(cid, corpse.uid, 0)
end
end

function onKill(cid, target)
local bp = getPlayerSlotItem(cid, CONST_SLOT_BACKPACK).uid
local freeSlotsInBp = math.max(0, getContainerCap(bp) - getContainerSize(bp))

if isPlayer(target) then
return true
end
local position = getCreaturePosition(target)
local info = getMonsterInfo(getCreatureName(target))
if freeSlotsInBp > 0 then
addEvent(autoloot, 100, cid, position, info.lookCorpse)
else
return false
end
return true
end

Try this.

When my first backpack no have slots free i killed a monster but it still onfoot, not transform to corpse while i dont have slots free in backpack

Zothion · Jun 11, 2016

what do you want it to do? give to player if player has cap & space, else drop inside body?
also thexamx, that will only check the first backpack afaik

Nekiro · Jun 11, 2016

Zothion said:
what do you want it to do? give to player if player has cap & space, else drop inside body?
also thexamx, that will only check the first backpack afaik

I cant understand what he wants XD
I made it like that, cause I think thats all he wants

warriorfrog · Jun 11, 2016

Thexamx said:
I cant understand what he wants XD
I made it like that, cause I think thats all he wants

I'm sorry to my english! Its was i want... you help me good
The problem is if i dont have slots in my backpack monster didnt die

They stay like this:
https://camo.githubusercontent.com/...32d396466342d3664616563373231656430382e706e67

No life, no name, but no die

Nekiro · Jun 11, 2016

Code:

local function doPlayerAddContainerMoney(cid, container, gold)
local extrac = 0
for slot = (getContainerSize(container) - 1), 0, -1 do
local item = getContainerItem(container, slot)
local amount = (item.itemid == 2148 and item.type or (item.itemid == 2152 and item.type * 100 or (item.itemid == 2160 and item.type * 10000)) or 0)
if item.uid > 0 then
if amount > 0 then
gold = gold + amount
doRemoveItem(item.uid)
end
end
if isContainer(item.uid) then
extrac = item.uid
end
end
if extrac > 0 then
doPlayerAddContainerMoney(cid, extrac, gold)
else
doPlayerAddMoney(cid, gold)
end
end

local function autoloot(cid, position, corpseID)
if not isPlayer(cid) then
return
end

local bp = getPlayerSlotItem(cid, CONST_SLOT_BACKPACK).uid
local freeSlotsInBp = math.max(0, getContainerCap(bp) - getContainerSize(bp))
local corpse = getTileItemById(position, corpseID)
if corpse and isContainer(corpse.uid) then
if freeSlotsInBp > 0 then
doPlayerAddContainerMoney(cid, corpse.uid, 0)
else
return false
end
end
end

function onKill(cid, target)
if isPlayer(target) then
return true
end
local position = getCreaturePosition(target)
local info = getMonsterInfo(getCreatureName(target))
addEvent(autoloot, 100, cid, position, info.lookCorpse)
return true
end

warriorfrog · Jun 11, 2016

Thexamx said:

Code:

local function doPlayerAddContainerMoney(cid, container, gold)
local extrac = 0
for slot = (getContainerSize(container) - 1), 0, -1 do
local item = getContainerItem(container, slot)
local amount = (item.itemid == 2148 and item.type or (item.itemid == 2152 and item.type * 100 or (item.itemid == 2160 and item.type * 10000)) or 0)
if item.uid > 0 then
if amount > 0 then
gold = gold + amount
doRemoveItem(item.uid)
end
end
if isContainer(item.uid) then
extrac = item.uid
end
end
if extrac > 0 then
doPlayerAddContainerMoney(cid, extrac, gold)
else
doPlayerAddMoney(cid, gold)
end
end

local function autoloot(cid, position, corpseID)
if not isPlayer(cid) then
return
end

local bp = getPlayerSlotItem(cid, CONST_SLOT_BACKPACK).uid
local freeSlotsInBp = math.max(0, getContainerCap(bp) - getContainerSize(bp))
local corpse = getTileItemById(position, corpseID)
if corpse and isContainer(corpse.uid) then
if freeSlotsInBp > 0 then
doPlayerAddContainerMoney(cid, corpse.uid, 0)
else
return false
end
end
end

function onKill(cid, target)
if isPlayer(target) then
return true
end
local position = getCreaturePosition(target)
local info = getMonsterInfo(getCreatureName(target))
addEvent(autoloot, 100, cid, position, info.lookCorpse)
return true
end

Ty u so much man!

Lua If have slots free in backpack

warriorfrog

Active Member

Nekiro

Legendary OT User

warriorfrog

Active Member

Zothion

Veteran OT User

Nekiro

Legendary OT User

warriorfrog

Active Member

Nekiro

Legendary OT User

warriorfrog

Active Member

Similar threads