PlayerInteract is called multiple times

[RobinCirex]

Hey so I'm working on a Survival Games mod right now and I'm using the PlayerInteractEvent for some things. However, it is called multiple times and I have no clue why. It is only registered once but gets called more than once, when aiming on something that isn't air. Can someone help me please

[desht]

Interact evens are fired on both the client and server, and potentially for both the main and off hands.

[DragonITA]

First, if you need help show the full mod on Github or Gitlab. Second: are you working on a weapon or what else? Third: Are you use it on the on both sides or Client or Server side?

[RobinCirex]

  On 2/27/2020 at 3:34 PM, DragonITA said:

First, if you need help show the full mod on Github or Gitlab. Second: are you working on a weapon or what else? Third: Are you use it on the on both sides or Client or Server side?

Expand  

First: I don't have it on github, but I can do it if necessary
Second: I'm just using a piece of paper to vote for a map. 

Third: I have a check if it's serverside

 

It gets called twice when aiming on a block and I found out that there is PlayerInteractEvent.RightClickEmpty but that only works Clientside, though the mod is completely serverside

[DragonITA]

Why do you need that your mod was completely on the serverside?

[RobinCirex]

  On 2/27/2020 at 4:07 PM, DragonITA said:

Why do you need that your mod was completely on the serverside?

Expand  

Because that's what the person I'm programing the mod for wants. He said the mod should be serverside only

[RobinCirex]

  On 2/27/2020 at 4:15 PM, diesieben07 said:

Need more info.

How many times? Show your code.

Expand  

	@SubscribeEvent
	public void onInteract(PlayerInteractEvent.RightClickEmpty event) {
		if (event.getWorld() instanceof ServerWorld) {
			if (event.getSide() == LogicalSide.SERVER) {
				if (event.getHand() == Hand.MAIN_HAND) {
					if (event.getPlayer().getHeldItem(Hand.MAIN_HAND) != null) {
						ItemStack item = event.getPlayer().getHeldItem(Hand.MAIN_HAND);
						ServerPlayerEntity player = (ServerPlayerEntity) event.getPlayer();
						String displayName = item.getDisplayName().getFormattedText();
						if (item.getItem() == Items.PAPER) {
							if (HungerGames.getInstance().getMapManager().maps.contains(displayName)) {
								HungerGames.getInstance().getVoteManager().vote(displayName, player);
							}
						} else if (item.getItem() == Items.ANVIL) {
							boolean selecting = HungerGames.getInstance().getKitManager().kitSelecting.get(player);

							if (selecting) {
								HungerGames.getInstance().getVoteManager().addItems(player);
								HungerGames.getInstance().getTeamManager().addItems(player);
								HungerGames.getInstance().getKitManager().addItems(player);

								ChatUtil.sendMessage(player, HungerGames.getInstance().getMessageManager().messages
										.get("kitChosen").replaceAll("%kit%", displayName));
								HungerGames.getInstance().getKitManager().chosenKits.put(player, displayName);

							} else {
								HungerGames.getInstance().getKitManager().addKitItems(player);
							}
							HungerGames.getInstance().getKitManager().kitSelecting.put(player, !selecting);
						} else if (item.getItem() == Items.BOOK) {
							if (HungerGames.getInstance().getMapManager().maps.contains(displayName)) {
								HungerGames.getInstance().getAdminManager().selectMap(displayName, player);
							}
						}
					}
				}
			}
		}
	}

This is the code. It gets executed once when looking in the air and twice when aiming on a block

[RobinCirex]

  On 2/27/2020 at 4:50 PM, diesieben07 said:

RightClickEmpty is fired when aiming at nothing ("looking in the air"). It is not fired when aiming at a block.

It is not possible for this event handler to fire when right-clicking a block.

Expand  

oh, yea, sorry. I only had PlayerInteractEvent in there before, I tried it out with RightClickEmpty, then it wasn't fired at all. Looks like it is a client-side event. 

This was my actual code

@SubscribeEvent
	public void onInteract(PlayerInteractEvent event) {
		if (event.getWorld() instanceof ServerWorld) {
			if (event.getSide() == LogicalSide.SERVER) {
				if (event.getHand() == Hand.MAIN_HAND) {
					if (event.getPlayer().getHeldItem(Hand.MAIN_HAND) != null) {
						ItemStack item = event.getPlayer().getHeldItem(Hand.MAIN_HAND);
						ServerPlayerEntity player = (ServerPlayerEntity) event.getPlayer();
						String displayName = item.getDisplayName().getFormattedText();
						if (item.getItem() == Items.PAPER) {
							if (HungerGames.getInstance().getMapManager().maps.contains(displayName)) {
								HungerGames.getInstance().getVoteManager().vote(displayName, player);
							}
						} else if (item.getItem() == Items.ANVIL) {
							boolean selecting = HungerGames.getInstance().getKitManager().kitSelecting.get(player);

							if (selecting) {
								HungerGames.getInstance().getVoteManager().addItems(player);
								HungerGames.getInstance().getTeamManager().addItems(player);
								HungerGames.getInstance().getKitManager().addItems(player);

								ChatUtil.sendMessage(player, HungerGames.getInstance().getMessageManager().messages
										.get("kitChosen").replaceAll("%kit%", displayName));
								HungerGames.getInstance().getKitManager().chosenKits.put(player, displayName);

							} else {
								HungerGames.getInstance().getKitManager().addKitItems(player);
							}
							HungerGames.getInstance().getKitManager().kitSelecting.put(player, !selecting);
						} else if (item.getItem() == Items.BOOK) {
							if (HungerGames.getInstance().getMapManager().maps.contains(displayName)) {
								HungerGames.getInstance().getAdminManager().selectMap(displayName, player);
							}
						}
					}
				}
			}
		}
	}

 

[RobinCirex]

  On 2/27/2020 at 5:38 PM, diesieben07 said:

Do not subscribe to event superclasses. Choose the appropriate sub-event for your use-case.

Expand  

yea, I know. But it isn't called, only clientside

Edited February 27, 2020 by RobinCirex
typo

[RobinCirex]

  On 2/27/2020 at 10:16 PM, diesieben07 said:

Show updated code.

Expand  

I already fixed it by using PlayerInteractEvent.RightClickItem

Thank you though