Quest based on class

[Kingmen30264]

I am trying to make a custom quest that allows ONE NPC to give out the proper class reward with a single item turn in (For now).

Right now when I do the turn in, it ONLY works for the first line (Beastlord).


Here is what I have:



#####
#NPC: Epic Test
#Author: Warlore
#####

sub EVENT_SAY
{

my $Epic = quest::saylink("Epic");

if($text=~/hail/i)
{
plugin::Whisper("Hello there $name. You have caught me at somewhat of a bad time. I am just finishing this item that I beleive will be [$Epic].");
}
elsif($text=~/Epic/i)
{
plugin::Whisper("Yes, Epic. All I need from you right now is a rusty dagger!");
}
}

sub EVENT_ITEM
{
if(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Beastlord")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(57054);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Bard")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(77640);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Cleric")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(20076);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Warrior")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(60332);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Shadowknight")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(48136);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Magician")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(19839);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Druid")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(62880);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Paladin")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(48147);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Enchanter")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(52962);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Monk")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(67742);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Berserker")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(18609);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Rogue")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(52348);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Necromancer")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(64067);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Ranger")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(62649);
}
elsif(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Shaman")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(57405);
}
else(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Wizard")
{
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::summonitem(16576);
}
}



When I first tested this, I composed ONLY the first line and when it worked, I simply copied and pasted the line just changing the class specified.



if(plugin::check_handin(\%itemcount, 7007 => 1) && $class eq "Beastlord")



Any and all help with this would be great. Thank you in advance.

[trevius]

This should work for you and is a bit cleaner and easier to manage:

sub EVENT_ITEM {

# Class => Reward,
my %Reward = (
"Warrior" => 60332,
"Rogue" => 52348,
"Monk" => 67742,
"Berserker" => 18609,
"Shadowknight" => 48136,
"Paladin" => 48147,
"Ranger" => 62649,
"Bard" => 77640,
"Beastlord" => 57054,
"Cleric" => 20076,
"Druid" => 62880,
"Shaman" => 57405,
"Wizard" => 16576,
"Magician" => 19839,
"Enchanter" => 52962,
"Necromancer" => 64067
);
if($Reward{$class})
{
my $RewardID = $Reward{$class}; # Reward

if (plugin::check_handin(\%itemcount, 7007 => 1))
{
quest::summonitem($RewardID);
plugin::Whisper("Thank you for your hard work $name. Enjoy your new weapon!");
quest::exp(1000);
quest::ding();
}
else
{
quest::say("Sorry, $name, but I have no use for that.");
}
}

plugin::return_items(\%itemcount);

}

[Kingmen30264]

Very nice. I will use this. (I am still trying to learn Perl, so all this helps me learn.)

Thank you :)