Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

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


Pherment
 Share

Recommended Posts

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));
           }
        }
    }
}

 

Link to comment
Share on other sites

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? 🙂 

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

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

Link to comment
Share on other sites

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

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.