More efficient way to do this routine?

[Greyhelm]

I was wondering if there was a more efficient way to do this.
Also I want to add a class check so that I particular class needs to be in the party or proximity of the trigger. I setup an invisible man trigger at a location and setup his proximity. I couldn't only get one mob to spawn while in the proximity so I opt'd to use depop. Though the spawned npc was set to unique it still kept spawning the NPC if I moved in and out of the proximity.

sub EVENT_SPAWN{
   quest::set_proximity($x-5,$x+5,$y-5,$y+5,$z-5,$z+5);
   }
sub EVENT_ENTER{
   if($class eq 'Ranger'){
   quest::spawn2(303091,0,0,1309,652,388,130);
   $npc->Depop();
   }
}

Any suggestions? I am reading over $class checks and tried if($class eq 'Ranger') but that didn't work under EVENT_ENTER. Thanks in advance for the help.

EDIT - I missed the squiggley's, class check works.. When you use the right syntax...