getPlayerSlotItem not working

[mikeware]

Hey, i made a script with the function getPlayerSlotItem but it's not working. using 2.9.

here's the script

function onCastSpell(cid, var)
if getPlayerSlotItem(cid, 2) == 2173 then
	doSendMagicEffect(getPlayerPosition(cid), 3)
else
	doPlayerSendCancel(cid, "blablabla")
	doSendMagicEffect(getPlayerPosition(cid), 2)
end
end

well, if the function was working, it would reproduce a magic effect of value 3 but is not, it's sending the cancel msg and the other magiceffect..

yea, im with a AOL equiped (id 2173) and the spells.xml is everything right
this is just a test script.

 

[slaw]

Try use getPlayerSlotItem(cid, 2).itemid

I think it must work, becouse this function wasnt changed long time, and is still working for me.

 

[mikeware]

like...
function onCastSpell(cid, var)
if getPlayerSlotItem(cid, 2).2173 then
doSendMagicEffect(getPlayerPosition(cid), 3)
else
doPlayerSendCancel(cid, "blablabla")
doSendMagicEffect(getPlayerPosition(cid), 2)
end
end

??

 

[mikeware]

Found a way to work, but the way that was suposed to work

getPlayerSlotItem(cid, 2) == itemid

is not workingo

 

[Mark]

if getPlayerSlotItem(cid, 2).itemid == 2173 then

 

[Jonern]

You should also use
getCreaturePosition(cid)
and not getPlayerPosition(cid) because that function is going to be removed sometime in the future.

 

[mikeware]

You should also use
getCreaturePosition(cid)
and not getPlayerPosition(cid) because that function is going to be removed sometime in the future.

that's bad cuz ill need to recheck all my scripts to see if there are a playerposition(cid) to change into creatureposition... would be better if was both of them.


@TALATUREN
ya, that could help, the way i made to work is similar.

 

[Jonern]

Well, it's going to be removed sooner or later so you can just start fixing it

 

[Mark]

It'll be removed in 0.3.