elsif is better to use so that once one condition is met the rest of the elsif blocks are skipped. otherwise every single check is made for every single hand in.
This is not breaking your script, but you are using the return items incorrectly. You need to call it at the end of the EVENT_ITEM sub outside of any if/else block. That is a common misunderstanding and a lot of quests are wrote incorrectly even in the PEQ SVN.
Code:
}
else {
quest::say("This does not interest me, $name.");
}
plugin::return_items(\%itemcount);
}