[Randymarsh9]

This might help

sub EVENT_TIMER {

 if ($timer == "sun") {
  quest::me("The lava pit shoots upward, revealing a new form as it melts back in to the pit.");
  quest::spawn("999354, 0, 0 -2.1, -2492.1, -795.4, 126.7");
  quest::stoptimer("sun");
 }
}

should instead be

sub EVENT_TIMER {

 if ($timer eq "sun") {
  quest::me("The lava pit shoots upward, revealing a new form as it melts back in to the pit.");
  quest::spawn(999354, 0, 0 -2.1, -2492.1, -795.4, 126.7);
  quest::stoptimer("sun");
 }
}

I'm pretty sure that for timers with a name like that, you use eq instead of ==, and I don't think you need the quotation marks around the spawn things.