I am trying to make it so that when the player shift+right clicks the pig, the pig is unsaddled and the player gets the saddle back.

 

This is my current code:

@SubscribeEvent
public void removeSaddle(EntityInteractEvent event) {
	if (event.target instanceof EntityPig && event.entity instanceof EntityPlayer) {
		EntityPig pig = (EntityPig) event.target;
		EntityPlayer player = (EntityPlayer) event.entity;
		if (pig.getSaddled() && player.getHeldItem() == null && player.isSneaking()) {
			player.inventory.setInventorySlotContents((player.inventory.currentItem), new ItemStack(Items.saddle));
			pig.setSaddled(false);
		}
	}
}

 

Right now, the pig doesn't get unsaddled, and the player only gets the saddle if they're in creative mode. If I comment out the part adding the saddle to the player's inventory, the pig does get unsaddled.

Did, but nothing changed.

Can you elaborate, please?

Makes sense. How do I cancel it?