Can you help to fix this????

[f@bio]

@@ Hello friends, i'm have a problem with a movement for sell items...

This is my script and him make a many error on console:

-- Sistema de vendas usando movimentos BY F@bio -- Fudera Otserver...

function onAddItem(moveitem, tileitem, position, cid)

local playerPos = getCreaturePosition(cid)

if moveitem.itemid == 2498 then
doPlayerAddMoney(cid, 18000)
doRemoveItem(moveitem.uid) --royal helmet
doSendMagicEffect(position, CONST_ME_HITBYFIRE)
doSendAnimatedText(playerPos, "$$$", TEXTCOLOR_GOLD)
end
return TRUE
end

-- When i'm take item on a tile send this error:

Lua Script Error: [MoveEvents Interface]
data/movements/scripts/sell_move.luanAddItem

attempt to index a number value
stack traceback:
[C]: in function 'doSendAnimatedText'
data/movements/scripts/sell_move.lua:10: in function <data/movements/scr
ipts/sell_move.lua:3>

-- And when i'm delete ANIMATED TXT, this make other error:

Lua Script Error: [MoveEvents Interface]
data/movements/scripts/sell_move.luanAddItem

luaGetCreaturePosition(). Creature not found

--- Plx someone help on this, i't a evolution on my server, to prevent lags and others...

Many thx for all...

 

[StreamSide]

It doesnt works because that function dont have (cid)

^.-

 

[kokorec]

maybe can you make it with "onStepIn".

 

[slaw]

It doesnt works because that function dont have (cid)

^.-

TFS 0.3 have this param

 

[kokorec]

@f@bio
you try with:

	isItemMoveable = isItemMovable

 

[Zonet]

try this

function onAddItem(moveitem, tileitem, position)


if moveitem.itemid == 2498 then
doRemoveItem(moveitem.uid)
doPlayerAddMoney(cid, 18000)
doSendMagicEffect(getPlayerPosition(cid), CONST_ME_HITBYFIRE)
doSendAnimatedText(getPlayerPosition(cid), "$$$", TEXTCOLOR_GOLD)
end
return TRUE
end

rep++ if work please

@f@bio
you try with:

	isItemMoveable = isItemMovable

isMoveable checks only if the item is moveable. Not if moved.

 

[Pitufo™]

Edit: No idea:

PD: Nahruto Gay

 

[Zonet]

@up

I got all functions on a lua file so that's what i found out?

 

[Jano]

cid was added to additem event in beta 2

just look at the soruces and

uint32_t MoveEvent::executeAddRemItem(Creature* actor, Item* item, Item* tileItem, const Position& pos)

so use

onAddItem(cid, moveitem, tileitem, position)

newbies

 

[f@bio]

try this

function onAddItem(moveitem, tileitem, position)


if moveitem.itemid == 2498 then
doRemoveItem(moveitem.uid)
doPlayerAddMoney(cid, 18000)
doSendMagicEffect(getPlayerPosition(cid), CONST_ME_HITBYFIRE)
doSendAnimatedText(getPlayerPosition(cid), "$$$", TEXTCOLOR_GOLD)
end
return TRUE
end

rep++ if work please



isMoveable checks only if the item is moveable. Not if moved.

When i'm use this appear thi error:

Lua Script Error: [MoveEvents Interface]
data/movements/scripts/sell_move.luanAddItem

luaDoPlayerAddMoney(). Player not found

Lua Script Error: [MoveEvents Interface]
data/movements/scripts/sell_move.luanAddItem

attempt to index a number value
stack traceback:
[C]: in function 'doSendMagicEffect'
data/movements/scripts/sell_move.lua:9: in function <data/movements/scri
pts/sell_move.lua:3>

More ideias??? many thx for all...

 

[Jano]

^.- why you dont read?

function onAddItem(cid, moveitem, tileitem, position)

if moveitem.itemid == 2498 then
doRemoveItem(moveitem.uid)
doPlayerAddMoney(cid, 18000)
doSendMagicEffect(getPlayerPosition(cid), CONST_ME_HITBYFIRE)
doSendAnimatedText(getPlayerPosition(cid), "$$$", TEXTCOLOR_GOLD)
end
return TRUE
end

of course you should use tfs 0.3 beta 2 or beta 3

 

[f@bio]

hum... i'm use TFS 0.2 patch 22

 

[Jano]

then you fucked it wont work, you could use anything like .

    local cid = get thing from pos -> where the player should be.
    if isPlayer(cid) then
        doPlayerAddMoney(cid, 18000)
        doSendMagicEffect(getPlayerPosition(cid), CONST_ME_HITBYFIRE)
        doSendAnimatedText(getPlayerPosition(cid), "$$$", TEXTCOLOR_GOLD)
    end

Time to learn baby, i wont make it.

 

[Zonet]

then you fucked it wont work, you could use anything like .

    local cid = get thing from pos -> where the player should be.
    if isPlayer(cid) then
        doPlayerAddMoney(cid, 18000)
        doSendMagicEffect(getPlayerPosition(cid), CONST_ME_HITBYFIRE)
        doSendAnimatedText(getPlayerPosition(cid), "$$$", TEXTCOLOR_GOLD)
    end

Time to learn baby, i wont make it.

if isPlayer(cid) == TRUE

xD

 

[Jano]

nop, you can use isPlayer(cid) without compare.

 

[Zonet]

@up
so if you do if isPlayer(cid) then it is TRUE or FALSE?

i think TRUE correct?

 

[Jano]

when you use...

if isPlayer(cid) == TRUE then

it can be.

if TRUE == TRUE then
-> Become true -> if true then
continue

or

if FALSE == TRUE then
-> become false -> if false then
stop

if isPlayer(cid) then

if true then continue

 

[f@bio]

where is the ITEM.ID on your script???