Help Item restriction

[Katara512]

Hello forum , im wondering how can i make my event restricted to only 1 type of weapon, im using use item by id command but i need some further help 

[melron]

Similar to oly restrictions.

 

2 things you should care about.

1st) check the type while entering at the event or teleporting? idk, and unequip it

2nd) when useItem will launch

[Katara512]

5 minutes ago, melron said:

Similar to oly restrictions.

 

2 things you should care about.

1st) check the type while entering at the event or teleporting? idk, and unequip it

2nd) when useItem will launch

the problem is that im forcing the player to equip a specific id but while the event is running he can change it 

[melron]

2 minutes ago, Sidereal Day said:

the problem is that im forcing the player to equip a specific id but while the event is running he can change it 

can you be more descriptive? give us examples or atleast give some code to help you

[Katara512]

2 minutes ago, melron said:

can you be more descriptive? give us examples or atleast give some code to help you

public boolean canUseItem(PlayerEventInfo player, ItemData item)
	{
		if(item.getItemId() == _bowItemId && item.isEquipped())
			return false;
  		return super.canUseItem(player, item);
	}

i want to make this part of code to

{
if(item.getItemId() != _bowItemId)
			player.sendMessage(LanguageEngine.getMsg("you cant use another weapon than the Bow"));
}

something like this 

[melron]

8 minutes ago, Sidereal Day said:

public boolean canUseItem(PlayerEventInfo player, ItemData item)
	{
		if(item.getItemId() == _bowItemId && item.isEquipped())
			return false;
  		return super.canUseItem(player, item);
	}

i want to make this part of code to

{
if(item.getItemId() != _bowItemId)
			player.sendMessage(LanguageEngine.getMsg("you cant use another weapon than the Bow"));
}

something like this 

you confused me. you have a method with name canUseItem.

You are checking if the item id is equal to ex. 7575 (db) and if yes & is equipped you returning false ? it means you cant use that? so you cant even unequip it?

 

you have to edit your method and check only the item id. so..

 

 

public boolean canUseItem(PlayerEventInfo player, ItemData item)
	{
		if(!item.isEquipped())
			return item.getItemId() == _bowItemId;
  		return super.canUseItem(player, item);
	}

 

 

[Katara512]

18 minutes ago, melron said:

you confused me. you have a method with name canUseItem.

You are checking if the item id is equal to ex. 7575 (db) and if yes & is equipped you returning false ? it means you cant use that? so you cant even unequip it?

 

you have to edit your method and check only the item id. so..

 

 

public boolean canUseItem(PlayerEventInfo player, ItemData item)
	{
		if(!item.isEquipped())
			return item.getItemId() == _bowItemId;
  		return super.canUseItem(player, item);
	}

 

 

will this code force the player to use only 1 specific id ? because thats what im trying to do 

[SweeTs]

If your code is working fine, but it's missing check onEquip. Why not simply go to (Request)UseItem, from memory, and add a check like 

 

if (player.isInEvent() && item.getId() != 7575)

{

   player.sendMessage("you can only equip 7575 Id".);

   return;

} 

[Kara]

11 minutes ago, SweeTs said:

If your code is working fine, but it's missing check onEquip. Why not simply go to (Request)UseItem, from memory, and add a check like 

 

if (player.isInEvent() && item.getId() != 7575)

{

   player.sendMessage("you can only equip 7575 Id".);

   return;

} 

Use Code tag next time. Last time i warn you.

 

@Sidereal Day you can simply go in UseItem.java (or idk on which project you're working on) and add a simple check as SweeTs said such as:

 

if (item.getItemType() == L2WeaponType.BOW)
{
	return;
}

To block BOW from any specific event you want.

Edited February 22, 2019 by Kara`

[SweeTs]

He want to block all, allow one kind of a wep, so yeah. Item Id check, as I suggested, is better, easier, faster.

 

@Kara`

Ps: next time share code which uses enum :3

[Katara512]

If i only could turn my logical idea on code it would be so easy, as sweets said  i just want to prevent the player to use another weapon than my specific id on this event , so i think im going to use that code sweets said 

 

----EDIT----

what return should i use because my class is public boolen not void so i cant return nothing

Edited February 23, 2019 by Sidereal Day

[SweeTs]

False. Eventually you could add a check regarding item type / instance. So, you will restrict only weapon type items, so you can freely change armor and such.

[Katara512]

9 hours ago, SweeTs said:

False. Eventually you could add a check regarding item type / instance. So, you will restrict only weapon type items, so you can freely change armor and such.

can you explain further what you mean by "(Request)UseItem, from memory" ?

[SweeTs]

54 minutes ago, Sidereal Day said:

can you explain further what you mean by "(Request)UseItem, from memory" ?

The file, it's name, RequestUseItem.java or UseItem.java. Normally it's UseItem.java, under network/clientpacket.

 

I would put the check directly there.

[Katara512]

34 minutes ago, SweeTs said:

The file, it's name, RequestUseItem.java or UseItem.java. Normally it's UseItem.java, under network/clientpacket.

 

I would put the check directly there.

ok found it , one more question if if the method is IsInEvent so the function will work for the rest of the events ? and 1 more, to validate the check on the ItemUse i have to add a return on canUseItem variable ?

[DenArt Designs]

14 hours ago, SweeTs said:

False.

if you dont know this show The Office you have to see it