mounts from chest

[Quickshot]

hey i have a small problem with one of my scripts. when you have no mounts and you use the chest its works fine, but if you have all the mounts and use the chest you get bugged plz help

heres the script

function onUse(cid, item, frompos, item2, topos)
if item.uid == 12345 then
queststatus = getPlayerStorageValue(cid,368)
if queststatus == -1 or queststatus == 0 then
doPlayerSendTextMessage(cid,19,"Now you have all mounts")
for i = 1, 12 do     
doPlayerAddMount(cid, i)
end
setPlayerStorageValue(cid,368,1)
else
doPlayerSendTextMessage(cid,25,"The chest is empty.")
end
end
return true
end

 

[MartyX]

cid,25 --- NO EXIST (8.61+)
Try,,

doPlayerSendTextMessage[COLOR=#66CC66]([/COLOR]cid,[COLOR=#CC66CC] MESSAGE_EVENT_ADVANCE[/COLOR],[COLOR=#FF0000] "Now you have all mounts"[/COLOR][COLOR=#66CC66])
[/COLOR]doPlayerSendTextMessage[COLOR=#66CC66]([/COLOR]cid,[COLOR=#CC66CC] MESSAGE_INFO_DESCR[/COLOR],[COLOR=#FF0000] "The chest is empty."[/COLOR][COLOR=#66CC66])[/COLOR]

 

[Apollo_117]

Try This one, not tested tho:

function onUse(cid, item, frompos, item2, topos)
    if item.uid == 12345 then
     queststatus = getPlayerStorageValue(cid, 368)
	else
        if queststatus == -1 or queststatus == 0 then
            doPlayerSendTextMessage(cid,19,"Now you have all mounts")
        for i = 1, 12 do     
            doPlayerAddMount(cid, i)
            setPlayerStorageValue(cid,368,1)
        else
            doPlayerSendCancel(cid, "You already claimed your mounts!")
	end	
    end    
    return true
end

 

[Cloow]

for i = 1, 12 do

I guess that's for the mounts ur gonna get? Can't I just make so, i recive 1 mount.

So..

for i = 1, 1 do

?

 

[Apollo_117]

I guess that's for the mounts ur gonna get? Can't I just make so, i recive 1 mount.

So.. ?

You're saying instead to give 12 mounts, give only 1?

 

[QuaS]

When he's loopig starting at 1 and ending at 1 then he gets 1 mount, isn't it obvious? What a bunch of %$#%#$

 

[Santi]

I guess that's for the mounts ur gonna get? Can't I just make so, i recive 1 mount.

So.. ?

No loop needed for only 1 mount

function onUse(cid, item, frompos, item2, topos)
    if item.uid == 12345 then
     queststatus = getPlayerStorageValue(cid, 368)
	else
        if queststatus == -1 or queststatus == 0 then
            doPlayerSendTextMessage(cid,19,"Now you have all mounts")     
            doPlayerAddMount(cid, 1) ----- see :D
            setPlayerStorageValue(cid,368,1)
        else
            doPlayerSendCancel(cid, "You already claimed your mounts!")
	end	
    end    
    return true
end