Lua Help me with an action script

[axas]

function onUse(cid, item, fromPosition, itemEx, toPosition)
if item.itemid == 1054 then
doPlayerAddItem(cid,2173,1)
	else 
	doPlayerSendTextMessage(cid,22,"You can't mine this.")
return TRUE
end

I want it to give player 2173 item id when you use a pick on 1054 item id.
Also would be good if it removed the 2173 and respawned it at the same spot in 120 seconds.

 

[SpiderOT]

function onUse(cid, item, fromPosition, itemEx, toPosition)
if (itemEx.itemid == 1054  and  item.itemid == pick id then
doPlayerAddItem(cid,2173,1)
    else 
    doPlayerSendTextMessage(cid,22,"You can't mine this.")
return TRUE
end

change the pick id

 

[axas]

function onUse(cid, item, fromPosition, itemEx, toPosition)
if (itemEx.itemid == 1054  and  item.itemid == pick id then
doPlayerAddItem(cid,2173,1)
    else 
    doPlayerSendTextMessage(cid,22,"You can't mine this.")
return TRUE
end

change the pick id

won't work

 

[Evil Mark]

local time = 120
function onUse(cid, item, fromPosition, itemEx, toPosition)
	if itemEx.itemid == 1054 and itemEx.actionid == 1111 then
		doPlayerAddItem(cid, 2173, 1)
		doPlayerSendTextMessage(cid, MESSAGE_INFO_DESCR, "Mining was sucessful")
		doTransformItem(itemEx.uid, 460, 1)
		doDecayItem(460, 1) time)
	end
		else
		doPlayerSendTextMessage(cid, MESSAGE_INFO_DESCR, "add me in facebook brooh plix")
end

I've got no fawking idea if it'll work, I was half asleep. Try it

I added actionid to mine, would be a little sad if it worked on every wall u did it on..

 

[Cykotitan]

function onUse(cid, item, fromPosition, itemEx, toPosition)
	if itemEx.itemid == 1054 then
		doPlayerAddItem(cid, 2173, 1)
		doRemoveItem(itemEx.uid)
		addEvent(doCreateItem, 120 * 1000, itemEx.itemid, 1, toPosition)
	else 
		doPlayerSendTextMessage(cid, MESSAGE_EVENT_ADVANCE, 'You can\'t mine this.')
	end
	return true
end

 

[Evil Mark]

function onUse(cid, item, fromPosition, itemEx, toPosition)
	if itemEx.itemid == 1054 then
		doPlayerAddItem(cid, 2173, 1)
		doRemoveItem(itemEx.uid)
		addEvent(doCreateItem, 120 * 1000, itemEx.itemid, 1, toPosition)
	else 
		doPlayerSendTextMessage(cid, MESSAGE_EVENT_ADVANCE, 'You can\'t mine this.')
	end
	return true
end

Where is the pick? It should be in the script too right?

 

[Doggynub]

No no need to check for the pick as you already stated the item id for this script on the action.xml line.

 

[axas]

function onUse(cid, item, fromPosition, itemEx, toPosition)
	if itemEx.itemid == 1054 then
		doPlayerAddItem(cid, 2173, 1)
		doRemoveItem(itemEx.uid)
		addEvent(doCreateItem, 120 * 1000, itemEx.itemid, 1, toPosition)
	else 
		doPlayerSendTextMessage(cid, MESSAGE_EVENT_ADVANCE, 'You can\'t mine this.')
	end
	return true
end

seems to drop the error message even if the item id I'm mining is correct :<

actions.xml:
<action itemid="2553" event="script" value="tools/pick.lua"/>

 

[Cykotitan]

Works here

 

[Evil Mark]

No no need to check for the pick as you already stated the item id for this script on the action.xml line.

AAAA!!! ok. I always forget about the XML, jeez me so nub

 

[axas]

Works here

Ah, i had item.itemid instead of itemEx.itemid. Why is it Ex though?


and why doesn't the variable work instead of the ids? :<

function onUse(cid, item, fromPosition, itemEx, toPosition)
local minables = {1356, 1357, 1358}
	if itemEx.itemid == minables then
			doPlayerAddItem(cid,2173,1)
			doRemoveItem(itemEx.uid)
			addEvent(doCreateItem, 5 * 1000, itemEx.itemid, 1, toPosition)
	else 
			doPlayerSendTextMessage(cid,22,"You can\'t mine this.")
		end
		return TRUE
end

 

[Alyhide]

Doesn't this already have a thread?

Professions System

 

[axas]

Doesn't this already have a thread?

Professions System

I'm just trying to learn lua at the moment, and this is one of my very first scripts. Anyways, got it to work with if(isInArray(minables, itemEx.itemid)). :>

 

[chojrak]

Ah, i had item.itemid instead of itemEx.itemid. Why is it Ex though?


and why doesn't the variable work instead of the ids? :<

function onUse(cid, item, fromPosition, itemEx, toPosition)
local minables = {1356, 1357, 1358}
	if itemEx.itemid == minables then
			doPlayerAddItem(cid,2173,1)
			doRemoveItem(itemEx.uid)
			addEvent(doCreateItem, 5 * 1000, itemEx.itemid, 1, toPosition)
	else 
			doPlayerSendTextMessage(cid,22,"You can\'t mine this.")
		end
		return TRUE
end

itemEx is second item that you use first item on, like you use shovel on sand then sand is itemeEx

This:

local minables = {1356, 1357, 1358}
	if itemEx.itemid == minables then

Should be:

local minables = {1356, 1357, 1358}
	if isInArray(minables, itemEx.itemid) then