Should a quest NPC hand money back ?

[ChaosSlayerZ]

hmm Trev, does all measurment done in "copper" ?

for example npc wants "10" - I assume 10 in copper?
so player can give npc 1 silver or 10 copper and both will work?

if yes - then you have just solved entire cash return problem

[trevius]

No, each coin type has it's own variable. If you want to be able to use all coin types, you could do this:

sub EVENT_ITEM {


	if($platinum == 10 || $gold == 100 || $silver == 1000 || $copper == 10000)
	{
		quest::summonitem(1001);
		$platinum = undef;
		$gold = undef;
		$silver = undef;
		$copper = undef;
	}
	else
	{
		if($platinum || $gold  || $silver || $copper) {
			$client->AddMoneyToPP($copper, $silver, $gold, $platinum, updateclient);
			quest::say ("That is not the required amount!");
			$platinum = undef;
			$gold = undef;
			$silver = undef;
			$copper = undef;
		}
	}

}

I haven't tested that yet, but it should work fine I think.

[ChaosSlayerZ]

I see, I can try this out
but it would be MUCH easier if it would have read value of ANY coins given as "copper"

[trevius]

You could do conversions if you want, but you might have to play with it a bit to make sure they work properly.

sub EVENT_ITEM {

#Convert all coin turn-ins to copper and add them up
my $totalcoin = $platinum * 1000;
my $totalcoin = $totalcoin + $gold * 100;
my $totalcoin = $totalcoin + $silver * 10;
my $totalcoin = $totalcoin + $copper;

	if($totalcoin == 10000) # 10 platinum
	{
		quest::summonitem(1001);
		$platinum = undef;
		$gold = undef;
		$silver = undef;
		$copper = undef;
	}
	else
	{
		if($platinum || $gold  || $silver || $copper) {
			$client->AddMoneyToPP($copper, $silver, $gold, $platinum, updateclient);
			quest::say ("That is not the required amount!");
			$platinum = undef;
			$gold = undef;
			$silver = undef;
			$copper = undef;
		}
	}

}

Again, I haven't tested this yet, but it should work almost perfectly for what you are wanting to do.

[ChaosSlayerZ]

we should post this as example into Custom Quest section before it gets lost =P

[cavedude]

Shendare, I am still interested in the plugin version of your code when you get a chance to clean it up

[Shendare]

Yep, working on it. There's a bit of ground to cover.