Personnally i prefer not to put the return item in a else . That way if they give you the right item with another item , the other items will be returned ...
Example:
Code:
sub EVENT_ITEM {
if (plugin::check_handin(\%itemcount, 1001=>1)) {
quest::say("You have done it $name! May you be graced with the strength of Brell Seliris!");
quest::exp(25000);
quest::ding();
quest::set_zone_flag(58 );
$client->Message(15, "You received a character flag!");
}
plugin::return_items(\%itemcount);
}