Weapon attribute

[Doomdice]

tfs 1.2 version i try makeing a script for movements so attribute will work which im confuse in my self where the heck i put it and is this script right?

function onEquip(cid, item, slot)
    if player:getSlotItem(CONST_SLOT_LEFT).itemid == 2376 then
    player:itemType:getAttack(25)
    player:getPosition():sendMagicEffect (CONST_ME_CAKE)
return true
end

 

[Doomdice]

bump im guessing everyone got no clue on this

 

[Codex NG]

bump im guessing everyone got no clue on this

If you want an answer asap then this code is incorrect and your missing an end to close out the if statement.

player:itemType:getAttack(25)

 

[Printer]

Can you explain, what you are trying todo?

 

[forgee]

I don't know what you're trying to do here, but you put the script in movements/scripts and movements.xml would be something like this:

<movevent event="Equip" itemid="2376" slot="hand" script="script.lua">

function onEquip(cid, item, slot)
if player:getSlotItem(CONST_SLOT_LEFT).itemid == 2376 then

'player' is undefined.

player:itemType:getAttack(25)

Player has no field called itemType, to call a method you're missing the parenthesis, besides Player doesn't have any method called itemType either. getAttack doesn't expect any arguments. It returns the attack value of an item, it doesn't change it.

Some relevant methods are ItemType:getAttack(), Item:getType(), Item:getAttribute(key), Item:setAttribute(key, value).

 

[Doomdice]

Can you explain, what you are trying todo?

Im trying to make it where when you loot a weapon it say bonus attack +25 or 10 shadow told me it very easy to make the script i thought i was on right line for it since i look in lua.cpp for the function

 

[Printer]

But onEquip is only to equip items, should look into events onMoveItem.

I would done something like this:

if fromPosition.x ~= CONTAINER_POSITION and toPosition.x == CONTAINER_POSITION then
    if item:getId() == 2376 then
        self:sendTextMessage(MESSAGE_INFO_DESCR, string.format("%s attack: %d.", item:getName(), item:getType():getAttack()))
        self:getPosition():sendMagicEffect(CONST_ME_CAKE)
    end
end

 

[Doomdice]

But onEquip is only to equip items, should look into events onMoveItem.

I would done something like this:

if toPosition.x == CONTAINER_POSITION then
    if item:getId() == 2376 then
        self:sendTextMessage(MESSAGE_INFO_DESCR, string.format("%s attack: %d.", item:getName(), item:getType():getAttack()))
    end
end

Do i need to do this ?

function onEquip(cid, item, slot)
if toPosition.x == CONTAINER_POSITION then
self:sendTextMessage(MESSAGE_INFO_DESCR, string.format("%s attack: %d.", item:getName(), item:getType():getAttack()))
else
player:getPosition():sendMagicEffect (CONST_ME_CAKE)
return true
end

 

[Printer]

Recopy the code, no it is in events folder. onMoveItem

 

[Doomdice]

Recopy the code, no it is in events folder. onMoveItem

xml or lua I look in player.lua dont see that code or in events.xml unless replace the first code with this function

 function Player:onMoveItem(item, count, fromPosition, toPosition)