Jump to content

[1.15.2] Had a problem applying potion effects to the player


Recommended Posts

I have an eventlistener which applies random potion effects to the player, but it works properly until player is dead. After player respawns function applies only specific and non-instant effects, so i have to restart the game to have the function work properly again.

Here's the code: 

EffectInstance effect;
                switch(*random n*)
                {case *random n*: effect=Potions.*Potion_Name*.getEffects().get(0);}

LivingEntity entity=(LivingEntity) src.getTrueSource();

entity.addPotionEffect(effect) //or entity.addPotionEffect(effect.getEffectInstance()); it doesn't matter.



-src.getTrueSource() is always instance of PlayerEntity

-default defined too

-i did Event listener according to forge docs, tried static and everything works till the player is dead.

-Then it works, but improperly (applies some effects with very short duration, about few ticks, maybe 1, so they don't have enough time to trigger).


Don't know why the problem takes place and what else i can do to make this work :(


Link to comment
Share on other sites

1 hour ago, diesieben07 said:

It seems like you are reusing EffectInstance instances. You cannot do that.

1) What do you mean reusing? Every time event happens i create new variables pick the player and apply the effect.

2) I also tried to call addPotionEffect directly in switch cases without any unnecessary variables and got the same issue.

Link to comment
Share on other sites

40 minutes ago, diesieben07 said:

Post your actual code instead of random snippets.

public class myEvents {
    public void DeathPotion(LivingDeathEvent event){
        DamageSource src=null;

            if(src.getTrueSource() instanceof PlayerEntity)

                LivingEntity entity=(LivingEntity) src.getTrueSource();
                int n=13;
                int random=(int) (Math.random()*n);
                EffectInstance killer;
                case 0:
                case 1:
                case 2:
                case 3:
                case 4:
                case 5:
                case 6:
                case 7:
                case 8:
                case 9:
                case 10:
                case 11:
                case 12:


This is registration:

myEvents m=new myEvents();

Edited by diesieben07
code formatting
Link to comment
Share on other sites

29 minutes ago, diesieben07 said:

This is the same EffectInstance every time. You need to copy it first using the respective constructor of the EffectInstance class. You can see this being done in PotionItem#onItemUseFinish.

Seems like everything is working now, thanks ?

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.

Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
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.


  • Create New...

Important Information

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