Nice quest npcs?

[sotonin]

hmmm interesting.

here's the problem though. I want the npc to
A) return item to player if they only hand in a single item from a multi-item turn in.
B) return any non-quest related item handed to him

here's what i have going so far, lemme know if i screwed anything up. I was thinking if there was some way to make a temporary variable. i could set a variable to true or 1 or whatever if any of the successful item turn in's are used. (item1 2 and 3 for quest are turned in, so set variable to true.) then below all the checks, do a if variable != true say "i dont need this, blah blah" and return all items in all 4 slots to the player.

sub EVENT_ITEM {
	if($item1 > 0 || $item2 > 0 || $item3 > 0 || $item4 > 0){
		if($itemcount{19001} == 1 && $itemcount{19002} == 1 && $itemcount{16507} == 1){
			quest::say("Wear this with pride!");
			quest::summonitem("4921");
		}
		if($itemcount{19003} == 1 && $itemcount{19004} == 1 && $itemcount{19047} == 1){
			quest::say("Wear this with pride!");
			quest::summonitem("4922");
		}
		if($itemcount{19005} == 1 && $itemcount{19006} == 1 && $itemcount{19048} == 1){
			quest::say("Wear this with pride!");
			quest::summonitem("4923");
		}
		if($itemcount{19007} == 1 && $itemcount{19008} == 1 && $itemcount{19049} == 1){
			quest::say("Wear this with pride!");
			quest::summonitem("4924");
		}
	}
}

i don't know exact perl syntax for this but here's my best complete guess

sub EVENT_ITEM {
	if($item1 > 0 || $item2 > 0 || $item3 > 0 || $item4 > 0){
		if($itemcount{19001} == 1 && $itemcount{19002} == 1 && $itemcount{16507} == 1){
			quest::say("Wear this with pride!");
			quest::summonitem("4921");
			$var1=true;
		}
		if($itemcount{19003} == 1 && $itemcount{19004} == 1 && $itemcount{19047} == 1){
			quest::say("Wear this with pride!");
			quest::summonitem("4922");
			$var1=true;
		}
		if($itemcount{19005} == 1 && $itemcount{19006} == 1 && $itemcount{19048} == 1){
			quest::say("Wear this with pride!");
			quest::summonitem("4923");
			$var1=true;
		}
		if($itemcount{19007} == 1 && $itemcount{19008} == 1 && $itemcount{19049} == 1){
			quest::say("Wear this with pride!");
			quest::summonitem("4924");
			$var1=true;
		}
		if ($var1!='true'){
			if($item1 > 0){
				quest::say("I don't need this.");
				quest::summonitem("$item1");
			}
			if($item2 > 0){
				quest::say("I don't need this.");
				quest::summonitem("$item2");
			}
			if($item3 > 0){
				quest::say("I don't need this.");
				quest::summonitem("$item3");
			}
			if($item4 > 0){
				quest::say("I don't need this.");
				quest::summonitem("$item4");
			}
		}
	}
}