attempt to call global getMana

[Tbol]

So im trying to make a code but it gives that error im pretty sure im using tfs 1.2 functions

function onUse(cid, item, frompos, item2, topos)
player1 = getThingfromPos(topos)
lol = getPlayerName(player1.uid)

local nick = getPlayerName(player1.uid)
local ki = getPlayerMagLevel(player1.uid)
local lvl = getPlayerLevel(player1.uid)
local mana = getMana(player1.uid)
local hp = getHealth(player1.uid)
local maxmana = getMaxMana(player1.uid)
local maxhp = getMaxHealth(player1.uid)
local access = getPlayerAccess(player1.uid)

if isPlayer(player1.uid) ~= 1 then
doPlayerSendCancel(cid,"You can only use on players.")


elseif access <= 3 then
doPlayerSendTextMessage(cid,22, 'Nick: '..nick..'')
doPlayerSendTextMessage(cid,22, 'Level: '..lvl..'')
doPlayerSendTextMessage(cid,22, 'Ki Level: '..ki..'')
doPlayerSendTextMessage(cid,22, 'Hp: '..hp..'/'..maxhp..'')
doPlayerSendTextMessage(cid,22, 'Ki Points: '..mana..'/'..maxmana..'')
else
return 0
end
return 1
end

 

[Nekiro]

these are not tfs 1.2 functions, your coding style is old < 1.x style. You should use objects (userdatas) and cast functions on them like

local player = Player(playerId)
if player then
    local mana = player:getMana()
end

onUse actually uses player as parameter now, its object (userdata), you can use it like

player:sendTextMessage()
player:getMana()

and so on.

 

[Tbol]

these are not tfs 1.2 functions, your coding style is old < 1.x style. You should use objects (userdatas) and cast functions on them like

local player = Player(playerId)
if player then
    local mana = player:getMana()
end

onUse actually uses player as parameter now, its object (userdata), you can use it like

player:sendTextMessage()
player:getMana()

and so on.

I guess im still doing something wrong. No errors but it doesnt work.

function onUse(cid, item, frompos, item2, topos)
player1 = getThingfromPos(topos)

local player = Player(playerId)
if player then
    local mana = player:getMana()
    local maxmana = player:getMaxMana()
end

if isPlayer(player1.uid) ~= 1 then
doPlayerSendCancel(cid,"You can scout only players.") 


elseif access <= 3 then
player:sendTextMessage(cid,22, 'Ki Points: '..mana..'/'..maxmana..'')
    else 
        return 0
    end
    return 1
end

 

[Tbol]

I guess im still doing something wrong. No errors but it doesnt work.

function onUse(cid, item, frompos, item2, topos)
player1 = getThingfromPos(topos)

local player = Player(playerId)
if player then
    local mana = player:getMana()
    local maxmana = player:getMaxMana()
end

if isPlayer(player1.uid) ~= 1 then
doPlayerSendCancel(cid,"You can scout only players.")


elseif access <= 3 then
player:sendTextMessage(cid,22, 'Ki Points: '..mana..'/'..maxmana..'')
    else
        return 0
    end
    return 1
end

bump

 

[Xikini]

bump

Sorry, I suck with TFS 1.2
Hope this works.

function onUse(player, item, fromPosition, target, toPosition, isHotkey)
    if player:getGroup():getAccess() then
        return true
    end
    if type(target) == "userdata" and not target:isPlayer() then
        return true
    end
    local mana = target:getMana()
    local maxmana = target:getMaxMana()
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Ki Points: " .. mana .. "/" .. maxmana .. "")
    return true
end

 

[Nekiro]

Sorry, I suck with TFS 1.2
Hope this works.

function onUse(player, item, fromPosition, target, toPosition, isHotkey)
    if player:getGroup():getAccess() then
        return true
    end
    if type(target) == "userdata" and not target:isPlayer() then
        return true
    end
    local mana = target:getMana()
    local maxmana = target:getMaxMana()
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Ki Points: " .. mana .. "/" .. maxmana .. "")
    return true
end

If not target is enough or target and not target:isPlayer()
If you wont gonna use mana and maxmana variables anywhere else, then its pointless to declare them into variables, concatenate them directly into string.

 

[Xikini]

If not target is enough or target and not target:isPlayer()
If you wont gonna use mana and maxmana variables anywhere else, then its pointless to declare them into variables, concatenate them directly into string.

Sorry wasn't sure about the userdata part, since the tfs source on github was using it in the potions.lua file.
Figured it was required in case you misclicked on a floor tile or something.

 

[Tbol]

Sorry, I suck with TFS 1.2
Hope this works.

function onUse(player, item, fromPosition, target, toPosition, isHotkey)
    if player:getGroup():getAccess() then
        return true
    end
    if type(target) == "userdata" and not target:isPlayer() then
        return true
    end
    local mana = target:getMana()
    local maxmana = target:getMaxMana()
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Ki Points: " .. mana .. "/" .. maxmana .. "")
    return true
end

It says you cannot use this object, not sure if its dat/otb problem

 

[Crypter]

try

function onUse(player, item, fromPosition, target, toPosition, isHotkey)
    if type(target) ~= "userdata" or not target:isPlayer() then
        player:sendCancelMessage("You can only use on players.")
        return false
    end
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Nick: " .. target:getName())
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Level: " .. target:getLevel())
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Health Points: " .. target:getHealth() .. "/" .. target:getMaxHealth())
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Ki Points: " .. target:getMana() .. "/" .. target:getMaxMana())
    return true
end

 

[Xikini]

It says you cannot use this object, not sure if its dat/otb problem

Is there an error?

 

[Tbol]

try

function onUse(player, item, fromPosition, target, toPosition, isHotkey)
    if type(target) ~= "userdata" or not target:isPlayer() then
        player:sendCancelMessage("You can only use on players.")
        return false
    end
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Nick: " .. target:getName())
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Level: " .. target:getLevel())
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Health Points: " .. target:getHealth() .. "/" .. target:getMaxHealth())
    player:sendTextMessage(MESSAGE_INFO_DESCR, "Ki Points: " .. target:getMana() .. "/" .. target:getMaxMana())
    return true
end

Works perfectly

Is there an error?

It was clean console.