Multiple item checker and nuker on EVENT_TARGET_CHANGE. Need help.

Drakiyth · #1 05-05-2012, 10:08 PM

player.pl <--- Possible?

Code:

sub EVENT_TARGET_CHANGE {

if(plugin::check_hasitem($client, xxxx) && if(plugin::check_hasitem($client, xxxx)
{
$client->NukeItem(xxxx);
$client->NukeItem(xxxx);
}
}

I put this in templates named as player.pl and it's purpose is to check 2 or more items on the player at once as soon as they switch targets and if they break this rule the nuke goes off and eliminates all of the items. (it's an anti-cheat for having items you're not supposed to.) This however is not working when I try it with a couple test items. Is there anyway to make this work?

NOTE: The player is not allowed to have both of these items and that's why the anti cheat burns them.

**joligario** · #2 05-05-2012, 10:20 PM

Need only 1 if and close your parenthesis

oslander · #3 05-05-2012, 10:29 PM

Quote:

Originally Posted by joligario

Need only 1 if and close your parenthesis

Code:

sub EVENT_TARGET_CHANGE {

if(plugin::check_hasitem($client, 4202) && plugin::check_hasitem($client, 4203)
{
$client->NukeItem(4202);
$client->NukeItem(4203);
}

I think this is what he means. I used bronze mask and collar as an example.

lerxst2112 · #4 05-05-2012, 10:46 PM

Close...

Code:

if(plugin::check_hasitem($client, 4202) && plugin::check_hasitem($client, 4203))

**joligario** · #5 05-05-2012, 11:29 PM

And just a note: the way you have it only triggers if they have both.

Drakiyth · #6 05-06-2012, 01:05 AM

Quote:

Originally Posted by joligario

And just a note: the way you have it only triggers if they have both.

It's not working. I switch the target and the items aren't deleted. Does the NukeItem feature not work with this or is there something wrong with this script?
EDIT: I forget to close it originally, a small mistake but it's not working even closed below.

Code:

#Player.pl  Under the Wayfarer's message in Templates.

sub EVENT_TARGET_CHANGE {

if(plugin::check_hasitem($client, 4202) && plugin::check_hasitem($client, 4203))
{
$client->NukeItem(4202);
$client->NukeItem(4203);
}

}

lerxst2112 · #7 05-06-2012, 02:06 AM

Add some debugging information to the script to make sure it is even running. Add messages at each stage so you can see where it goes wrong, and simplify it into small steps you can test individually and then put them together.

Drakiyth · #8 05-06-2012, 05:12 AM

This is goofy. Why does it do this?

THIS WILL PLAY A FANFARE AND NUKE THE CHOSEN ITEMS EVERY TIME A
DIFFERENT TARGET IS CHOSEN OR DROPPED.

Code:

# File is player.pl in the templates folder.

sub EVENT_TARGET_CHANGE {

quest::ding();
$client->NukeItem(4202);
$client->NukeItem(4203);
}

}

THIS WILL NOT DO ANYTHING. THE PLUGIN IS CORRECT BECAUSE I TESTED IT ON A NPC INSTEAD OF PLAYER.PL

Code:

# File is player.pl in the templates folder.

sub EVENT_TARGET_CHANGE {

if(plugin::check_hasitem($client, "4202")
{
quest::ding();
$client->NukeItem(4202);
$client->NukeItem(4203);
}

}

So what in the world do I need to do to check for those items that it's not passing through to get to the next stage on EVENT_TARGET_CHANGE or is it impossible?

Secrets · #9 05-06-2012, 07:44 AM

Code:

# File is player.pl in the templates folder.

sub EVENT_TARGET_CHANGE {

if(plugin::check_hasitem($client, "4202") )
{
quest::ding();
$client->NukeItem(4202);
$client->NukeItem(4203);
}

}

You are missing a closing ). I highlighted where it was missing. A perl interpreter will tell you these errors if you run "perl script.pl" on the windows command line.

If it shows nothing when you run it, then it's an issue with the emulator code and not a syntax error.

lerxst2112 · #10 05-06-2012, 05:44 PM

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

<3 Secrets

Drakiyth · #11 05-06-2012, 09:15 PM

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** · #12 05-06-2012, 10:22 PM

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.

Code:

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:

Code:

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 · #13 05-06-2012, 11:10 PM

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 · #14 05-07-2012, 12:10 AM

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

Drakiyth · #15 05-07-2012, 02:45 AM

I apologize if I came off rude. I'm just extremely stressed out right now with trying to fix this stuff up. I decided to make the first tier and higher path charms epic lore so now people can't have two of them in their inventory at once, however bots are bringing in a problem with being able to stash these charms. Trevius, Secrets, Lerxst, Joligaro, Oslander, thank you for your help.

Would you good folks know how to do away with unwanted items on bots by any chance?