1 - Verificação de variáveis, sempre usamos 2 iguais (==)
Troque:
if queststatus = -1 then
if item.uid = 16591 then
Por:
if queststatus == -1 then
if item.uid == 16591 then
2 - Faltava um "end" no final.
3 - Parabéns, era só isso, agora fazendo do meu jeito ficaria:
http://pastebin.com/fjgu5jM3
Não precisa fazer a verificação:
if item.uid == 16591 then
Se colocar isso no actions.xml na tag do script não precisa por dentro dele
E eu acho que:
if queststatus == -1 then
Só acumula linhas, fazendo do meu jeito fica menor e mais simples, não precisa se preocupar com isso, apenas crie variáveis, se for precisar ou se quiser fazer configurações para o leitor arrumar
~~
Boa sorte com a sua carreira de scripter