Multiple item checker and nuker on EVENT_TARGET_CHANGE. Need help.

[lerxst2112]

lol, I made it bold in mine too, but making it giant is a sure win. <3 Secrets

[Drakiyth]

The extra ) actually broke the script on the npc and wasn't needed. (I already said I tested this on an NPC with the single ")") It was fine as is. Thanks for trying to help though. This call doesn't work with this EVENT type so it's impossible to do with it. I asked if you all knew a way to get this to work but I doubt it. I'll have to figure something else out.

NOTE: If I knew the answer to something I would of already made a script for the person who was asking for help. If you want to show you're true tech gods that know anything it would be better if you actually showed it instead of act like I should know everything already. If I did I wouldn't be asking for help. "Closing" that isn't even needed. The extra "IF" was a mistake but the single ) was fine. Try the script for yourself on an NPC.

[trevius]

First of all, with your attitude in your last reply, you really don't deserve further help. The answer has already been handed to you and you are complaining about that.

I tested the following script just now on a player.pl in my blackburrow zone and it worked perfectly fine.

sub EVENT_TARGET_CHANGE {

	if (plugin::check_hasitem($client, 2632))
	{
		quest::ding();
		$client->Message(13, "Target Changed");
	}

}

Note that I used a different item ID, because I just picked a random item my character had on them for testing. I also didn't use your nuke item, but used a debug message instead to prove it was getting that far and it did.

Also, anytime you have an opening parenthesis "(", or curly bracket "{", you always need a closing one that goes with that specific one. I highly recommend you download notepad++ for your script editing, because it will make it really easy for you to make sure you have your brackets and parenthesis properly closed. Just click next to the one you want to check and it highlights the other one in red.

The people that were responding in this thread to help you have been doing this stuff for years, so they know what they are talking about.

If you are still having a problem with that script, then your issue is most likely with the plugin itself. Maybe you don't have plugins in the correct folder. By default from PEQ, the plugin folder is inside the quest folder. You need to move it directly into your server folder for them to actually work.

As mentioned earlier, you are using && in your example which means they need to have both items for the nuke item to happen. What you want is || for OR instead of AND. This should work:

sub EVENT_TARGET_CHANGE {

	if(plugin::check_hasitem($client, 4202) || plugin::check_hasitem($client, 4203))
	{
		$client->Message(13, "Nuking Items");
		$client->NukeItem(4202);
		$client->NukeItem(4203);
	}

}

[Secrets]

I didn't mean it as offensive, I was just pointing out a syntax error. Trevius' reply is right though, the script he posted should work.

I didn't mean to come of as a know-it-all. I just wanted to help.

[chrsschb]

I use a shit load of check_hasitem plugins on my server, Trevius is 100% right!