Need help with chest spawn quest

[kaos00723]

Hi, I can't figure out what I'm doing wrong here. I'm trying to get this chest spawn script installed so that all NPC's up to lvl 50 have a chance of spawning a treasure chest.

sub EVENT_DEATH
{
$rand = quest::ChooseRandom(1,2,3,4,5,6,7,8,9,10);
if ($npc->GetLevel() >= $ulevel)
{
	if ($rand > 9)
	{

		if ($ulevel < 11)
		{
			quest::emote(" was holding extraordinary treasure!");
			quest::spawn(1250,0,0,$x,$y,$z);
		} else {
			if ($ulevel < 21)
			{
				quest::emote(" was holding extraordinary treasure!");
				quest::spawn(1251,0,0,$x,$y,$z);
			} else {
				if ($ulevel < 31)
				{
					quest::emote(" was holding extraordinary treasure!");
					quest::spawn(1252,0,0,$x,$y,$z);
				} else {
					if ($ulevel < 41)
					{
						quest::emote(" was holding extraordinary treasure!");
						quest::spawn(1253,0,0,$x,$y,$z);
					} else {
						if ($ulevel < 51)
						{
							quest::emote(" was holding extraordinary treasure!");
							quest::spawn(1254,0,0,$x,$y,$z);
						} else {
					// We don't support level 51+ characters
				}
			}
		}
	} else {
		quest::emote(" was worth no treasure this time.");
	}
} else {
	quest::emote(" is worth no treasure as it is weaker than you.");
}
}

I've tried adding this to the default.pl in the quests directory, the plugins directory, the templates directory, and even individual zones with absolutely no luck. I've tried adding it to the stock default.pl and tried renaming it to default.pl by itself, nothing seems to work.

Can someone please explain to me how to get this to work?

Thanks!

[KLS]

There's an issue with the script. The closing brackets( } ) don't all match the opening brackets( { ). Go through and make sure they match up and try it again.

sub EVENT_DEATH
{
	$rand = quest::ChooseRandom(1,2,3,4,5,6,7,8,9,10);
	if ($npc->GetLevel() >= $ulevel)
	{
		if ($rand > 9)
		{
			if ($ulevel < 11)
			{
				quest::emote(" was holding extraordinary treasure!");
				quest::spawn(1250,0,0,$x,$y,$z);
			} 
			else {
				if ($ulevel < 21)
				{
					quest::emote(" was holding extraordinary treasure!");
					quest::spawn(1251,0,0,$x,$y,$z);
				} 
				else 
				{
					if ($ulevel < 31)
					{
						quest::emote(" was holding extraordinary treasure!");
						quest::spawn(1252,0,0,$x,$y,$z);
					} 
					else 
					{
						if ($ulevel < 41)
						{
							quest::emote(" was holding extraordinary treasure!");
							quest::spawn(1253,0,0,$x,$y,$z);
						} 
						else 
						{
							if ($ulevel < 51)
							{
								quest::emote(" was holding extraordinary treasure!");
								quest::spawn(1254,0,0,$x,$y,$z);
							}
						}
					}
				}
			}
		} 
		else 
		{
			quest::emote(" was worth no treasure this time.");
		}
	} 
	else 
	{
		quest::emote(" is worth no treasure as it is weaker than you.");
	}
}

There might be other problems I didn't check but that was the most obvious. I try to tell people to have consistent clean style when they code for the project to avoid things like this, if everything lines up it's a lot easier to catch and fix mistakes.

[kaos00723]

Thanks KLS, adding a couple closing brackets did the trick.