Think I broke Absor

[xmimx1213]

I was messing around on my server today trying to get Absor in tutorialb to give out epic 1.0's instead of the normal reward when u hand in a chink of bronze. Just a little preface I know nothing about scripting this is THE VERY FIRST THING I have tried to do (besides make myself a GM but that was easy :D).

Anyway this is what i ended up with:

elsif (plugin::check_handin(\%itemcount, 54229 => 1)) {
#Chunk of Bronze -> Bronze Gloomingdeep weapons
quest::say("Now let me see... Ah ha! Here ya go! A much better weapon to help fend off those nasties!");
if($class eq 'Warrior'){quest::summonitem(66175)(10910)(10909); }
elsif($class eq 'Cleric'){quest::summonitem(5532);}
elsif($class eq 'Paladin'){quest::summonitem(10099);}
elsif($class eq 'Ranger'){quest::summonitem(20488)(20487);}
elsif($class eq 'Shadowknight'){quest::summonitem(14383);}
elsif($class eq 'Druid'){quest::summonitem(20490);}
elsif($class eq 'Monk'){quest::summonitem(10652);}
elsif($class eq 'Bard'){quest::summonitem(20542);}
elsif($class eq 'Rogue'){quest::summonitem(11057);}
elsif($class eq 'Shaman'){quest::summonitem(10651);}
elsif($class eq 'Necromancer'){quest::summonitem(20544);}
elsif($class eq 'Wizard'){quest::summonitem(14341);}
elsif($class eq 'Magician'){quest::summonitem(28034);}
elsif($class eq 'Enchanter'){quest::summonitem(10650);}
elsif($class eq 'Beastlord'){quest::summonitem(8495)(8496);}
elsif($class eq 'Berserker'){quest::summonitem(68299);}


I have been testing him on fresh characters. Now he eats my regular dagger/sword instead of giving me a sharpened version and he also eats the chunk of bronze when I hand it to him... Ultimately the only thing I care about from Absor is him handing out the 1.0's.... I really want him to also hand out 1.5's for a chunk of iron but that will be after i figure out 1.0's :) any help would be very much appreciated!

Edit: there isnt a smiley face in my script its supposed to read 20488 ) lol

[Kingly_Krab]

This is not how summoning multiple items is done: if($class eq 'Warrior'){quest::summonitem(66175)(10910)(10909); }
elsif($class eq 'Ranger'){quest::summonitem(20488)(20487);}
elsif($class eq 'Beastlord'){quest::summonitem(8495)(8496);}This is how summoning multiple items is done: if($class eq 'Warrior'){quest::summonitem(66175); quest::summonitem(10910); quest::summonitem(10909); }
elsif($class eq 'Ranger'){quest::summonitem(20488); quest::summonitem(20487);}
elsif($class eq 'Beastlord'){quest::summonitem(8495); quest::summonitem(8496);}

[Uleat]

I highly recommend the use ofcode bracketswhen submitting code snippets.

Also, post your entire script so that it's integrity can be verified.

From the looks of your 'snippet,' you're probably missing a closing '}'

Remember: if you open something, you also have to close it

[xmimx1213]

Thank you so much guys, that was so fast! I will be sure to use the brackets next time and will try and fix Absor right now. Thanks again incredible support. :)