Solved Trying to do quest where you choose 1/x items

[Thorn]

Hello guys, i have been trying to do a quest that you can choose between 5 chest, you can only choose one, you know, this type of quest, but i cant get it to work!!
this is what i have so far

plz guys help me here, i use otx 2.52

<action uniqueid="23450" script="Quests/rareprestigeweapons.lua" />
        <action uniqueid="23450" script="Quests/rareprestigeweapons.lua" />
        <action uniqueid="23450" script="Quests/rareprestigeweapons.lua" />
        <action uniqueid="23450" script="Quests/rareprestigeweapons.lua" />
        <action uniqueid="23450" script="Quests/rareprestigeweapons.lua" />

function onUse(cid, item, fromPosition, itemEx, toPosition)
if item.uid == 23450 then
queststatus = getPlayerStorageValue(cid,20970)
if queststatus == -1 then
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "You have found a Rare Prestige Sword.")
doPlayerAddItem(cid,20970,1)
setPlayerStorageValue(cid,6076,1)
else
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "It is empty.")
end
elseif item.uid == 23450 then
queststatus = getPlayerStorageValue(cid,20902)
if queststatus == -1 then
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "You have found a Rare Prestige Axe.")
doPlayerAddItem(cid,20902,1)
setPlayerStorageValue(cid,6076,1)
else
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "It is empty.")
end
elseif item.uid == 23450 then
queststatus = getPlayerStorageValue(cid,20915)
if queststatus == -1 then
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "You have found a Rare Prestige Hammer.")
doPlayerAddItem(cid,20915,1)
setPlayerStorageValue(cid,6076,1)
else
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "It is empty.")
end
elseif item.uid == 23450 then
queststatus = getPlayerStorageValue(cid,21035)
if queststatus == -1 then
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "You have found a Rare Prestige Spear.")
doPlayerAddItem(cid,21035,1)
setPlayerStorageValue(cid,6076,1)
else
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "It is empty.")
end
elseif item.uid == 23450 then
queststatus = getPlayerStorageValue(cid,21160)
if queststatus == -1 then
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "You have found a Rare Prestige Magic Staff.")
doPlayerAddItem(cid,21160,1)
setPlayerStorageValue(cid,6076,1)
else
doPlayerSendTextMessage(cid,MESSAGE_INFO_DESCR, "It is empty.")
end

end
return 1
end

CONSOLE

[19/11/2016 20:32:56] [Warning - Actions::registerEvent] Duplicate registered item uid: 23450
[19/11/2016 20:32:56] [Warning - Actions::registerEvent] Duplicate registered item uid: 23450
[19/11/2016 20:32:56] [Warning - Actions::registerEvent] Duplicate registered item uid: 23450
[19/11/2016 20:32:56] [Warning - Actions::registerEvent] Duplicate registered item uid: 23450

 

[RazorBlade]

try setting actionid (any actionid) to the chests. I think this prevents it from being actually opened.
as for the other 5 chests, just use a normal quest script for them, since this script is for limiting to one item per storage.

 

[Thorn]

try setting actionid (any actionid) to the chests. I think this prevents it from being actually opened.
as for the other 5 chests, just use a normal quest script for them, since this script is for limiting to one item per storage.

im sorry i dont understand, the other 5 chests i have there were doing ok before this, they have scripts for their one, one by one

Just added action id for the 5 weapons chest and they still open

 

[Infernum]

doesnt work for me, i'll try anihilator >.<

you sure you added the code for actions.xml that i posted and changed the chests to those action ids?
the script i posted has nothing wrong with it

 

[Thorn]

you sure you added the code for actions.xml that i posted and changed the chests to those action ids?
the script i posted has nothing wrong with it

i will try one more time, but im quite sure i did it right, i'll let u know

 

[Thorn]

you sure you added the code for actions.xml that i posted and changed the chests to those action ids?
the script i posted has nothing wrong with it

is that well? the other chests are from 23451-23454

All chests (the 5 weapon chests) are only being opened, you can see the slots (no item inside ofc) and none of these chests give you anything, this is what i have

<action fromaid="23450" toaid="23454" event="script" value="Quests/rareprestigeweapons.lua"/>

local t = {
    [23450] = {string = 'Sword', item = 20970},
    [23451] = {string = 'Axe', item = 20902},
    [23452] = {string = 'Hammer', item = 20915},
    [23453] = {string = 'Spear', item = 21035},
    [23454] = {string = 'Staff', item = 21160}
}

function onUse(cid, item, fromPosition, itemEx, toPosition)
    local tmp = t[item.actionid]
    local queststatus = getPlayerStorageValue(cid, 20970)
    if tmp then
        if queststatus == -1 then
            doPlayerSendTextMessage(cid, MESSAGE_INFO_DESCR, 'You have found a Rare Prestige ' .. tmp.string)
            doPlayerAddItem(cid, tmp.item, 1)
            doPlayerSetStorageValue(cid, 20970, 1)
        else
            doPlayerSendTextMessage(cid, MESSAGE_INFO_DESCR, 'It is empty.')
        end
    end
    return true
end

 

[Infernum]

is that well? the other chests are from 23451-23454

All chests (the 5 weapon chests) are only being opened, you can see the slots (no item inside ofc) and none of these chests give you anything, this is what i have

<action fromaid="23450" toaid="23454" event="script" value="Quests/rareprestigeweapons.lua"/>

local t = {
    [23450] = {string = 'Sword', item = 20970},
    [23451] = {string = 'Axe', item = 20902},
    [23452] = {string = 'Hammer', item = 20915},
    [23453] = {string = 'Spear', item = 21035},
    [23454] = {string = 'Staff', item = 21160}
}

function onUse(cid, item, fromPosition, itemEx, toPosition)
    local tmp = t[item.actionid]
    local queststatus = getPlayerStorageValue(cid, 20970)
    if tmp then
        if queststatus == -1 then
            doPlayerSendTextMessage(cid, MESSAGE_INFO_DESCR, 'You have found a Rare Prestige ' .. tmp.string)
            doPlayerAddItem(cid, tmp.item, 1)
            doPlayerSetStorageValue(cid, 20970, 1)
        else
            doPlayerSendTextMessage(cid, MESSAGE_INFO_DESCR, 'It is empty.')
        end
    end
    return true
end

action id
not unique id

 

[Thorn]

action id
not unique id

OMG right! haha sorry, thanks man!! now it works like a charm! thank you so much