[1.19.2] Please help me add an effect to the player

[Pherment]

I have this code

package com.pherment.kirillcraft.events;

import com.pherment.kirillcraft.KirillCraft;
import net.minecraftforge.event.entity.EntityMountEvent;
import net.minecraftforge.eventbus.api.SubscribeEvent;
import net.minecraftforge.fml.common.Mod;

@Mod.EventBusSubscriber(modid = KirillCraft.MODID)
public class KCEvents {
    @SubscribeEvent
    public static void setHeadacheWhenPlayerMountingOnBoat(EntityMountEvent event) {
        if (!event.getEntity().level.isClientSide()) {
           if (event.isMounting()) {
                //code add effect
           }
        }
    }
}

And I don't understand how you can give the player an effect. I looked at all the documentation, and livingEntities and mobEffects and everything that can be related to effects and the player. But nothing helps or works. I even tried this code. It would seem that it should work, but it does not work and minecraft gives an error

@Mod.EventBusSubscriber(modid = KirillCraft.MODID)
public class KCEvents {
    @SubscribeEvent
    public static void setHeadacheWhenPlayerMountingOnBoat(EntityMountEvent event, LivingEntity player) {
        if (!event.getEntity().level.isClientSide()) {
           if (event.isMounting()) {
                player.addEffect(new MobEffectInstance(KCEffect.HEADACHE.get(), 400));
           }
        }
    }
}

 

[warjort]

This is the correct signature for an event.

   public static void setHeadacheWhenPlayerMountingOnBoat(EntityMountEvent event) {

 

Not just players can mount boats. Many people use them for transporting animals or villagers.

You need to check the entity mounting the boat is a player.

You also need to check it is actually a boat being mounted.

Entity passenger = event.getEntityMounting();
Entity vehicle = event.getEntityBeingMounted();
if (event.isMounting() && vehicle instanceof Boat && passenger instanceof Player player) {
    player.addEffect(new MobEffectInstance(KCEffect.HEADACHE.get(), 400));
}

 

Quote

but it does not work and minecraft gives an error

Why did you not share this error? Do you think we have psychic powers? 🙂 

[Pherment]

4 minutes ago, warjort said:

This is the correct signature for an event.

   public static void setHeadacheWhenPlayerMountingOnBoat(EntityMountEvent event) {

 

Not just players can mount boats. Many people use them for transporting animals or villagers.

You need to check the entity mounting the boat is a player.

You also need to check it is actually a boat being mounted.

Entity passenger = event.getEntityMounting();
Entity vehicle = event.getEntityBeingMounted();
if (event.isMounting() && vehicle instanceof Boat && passenger instanceof Player player) {
    player.addEffect(new MobEffectInstance(KCEffect.HEADACHE.get(), 400));
}

 

Why did you not share this error? Do you think we have psychic powers? 🙂 

https://imgur.com/a/OlKHubc

[Pherment]

8 minutes ago, warjort said:

This is the correct signature for an event.

   public static void setHeadacheWhenPlayerMountingOnBoat(EntityMountEvent event) {

 

Not just players can mount boats. Many people use them for transporting animals or villagers.

You need to check the entity mounting the boat is a player.

You also need to check it is actually a boat being mounted.

Entity passenger = event.getEntityMounting();
Entity vehicle = event.getEntityBeingMounted();
if (event.isMounting() && vehicle instanceof Boat && passenger instanceof Player player) {
    player.addEffect(new MobEffectInstance(KCEffect.HEADACHE.get(), 400));
}

 

Why did you not share this error? Do you think we have psychic powers? 🙂 

Thanks, this helped! I feel stupid because it was so easy