[Fadedspirit]

Originally Posted by Akkadius

First - Because it is not a client script - it's a spell script. $caster_id is the exported object of this trigger EVENT:

                case EVENT_SPELL_EFFECT_CLIENT:
		case EVENT_SPELL_EFFECT_NPC:
		case EVENT_SPELL_EFFECT_BUFF_TIC_CLIENT:
                case EVENT_SPELL_EFFECT_BUFF_TIC_NPC:
		{
			ExportVar(packagename.c_str(), "caster_id", extradata);
			break;
		}

Second - GetTarget() is a valid mob method

Third - He made a syntax error

sub EVENT_SPELL_EFFECT_CLIENT {
	my $client = $entity_list->GetClientByID($caster_id);
	my $ClientTarget = $Client->GetTarget();
	if($ClientTarget->IsNPC()) {	 
		TESTLIFETAP();
	}
}

Should be:

sub EVENT_SPELL_EFFECT_CLIENT {
	my $Client = $entity_list->GetClientByID($caster_id);
	my $ClientTarget = $Client->GetTarget();
	if($ClientTarget->IsNPC()) {	 
		TESTLIFETAP();
	}
}

You're making your own client object - since client is not implied you have to fetch it through the data that is exported to the sub.

There is no need to respond in such a way - he legitimately asked for help.

Sorry, didn't see it was a spell script.

Also, the way I replied wasn't any different than what many would reply. There are only a few people whom are actually nice when it comes to code . Doesn't excuse me, but just saying.