Linking item and capability

[Blooode]

I can't link using item with thirst capability. I want to use item and add thirst points after that. I try lots of times, that is one of examples. Version 1.19.2

@SubscribeEvent
	    public void OnDrinking(LivingEntityUseItemEvent.Finish event)
	    {
			LivingEntity player = event.getEntity();
	        if (event.getItem().equals(Items.APPLE))
	        {
	        	player.getCapability(PlayerThirstProvider.PLAYER_THIRST).ifPresent(thirst -> {
	        		thirst.addThirst(10);
	            });
	        }
	    }

 

[warjort]

https://forge.gemwire.uk/wiki/Capabilities/Attaching

[Blooode]

I have attached capability. 

@SubscribeEvent
	    public static void OnAttachCapabilitiesPlayer(AttachCapabilitiesEvent<Entity> event) {
	        if(event.getObject() instanceof Player) {
	            if(!event.getObject().getCapability(PlayerThirstProvider.PLAYER_THIRST).isPresent()) {
	                event.addCapability(new ResourceLocation(HardLife.MODID, "properties"), new PlayerThirstProvider());
	            }
	        }
	    }

This capability working, but I can't understand how to add thirst points via using items such food or bottle of water.

[Blooode]

Also I have method for add thirst by pushing the keybord button.

public boolean handle(Supplier<NetworkEvent.Context> supplier) {
        NetworkEvent.Context context = supplier.get();
        context.enqueueWork(() -> {
            ServerPlayer player = context.getSender();
            ServerLevel level = player.getLevel();
            	
            if(hasWaterAroundThem(player, level, 2)) {
                player.sendSystemMessage(Component.translatable(MESSAGE_DRINK_WATER).withStyle(ChatFormatting.DARK_AQUA));
                level.playSound(null, player.getOnPos(), SoundEvents.GENERIC_DRINK, SoundSource.PLAYERS,
                        0.5F, level.random.nextFloat() * 0.1F + 0.9F);
                player.getCapability(PlayerThirstProvider.PLAYER_THIRST).ifPresent(thirst -> {
                    thirst.addThirst(1);
                    player.sendSystemMessage(Component.literal("Current Thirst " + thirst.getThirst())
                            .withStyle(ChatFormatting.AQUA));
                    ModMessages.sendToPlayer(new ThirstDataSyncS2CPacket(thirst.getThirst()), player);
                });


            } else {
                player.sendSystemMessage(Component.translatable(MESSAGE_NO_WATER).withStyle(ChatFormatting.RED));
                player.getCapability(PlayerThirstProvider.PLAYER_THIRST).ifPresent(thirst -> {
                    player.sendSystemMessage(Component.literal("Current Thirst " + thirst.getThirst())
                            .withStyle(ChatFormatting.AQUA));
                    ModMessages.sendToPlayer(new ThirstDataSyncS2CPacket(thirst.getThirst()), player);
                });
            }
        });
        return true;
    }

Maybe that can be made in this method?

[Blooode]

UP

[warjort]

You don't have an answerable question. All you say is something about your code doesn't work.

The code you do show references other code you don't show and lacks context, so we can't even reverse engineer (guess) what your problem is.

For a question about capabilities, you don't even show yours.

 

One way to get people not to ignore your question is to put your code on github where they can run it for themselves (and see all the relevant code).

But if you don't clearly define what the problem is, nobody can help you.

"Please find the bug somewhere in this random code I wrote", is question that won't attract many answers.