Chest refill

[Maezukie]

the error I'm getting is

There's no loop that matches 'loop-value's inventory' (chest.sk, line 51: add {_randomitem} to the block at loop-value's inventory')

command /chest:
    permission: chest.sk
    trigger:
        loop {chestloc::*}:
            if loop-value is a chest:
                clear loop-value's inventory
                set {_item} to a random number between 3 and 6
                loop {_item} times:
                    set {_randomitem} to random element out of {random.item.list::*}
                    add {_randomitem} to the block at loop-value's inventory #<--- The error line
        broadcast "&aChests Refilled! &fNext refill, 5 minutes"

 

[TPGamesNL]

the error I'm getting is

command /chest:
    permission: chest.sk
    trigger:
        loop {chestloc::*}:
            if loop-value is a chest:
                clear loop-value's inventory
                set {_item} to a random number between 3 and 6
                loop {_item} times:
                    set {_randomitem} to random element out of {random.item.list::*}
                    add {_randomitem} to the block at loop-value's inventory #<--- The error line
        broadcast "&aChests Refilled! &fNext refill, 5 minutes"

Try removing the "block at" part since loop-value is already a block.

 

[Maezukie]

Try removing the "block at" part since loop-value is already a block.

Nope same error

 

[ShaneBee]

try "clear inventory of loop-value"

 

[Maezukie]

try "clear inventory of loop-value"

that's not the problem

 

[ShaneBee]

"There's no loop that matches 'loop-value's inventory' (chest.sk, line 51: add {_randomitem} to the block at loop-value's inventory')"
try add {_randomitem} to inventory of block at loop-value

 

[Maezukie]

"There's no loop that matches 'loop-value's inventory' (chest.sk, line 51: add {_randomitem} to the block at loop-value's inventory')"
try add {_randomitem} to inventory of block at loop-value
New error message

There are multiple loops that match loop-value. use loop-value-1/2/3/etc. to specify which loop's valse you want. (chest.sk, line 51: add {_randomitem} to inventory of block at loop-value')

 

[ShaneBee]

loop-value-1

 

[Maezukie]

loop-value-1

no errors but the chests dont refill
[doublepost=1534894279,1534894184][/doublepost]Heres the full code

on load:
    delete {random.item.list::*}
    add 2 diamond to {random.item.list::*}
    add 2 iron ingot to {random.item.list::*}
    add 2 leather to {random.item.list::*}
    add 2 tnt to {random.item.list::*}
    add 1 flint and steel to {random.item.list::*}
    add 1 golden apple to {random.item.list::*}
    add 4 cooked beef to {random.item.list::*}
    add 32 stone to {random.item.list::*}
    add 32 cobble stone to {random.item.list::*}
    add 4 cooked porkchop to {random.item.list::*}
    add 32 wooden plank to {random.item.list::*}
    add 2 apple to {random.item.list::*}
    add 1 stone sword to {random.item.list::*}
    add 1 stone pickaxe to {random.item.list::*}
    add 1 stone axe to {random.item.list::*}
    add 1 leather leggings to {random.item.list::*}
    add 1 leather boots to {random.item.list::*}
    add 1 leather chestplate to {random.item.list::*}
    add 1 leather helmet to {random.item.list::*}
    add 4 snowball to {random.item.list::*}
    add 4 arrow to {random.item.list::*}
    add 1 bow to {random.item.list::*}
    add 1 iron sword to {random.item.list::*}
    add 1 iron pickaxe to {random.item.list::*}
    add 1 iron axe to {random.item.list::*}
    add 1 iron leggings to {random.item.list::*}
    add 1 iron boots to {random.item.list::*}
    add 1 iron chestplate to {random.item.list::*}
    add 1 iron helmet to {random.item.list::*}
    add 1 diamond sword to {random.item.list::*}
    add 1 diamond pickaxe to {random.item.list::*}
    add 1 diamond axe to {random.item.list::*}
    add 1 diamond boots to {random.item.list::*}
    add 1 diamond leggings to {random.item.list::*}
    add 1 diamond helmet to {random.item.list::*}
    add 1 diamond chestplate to {random.item.list::*}
    wait 1 tick
    execute console command "/chest"
  
command /chest:
    permission: chest.sk
    trigger:
        loop {chestloc::*}:
            if loop-value is a chest:
                clear loop-value's inventory
                set {_item} to a random number between 3 and 6
                loop {_item} times:
                    set {_randomitem} to random element out of {random.item.list::*}
                    add {_randomitem} to inventory of block at loop-value-1
        broadcast "&aChests Refilled! &fNext refill, 5 minutes"

command /setloc:
    permission: Setchest.loc
    trigger:
        if target block is a chest:
            add target block's location to {chestloc::*}
            send "&aChest location added!"
        else:
            send "&cNot a chest"
 
every 5 minutes:
    execute console command "/chest"

 

[ShaneBee]

okay, so:
add target block's location to {chestloc::*}
You are adding a location the variable

then:
loop {chestloc::*}:
if loop-value is a chest:

you are checking if the loop value is a chest? But the loop value is a location

 

[Maezukie]

okay, so:
add target block's location to {chestloc::*}
You are adding a location the variable

then:
loop {chestloc::*}:
if loop-value is a chest:

you are checking if the loop value is a chest? But the loop value is a location

oh, how would I fix this

 

[ShaneBee]

if target block is a chest:
            add target block's location to {chestloc::*}

when setting the variable, you already have it check for the chest....

so in this one... remove the condition checking the chest

 loop {chestloc::*}:
            if loop-value is a chest:
                clear loop-value's inventory