Quest::Bug Collection

**AndMetal** · #1 03-04-2008, 08:17 PM

Quote:

Originally Posted by shaznito

ok this looks about right.

Code:

$random = quest::ChooseRandom(14010,13281,13280,14015);

Just one thing to add: you'll probably need to add the following for the reward to actually be given

Code:

quest::summonitem($random)

You may also want to use my with $random, so it doesn't affect any other scripts:

Code:

my $random = quest::ChooseRandom(14010,13281,13280,14015);

Other than that, looks good to me

**cavedude** · #2 03-05-2008, 12:07 AM

I normally use:

Code:

quest::summonitem(quest::ChooseRandom(14010,13281,13280,14015));

Simply because it's one line, without a need to declare a variable. My feeling is the fewer variables we have, the better off we are. Now, if the reward is going to appear multiple times, then yes use a variable. That's part of the reason why they exist, for the lazy coder

shaznito · #3 03-05-2008, 03:30 AM

I was to tired to test this last night, and for some reason even if I wanted to I couldn't because blixkin entopop refuses to respond. Not sure why but oh well no matter I'll figure it out sooner or later.

**cavedude** · #4 03-05-2008, 04:41 AM

You had a closing bracket in the wrong spot. It should have been after the item return plugin.