Rubies Quest

[Creon]

So I have this script for the Small Rubies Quest, but it's giving an unlimited amount of rubies. How do I add a storage for it?

function onUse(cid, item, fromPosition, itemEx, toPosition)
	if getPlayerStorageValue(cid, 10005) < 1 then
        local bag = doPlayerAddItem(cid, 2147, 6)
        doPlayerSendTextMessage(cid, MESSAGE_INFO_ADVANCE, "You have found 6 small rubies.")
    else
        doPlayerSendTextMessage(cid, MESSAGE_INFO_ADVANCE, "It is empty.")
    end
return TRUE
end

 

[Bogart]

local storage  = 10005 
function onUse(cid, item, fromPosition, itemEx, toPosition)
	if getPlayerStorageValue(cid, storage) < 1 then
		local bag = doPlayerAddItem(cid, 2147, 6)     
		doPlayerSendTextMessage(cid, MESSAGE_INFO_ADVANCE, "You have found 6 small rubies.")
		doPlayerSetStorageValue(cid, storage, 1)
	else
		doPlayerSendTextMessage(cid, MESSAGE_INFO_ADVANCE, "It is empty.")
	end
return TRUE
end

 

[Creon]

The message "You have found 6 small rubies." does not appear, nor does the storage. I still get unlimited rubies when I click it.

 

[4Muki4]

u clicking it with a player or admin account?

 

[Creon]

Player.

 

[Zuma Master]

local storage  = 23514 
function onUse(cid, item, fromPosition, itemEx, toPosition)
    if getPlayerStorageValue(cid, storage) < 1 then
	local bag = doPlayerAddItem(cid, 2147, 6)     
	doPlayerSendTextMessage(cid, MESSAGE_INFO_ADVANCE, "You have found 6 small rubies.")
	doPlayerSetStorageValue(cid, storage, 1)
    else
	doPlayerSendTextMessage(cid, MESSAGE_INFO_ADVANCE, "It is empty.")
    end
    return TRUE
end

*EDIT-1*: I simply changed the storage key, as you are apparently using it already. (Bogart's one)