Why this doesn't work?

[Acordion]

function onUse(cid, item, frompos, item2, topos)

container = doPlayerAddItem(cid, 2001, 1)

if item.itemid == 1945 and doPlayerRemoveMoney(cid,4000) == 1 then
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doTransformItem(item.uid,item.itemid+1)

elseif item.itemid == 1946 and doPlayerRemoveMoney(cid,4000) == 1 then
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doTransformItem(item.uid,item.itemid-1)

else
doPlayerSendCancel(cid,"Sorry, you don't have enough money!")
end
return 1
end

Please :blink:

 

[Ecstacy]

What exactly do you want to do and what TFS version?

 

[Acordion]

i want to sell potions for bps, I do have tfs 0.3.6pl1

 

[QuaS]

Search

 

[Acordion]

Ok i will search, but now. why doesn't work my script?

 

[Summ]

It doesn't work because the script is pissed off that you don't use loops

Errors?

 

[QuaS]

lets keep ur non-loop, non secured, noobish style:

function onUse(cid, item, frompos, item2, topos)

if doPlayerRemoveMoney(cid,4000) then
container = doCreateItemEx(2001,1)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doAddContainerItem(container, 2213, 100)
doPlayerAddItemEx(cid, container , 1)
doTransformItem(item.uid,(item.itemid==1945) and 1946 or 1945)
else
doPlayerSendCancel(cid,"Sorry, you don't have enough money!")
end
return 1
end

There are many scripts kinda this, f.ex JDB one which is i think the best relased on this forum. (instead my one which i relased in one of request threads) xD

 

[Acordion]

Thanks, Works Perfect