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 ).

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(2048(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 of

code brackets

when 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.