Lua Turn sword into Mace

[highclick]

Hi! I'm trying too transform a sword (ID 11395) into a mace (ID 2448). I'm very new at using .lua and im unfamiliar with their functions. If anyone could explain what i should put into the parameters and why that would be great

This is how far I've come with my script:

Spoiler: Script

function onUse(cid, item, fromPosition, itemEx, toPosition)
    if item.itemid == 11395 then
        doTransformItem(uid, toitemid[, count/subtype])

 

[Breed]

I kept it simple for you so you could understand how it works

local transformItem = {
    from = 11395, -- sword
    to = 2448 -- studded club
}
function onUse(cid, item, fromPosition, itemEx, toPosition)
    if item.itemid == transformItem.from then
        doTransformItem(item.uid, transformItem.to)
    end
    return true
end

 

[highclick]

I kept it simple for you so you could understand how it works

local transformItem = {
    from = 11395, -- sword
    to = 2448 -- studded club
}
function onUse(cid, item, fromPosition, itemEx, toPosition)
    if item.itemid == transformItem.from then
        doTransformItem(item.uid, transformItem.to)
    end
    return true
end

Wow thanks alot, it took some headscratching too fully understand but I got it now!

One more question:

I've added this line into movements.xml :

    <movevent event="Equip" itemid="11395" function="onEquipItem" slot="sword" script="swordtomace.lua"/>
    <movevent event="DeEquip" itemid="11395" function="onDeEquipItem" slot="sword" script="swordtomace.lua"/>

And then I've created a .lua file in the scripts folder thats names "swordtomace.lua" where I've used your script.

Why doesn't it work in game?

 

[Breed]

onUse is for actions, you would apply this to actions.xml

<action itemid="11395" script="swordtomace.lua"/>

 

[highclick]

onUse is for actions, you would apply this to actions.xml

<action itemid="11395" script="swordtomace.lua"/>

I added that line into actions.xml and i also moved swordtomace.lua from /movements/scripts too /actions/scripts.

It works perfect, thank you very much Breed!

 

[Breed]

Np

 

[highclick]

Np

I tried too modify the script too loop between sword-mace-axe. So that when its a sword it becomes a mace, if it's a mace it becomes an axe, if it's an axe it becomes a sword and so on.

I added this script:

local transformItem = {
    sword = 11395, -- sword
    mace = 2448, -- mace
    axe = 2441 -- axe
}
function onUse(cid, item, fromPosition, itemEx, toPosition)
    if item.itemid == transformItem.sword then
        doTransformItem(item.uid, transformItem.mace)
    elseif item.itemid == transformItem.mace
        doTransformItem(item.uid, transformItem.axe)
    elseif item.itemid == transformItem.axe
        doTransformItem(item.uid, transformItem.sword)
    end
    return true
end

and these lines too action.xml:

    <action itemid="11395" script="weapons/swordtomace.lua"/>
    <action itemid="2448" script="weapons/swordtomace.lua"/>
    <action itemid="2441" script="weapons/swordtomace.lua"/>

Do you know why it doesnt work?

 

[Breed]

did you get an error?
what happened when you clicked the initial item and did it change at all and upto what point.

 

[Luiz Vitor]

you forgot "then" in both "elseif"

 

[Breed]

you forgot "then" in both "elseif"

I totally over looked that lol

 

[highclick]

did you get an error?
what happened when you clicked the initial item and did it change at all and upto what point.

No errors, It works now, I had forgotten the "then" in the "elseif" statements.

Thanks again for all your help

 

[highclick]

you forgot "then" in both "elseif"

Thanks alot Luiz, really helped me out. Feelin' stoopid!

 

[Xikini]

When using this method, it attempts to use the item, and then the script.
Is there a way to stop the on-use method of the item?

 

[Breed]

Idk try that, love the sig btw

local transformItem = {
    sword = 11395, -- sword
    mace = 2448, -- mace
    axe = 2441 -- axe
}
function onUse(cid, item, fromPosition, itemEx, toPosition)
    if item.itemid == transformItem.sword then
        doTransformItem(item.uid, transformItem.mace)
        return true
    elseif item.itemid == transformItem.mace then
        doTransformItem(item.uid, transformItem.axe)
        return true
    elseif item.itemid == transformItem.axe then
        doTransformItem(item.uid, transformItem.sword)
        return true
    end
    return true
end

 

[Xikini]

I tried the return true already, it didn't work
And thank you

 

[Breed]

I tried the return true already, it didn't work
And thank you

how about return false lol..

 

[Xikini]

Tried just now, didn't work.

 

[Breed]

Tried just now, didn't work.

I couldn't tell ya then coz I'm only running tfs 1.x

I wonder if we removed the last 3 param's would it make a different, coz if you think about it, there is a thing called function overloading.

It is where there is a several functions with the same name but with different parameters.
If this is the case the server would choose the correct functions to use or what information to pass based on the parameters given.

So instead of this

onUse(cid, item, fromPosition, itemEx, toPosition)

You could try

onUse(cid, item)

Because itemEx is effected by item which is why you get the little icon when you right click the item, if I am wrong that is ok too.

 

[Xikini]

@Limos

 

[Limos]

If you want to remove the usable option of an item, edit tibia.dat and items.otb.
If you want to prevent players from doing something, check for itemids of the item/itemEx.