• There is NO official Otland's Discord server and NO official Otland's server list. The Otland's Staff does not manage any Discord server or server list. Moderators or administrator of any Discord server or server lists have NO connection to the Otland's Staff. Do not get scammed!

TFS 0.X [7.72] Craft System

potinho

potinho

Advanced OT User
Joined
Oct 11, 2009
Messages
1,402
Solutions
17
Reaction score
150
Location
Brazil
Good morning guys, I am trying to deploy a craft system on my server with NPC, however when I choose the item I want to craft, it gives the following error on the console. If you can help me make it work.

NPC.XML
XML: 
<?xml version="1.0"?>
<npc name="Drake" script="data/npc/scripts/drake.lua" walkinterval="50000" floorchange="0">
<health now="100" max="100"/>
<look type="129" head="95" body="116" legs="121" feet="115" addons="3"/>
<parameters>
<parameter key="message_greet" value="Hello |PLAYERNAME|. I can craft some {items} for you, but I need specific items to do them." />
</parameters>
</npc>

NPCSCRIPT.LUA
Lua: 
function getItemsFromList(items)
    local str = ''
    if table.maxn(items) > 0 then
        for i = 1, table.maxn(items) do
            str = str .. items[i][2] .. ' ' .. getItemNameById(items[i][1])
            if i ~= table.maxn(items) then str = str .. ', ' 
            end 
        end 
    end
    return str
end
function doRemoveItemsFromList(cid,items)
    local count = 0
    if table.maxn(items) > 0 then
        for i = 1, table.maxn(items) do
            if getPlayerItemCount(cid,items[i][1]) >= items[i][2] then
            count = count + 1 end 
        end 
    end
    if count == table.maxn(items) then
        for i = 1, table.maxn(items) do doPlayerRemoveItem(cid,items[i][1],items[i][2]) end
    else 
        return false 
    end
    return true 
end    
local keywordHandler = KeywordHandler:new()
local npcHandler = NpcHandler:new(keywordHandler)
NpcSystem.parseParameters(npcHandler)
local talkState = {}
function onCreatureAppear(cid) npcHandler:onCreatureAppear(cid) end
function onCreatureDisappear(cid) npcHandler:onCreatureDisappear(cid) end
function onCreatureSay(cid, type, msg) npcHandler:onCreatureSay(cid, type, msg) end
function onThink() npcHandler:onThink() end
function creatureSayCallback(cid, type, msg)
    if(not npcHandler:isFocused(cid)) then
        return false
    end
    local talkUser,msg = NPCHANDLER_CONVbehavior == CONVERSATION_DEFAULT and 0 or cid, msg:lower()
    local craft = {
                ["demon helmet"] = {reward = {{2493, 1}}, items = {{2173,1}, {8925,2}}},
                ["magic plate armor"] = {reward = {{2472, 1}}, items = {{2173,1}, {2160,10}, {2493,2}}}
                }
    
    if isInArray({"item","craft","itens","items", "iten"}, msg) then
        local text = ""
        for i, v in pairs(craft) do
            text = text .. " {"..i.."}, "
        end
        npcHandler:say("I can make " .. text .. " which one do you want?", cid)    
        talkState[talkUser] = 1 
    elseif talkState[talkUser] == 1 then
      ret = craft[msg] 
        if not ret then
             npcHandler:say("I am not crafting this item, perhaps I can consider your order later.", cid) return true
        end
        npcHandler:say("Oh, " .. msg .. " is a good choice. Let me see... I need " .. getItemsFromList(ret.items) .. ", do you have it?" , cid)        
         talkState[talkUser] = 2 
    elseif talkState[talkUser] == 2 and isInArray({"yes","yeah","sim","si"}, msg) then
      if not doRemoveItemsFromList(cid, ret.items) then
               talkState[talkUser] = 0 
            npcHandler:say("Sorry, but you don't have the items that I need.", cid) return true
        end
    for _, i_i in ipairs(ret.reward) do
        local item, amount = i_i[1], i_i[2]
        if isItemStackable(item) or amount == 1 then
             doPlayerAddItem(cid, item, amount)
        else
            for i = 1, amount do
                 doPlayerAddItem(cid, item, 1)
            end
        end
    end
        talkState[talkUser] = 0 
        doSendMagicEffect(getPlayerPosition(cid), math.random(28,30))
        npcHandler:say("Thank you very much! Here is your item as I promised.", cid) return true                    
    elseif msg == "no" then 
        selfSay("Ok... Bye!! Maybe on the next.", cid) 
        talkState[talkUser] = 0 
        npcHandler:releaseFocus(cid) 
    end
    return true
end
npcHandler:setCallback(CALLBACK_MESSAGE_DEFAULT, creatureSayCallback)
npcHandler:addModule(FocusModule:new())

ERROR IN CONSOLE

[20/3/2021 11:8:29] [Error - NpcScript Interface]
[20/3/2021 11:8:29] data/npc/scripts/drake.lua:eek:nCreatureSay
[20/3/2021 11:8:29] Description:
[20/3/2021 11:8:29] data/lib/100-shortcut.lua:260: attempt to index a boolean value
[20/3/2021 11:8:29] stack traceback:
[20/3/2021 11:8:29] data/lib/100-shortcut.lua:260: in function 'getItemNameById'
[20/3/2021 11:8:29] [string "tasktabble = {..."]:232: in function 'getItemsFromList'
[20/3/2021 11:8:29] data/npc/scripts/drake.lua:57: in function 'callback'
[20/3/2021 11:8:29] data/npc/lib/npcsystem/npchandler.lua:363: in function 'onCreatureSay'
[20/3/2021 11:8:29] data/npc/scripts/drake.lua:33: in function <data/npc/scripts/drake.lua:33>
 
Solution
Xikini
I think this is because it's getting a nil value.

Probably because it can't find the items.

Probably because the local is inside of the callback..?

Idk...

Here's my suggestion.

Lua: 
local craft = {
    ["demon helmet"] = {reward = {{2493, 1}}, items = {{2173,1}, {8925,2}}},
    ["magic plate armor"] = {reward = {{2472, 1}}, items = {{2173,1}, {2160,10}, {2493,2}}}
}
function getItemsFromList(items)
    local str = ''
    if table.maxn(items) > 0 then
        for i = 1, table.maxn(items) do
            str = str .. items[i][2] .. ' ' .. getItemNameById(items[i][1])
            if i ~= table.maxn(items) then str = str .. ', ' 
            end 
        end 
    end
    return str
end
function doRemoveItemsFromList(cid,items)
    local...
Sort by date Sort by votes
I think this is because it's getting a nil value.

Probably because it can't find the items.

Probably because the local is inside of the callback..?

Idk...

Here's my suggestion.

Lua: 
local craft = {
    ["demon helmet"] = {reward = {{2493, 1}}, items = {{2173,1}, {8925,2}}},
    ["magic plate armor"] = {reward = {{2472, 1}}, items = {{2173,1}, {2160,10}, {2493,2}}}
}
function getItemsFromList(items)
    local str = ''
    if table.maxn(items) > 0 then
        for i = 1, table.maxn(items) do
            str = str .. items[i][2] .. ' ' .. getItemNameById(items[i][1])
            if i ~= table.maxn(items) then str = str .. ', ' 
            end 
        end 
    end
    return str
end
function doRemoveItemsFromList(cid,items)
    local count = 0
    if table.maxn(items) > 0 then
        for i = 1, table.maxn(items) do
            if getPlayerItemCount(cid,items[i][1]) >= items[i][2] then
            count = count + 1 end 
        end 
    end
    if count == table.maxn(items) then
        for i = 1, table.maxn(items) do doPlayerRemoveItem(cid,items[i][1],items[i][2]) end
    else 
        return false 
    end
    return true 
end    
local keywordHandler = KeywordHandler:new()
local npcHandler = NpcHandler:new(keywordHandler)
NpcSystem.parseParameters(npcHandler)
local talkState = {}
function onCreatureAppear(cid) npcHandler:onCreatureAppear(cid) end
function onCreatureDisappear(cid) npcHandler:onCreatureDisappear(cid) end
function onCreatureSay(cid, type, msg) npcHandler:onCreatureSay(cid, type, msg) end
function onThink() npcHandler:onThink() end
function creatureSayCallback(cid, type, msg)
    if not npcHandler:isFocused(cid) then
        return false
    end
    local talkUser,msg = NPCHANDLER_CONVbehavior == CONVERSATION_DEFAULT and 0 or cid, msg:lower()
    
    if isInArray({"item","craft","itens","items", "iten"}, msg) then
        local text = ""
        for i, v in pairs(craft) do
            text = text .. " {"..i.."}, "
        end
        npcHandler:say("I can make " .. text .. " which one do you want?", cid)    
        talkState[talkUser] = 1 
    elseif talkState[talkUser] == 1 then
        ret = craft[msg] 
        if not ret then
            npcHandler:say("I am not crafting this item, perhaps I can consider your order later.", cid) return true
        end
        npcHandler:say("Oh, " .. msg .. " is a good choice. Let me see... I need " .. getItemsFromList(ret.items) .. ", do you have it?" , cid)        
        talkState[talkUser] = 2 
    elseif talkState[talkUser] == 2 and isInArray({"yes","yeah","sim","si"}, msg) then
        if not doRemoveItemsFromList(cid, ret.items) then
                talkState[talkUser] = 0 
                npcHandler:say("Sorry, but you don't have the items that I need.", cid) return true
            end
        for _, i_i in ipairs(ret.reward) do
            local item, amount = i_i[1], i_i[2]
            if isItemStackable(item) or amount == 1 then
                doPlayerAddItem(cid, item, amount)
            else
                for i = 1, amount do
                    doPlayerAddItem(cid, item, 1)
                end
            end
        end
        talkState[talkUser] = 0 
        doSendMagicEffect(getPlayerPosition(cid), math.random(28,30))
        npcHandler:say("Thank you very much! Here is your item as I promised.", cid) return true                    
    elseif msg == "no" then 
        selfSay("Ok... Bye!! Maybe on the next.", cid) 
        talkState[talkUser] = 0 
        npcHandler:releaseFocus(cid) 
    end
    return true
end
npcHandler:setCallback(CALLBACK_MESSAGE_DEFAULT, creatureSayCallback)
npcHandler:addModule(FocusModule:new())
 
Upvote 1 Downvote
Solution

Similar threads

Mateus Robeerto
  • Question
npc who exchange item for another item that gives storage only once exchange TFS 1.5 NEKIRO
Replies
8
Views
614
Idletibia
I
E
  • Question
TFS 0.X 0.4 Recovery Exp only recovers levels from level 1 to 100, it does not recover levels from level 717217, I want it to recover levels 1 until 717217
Replies
1
Views
153
Enderlop
E
E
  • Question
TFS 0.X 8.60 0.4 NPC Promotion with VIP coin price ID: 11192 Amount: 100 script no function
Replies
2
Views
149
Mateus Robeerto
Mateus Robeerto
bolachapanco
  • Question
NPC Addoner - Canary Source
Replies
0
Views
438
bolachapanco
bolachapanco
Belfahar
  • Question
RevScripts Craft table
Replies
0
Views
218
Belfahar
Belfahar
Shoorkill
  • Question
[TFS 1.5]reset level, life and mana, when acquiring vocation evolution
Replies
2
Views
615
Mateus Robeerto
Mateus Robeerto
J
  • Solved
Grizzly Adams tfs1.5 problem
Replies
10
Views
667
jareczekjsp
J
S
  • Question
Svarground Arena script bug
Replies
0
Views
358
specail
S
H
  • Question
TFS 1.X+ data/npc/lib/npcsystem/modules items[itemid] == nil error
Replies
6
Views
309
Mateus Robeerto
Mateus Robeerto
SixNine
  • Solved
TFS 1.X+ Trying to add storage and mission to Trader
Replies
7
Views
368
SixNine
SixNine
Back
Top