I think you may be missing a few semicolons at the end:
Code:
sub EVENT_ITEM {
if (plugin::check_handin(\%itemcount, 51624 => 1, 1563 => 1, 1568 => 1)) {
quest::say("My mind can be at ease knowing those fiends are no more.");
quest::emote("pulls a gleaming weapon out of s bag.");
quest::say("Take this weapon. I am sure you will be able to make use of it.");
if($class eq 'Warrior') {quest::summonitem(1569);}
elsif($class eq 'Cleric') {quest::summonitem(1582);}
elsif($class eq 'Paladin') {quest::summonitem(1577);}
elsif($class eq 'Ranger') {quest::summonitem(1569);}
elsif($class eq 'Shadowknight') {quest::summonitem(1577);}
elsif($class eq 'Druid') {quest::summonitem(1582);}
elsif($class eq 'Monk') {quest::summonitem(1579);}
elsif($class eq 'Bard') {quest::summonitem(1569);}
elsif($class eq 'Rogue') {quest::summonitem(1581);}
elsif($class eq 'Shaman') {quest::summonitem(1582);}
elsif($class eq 'Necromancer') {quest::summonitem(1581);}
elsif($class eq 'Wizard') {quest::summonitem(1581);}
elsif($class eq 'Magician') {quest::summonitem(1581);}
elsif($class eq 'Enchanter') {quest::summonitem(1581);}
elsif($class eq 'Beastlord') {quest::summonitem(1579);}
elsif($class eq 'Berserker') {quest::summonitem(1577);};
};
}
On a side note, I would recommend using a switch/case (or possibly an array) instead of a string of if/elsif statements:
Code:
sub EVENT_ITEM {
if (plugin::check_handin(\%itemcount, 51624 => 1, 1563 => 1, 1568 => 1)) {
quest::say("My mind can be at ease knowing those fiends are no more.");
quest::emote("pulls a gleaming weapon out of s bag.");
quest::say("Take this weapon. I am sure you will be able to make use of it.");
my $reward = 1001; #Cloth Cap is the booby prize if you're none of these classes
switch ($class) {
case "Warrior" {$reward = 1569;}
case "Cleric" {$reward = 1582;}
case "Paladin" {$reward = 1577;}
case "Ranger" {$reward = 1569;}
case "Shadowknight" {$reward = 1577;}
case "Druid" {$reward = 1582;}
case "Monk" {$reward = 1579;}
case "Bard" {$reward = 1569;}
case "Rogue" {$reward = 1581;}
case "Shaman" {$reward = 1582;}
case "Necromancer" {$reward = 1581;}
case "Wizard" {$reward = 1581;}
case "Magician" {$reward = 1581;}
case "Enchanter" {$reward = 1581;}
case "Beastlord" {$reward = 1579;}
case "Berserker" {$reward = 1577;}
};
quest::summonitem($reward);
};
}