Anything wrong with this?

[Murdoc]

```
If CanTake(Index, 1, 1) = true And CanTake(Index, 2, 1) = true And CanTake(Index, 3, 1) = true Then

Call PlayerMsg(index, "Here is the item4 I promised!", 12)
Call TakeItem(index, 1, 1)
Call TakeItem(index, 2, 1)
Call TakeItem(index, 3, 1)
Call GiveItem(index, 4, 1)

Else

Call PlayerMsg(index, "Bring me an item1, item2 and item3 and I'll make you an item4!", 12)

End If
```
It's supposed to be a scripted npc that checks for 3 different items, and if you have them you get another item. I'm pretty new at scripting, so I'll be surprised if it's totally right…

[Godlord]

Try:

> If CanTake(Index, 1, 0) = True And CanTake(Index, 2, 0) = True And CanTake(Index, 3, 0) = True Then
> Call PlayerMsg(Index, "Here is the item4 I promised!", 12)
> Call TakeItem(Index, 1, 1)
> Call TakeItem(Index, 2, 1)
> Call TakeItem(Index, 3, 1)
> Call GiveItem(Index, 4, 1)
> Else
> Call PlayerMsg(Index, "Bring me an item1, item2 and item3 and I'll make you an item4!", 12)
> End If

Regards,
  Godlord.