Right now you are assuming that every loot item in a treasure chest is an item, and any coins in the chest cause an error. Specifically, GetLootSlotLink() returns nil if the item is money, and you aren't handling that case.

i will have a look at this when classic is available so i can find some treasure chests...

i think i have fixed this, could you give it a try?