Multiple item return on single item handin

[EliteSting]

Npc seems to skip the quest:summonitem part and goes right to the end where it hands me back my item.

sub EVENT_SAY 
{  if($text=~/Hail/i){quest::say("Hello hello! I see you wish to begin your [epic] journey , what do you wish to know?"); } if($text=~/epic/i){quest::say("Well, I can tell you that a long , long time ago there were weapons of incredible power given to the mortals of this world. But through time thier locations have been lost, and thier power has grown dim. Such treasures are not to be handled lightly. Recently however, some of these magnificent pieces of art have come into my possession. Perhaps we can arrange a [trade]?"); } if($text=~/trade/i){quest::say("You wish to trade eh? Well then , for one of these magnificent treatures you must perform a task. There is a most grusome pair of creatures which inhabit The Overthere . They block access into the great chasm , if they should be killed and a piece of thier flesh brought to me, I will reward you. Now go and return with what I ask.");}
}

sub EVENT_ITEM {
if ($class == 'Shaman' || $class == 'Rogue' || $class == 'Monk' || $class == 'Druid' || $class == 'Berserker' || $class == 'Wizard' || $class == 'Magician' || $class == 'Necromancer' || $class == 'Enchanter' || $class == 'Shadowknight' || $class == 'Cleric' || $class == 'Bard' || $class == 'Paladin') {
  if (plugin::check_handin(\%itemcount, 1737 == 1)) {
    my %rewards = (
      "Shaman" => 10651, "Rogue" => 11057, "Monk" => 10652, "Druid" => 20490, "Berserker" => 68299, "Wizard" => 14341, "Magician" => 28034, "Necromancer" => 20544, "Enchanter" => 10650, "Shadowknight" => 14383, "Cleric" => 5532, "Bard" => 20542, "Paladin" => 10099
    );

    if(defined($rewards{$class})) {
      quest::summonitem($rewards{$class});
      quest::emote("Feels deeply satisfied knowing one of his worst enemies has been slain.'");
      quest::say("As we agreed. Our deal is complete.")
    }
	else {
        quest::say("I did not ask for this from you, $name.");
	plugin::return_items(\%itemcount);
	}
	}
}
}

sub EVENT_ITEM {
if ($class == 'Warrior') {
  if (plugin::check_handin(\%itemcount, 1737 == 1)) {
       quest::summonitem(10909);
       quest::summonitem(10910);
       quest::summonitem(17859);
       quest::emote("Feels deeply satisfied knowing one of his worst enemies has been slain.'");
       quest::say("As we agreed. Our deal is complete.")
  }
        else {
        quest::say("I did not ask for this from you, $name.");
	plugin::return_items(\%itemcount);
	}
  }
}

sub EVENT_ITEM {
if ($class == 'Ranger') {
  if (plugin::check_handin(\%itemcount, 1737 == 1)) {
    quest::summonitem(20487);
    quest::summonitem(20488);
    quest::emote("Feels deeply satisfied knowing one of his worst enemies has been slain.'");
    quest::say("As we agreed. Our deal is complete.")
  }
        else {
        quest::say("I did not ask for this from you, $name.");
	plugin::return_items(\%itemcount);
	}
  }
}

sub EVENT_ITEM {
if ($class == 'Beastlord') {
  if (plugin::check_handin(\%itemcount, 1737 == 1)) {
    quest::summonitem(8495);
    quest::summonitem(8496);
    quest::emote("Feels deeply satisfied knowing one of his worst enemies has been slain.'");
    quest::say("As we agreed. Our deal is complete.")
  }
        else {
        quest::say("I did not ask for this from you, $name.");
	plugin::return_items(\%itemcount);
	}
}
}

Any suggestions please ?