Jump to content

Event for Riding Entities Underwater


GhostWolf2398

Recommended Posts

Hello, I am GhostWolf, one of the coding developers for the Pixelmon mod (essentially Pokemon in Minecraft).

 

We were wondering if it would be possible to add an event into entityLivingBase that would allow us to specify custom behavior for when a player rides an entity underwater. We currently have several mounts that are capable of surfing and diving, and with the code in entityLivingBase lines 281-309 (and 305-309 in particular), as soon as a player goes fully underwater they unmount (whether it is just the natural up and down swimming on the mount or actually diving). Ideally we would be able to just override this or change the event response to this so that the player can remain on the mount.

 

Any help and suggestions you can give on this would be greatly appreciated. We would also be more than happy to take care of the needed coding if you had the way this should be done figured out.

 

 

 

        if (this.isEntityAlive() && this.isInsideOfMaterial(Material.water))

        {

            if (!this.canBreatheUnderwater() && !this.isPotionActive(Potion.waterBreathing.id) && !flag)

            {

                this.setAir(this.decreaseAirSupply(this.getAir()));

 

                if (this.getAir() == -20)

                {

                    this.setAir(0);

 

                    for (int i = 0; i < 8; ++i)

                    {

                        float f = this.rand.nextFloat() - this.rand.nextFloat();

                        float f1 = this.rand.nextFloat() - this.rand.nextFloat();

                        float f2 = this.rand.nextFloat() - this.rand.nextFloat();

                        this.worldObj.spawnParticle("bubble", this.posX + (double)f, this.posY + (double)f1, this.posZ + (double)f2, this.motionX, this.motionY, this.motionZ);

                    }

 

                    this.attackEntityFrom(DamageSource.drown, 2.0F);

                }

            }

 

            this.extinguish();

 

            if (!this.worldObj.isRemote && this.isRiding() && this.ridingEntity instanceof EntityLivingBase)

            {

                this.mountEntity((Entity)null);

            }

        }

 

Link to comment
Share on other sites

Could easily add a boolean Entity.shouldDismountWhenUnderwater(), Should be a Simple PR for someone to make. {I iz busy till the end of next week}

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Link to comment
Share on other sites

I went ahead and wrote in the needed code. I can't get the patches to update or commit correctly to github, so here is the code for someone who can do that, all written out for you (red text is added, the rest is just to help you locate where to put it).

 

/** Number of ticks since last jump */

    private int jumpTicks;

    private float field_110151_bq;

   

    /** Determines if entity stays on mount when submerged in water. */

    public boolean stayMountedUnderWater = false;

 

    public EntityLivingBase(World par1World)

    {

 

and

 

                    this.attackEntityFrom(DamageSource.drown, 2.0F);

                }

            }

 

            this.extinguish();

 

            if (!this.worldObj.isRemote && this.isRiding() && this.ridingEntity instanceof EntityLivingBase)

            {

            if (!stayMountedUnderWater){

            this.mountEntity((Entity)null);

            }

            }

        }

 

Link to comment
Share on other sites

This is the location in the code that causes the issue with underwater riding. Based on your comment, I looked at putting something inside the mountEntity() function, but that is only called when you change from mounting to unmounting, and that boolean variable if used there could cause other behavioral issues (such as never letting you get off the mount, etc)

 

The line I placed the check at is the only point at which you would want to circumvent the normal unmounting code for this kind of problem, as it is the only point where the unmounting is caused by the rider being located in water. By locating that boolean check there, you only get the bypass behavior under that one condition, and riding should behave entirely normally for all other situations.

 

If you meant a function in the code of the entity being mounted, it likely wouldn't solve the issue we are having, as it is this particular point in the code where it forces the unmounting, and any other function in the mounted entity that would say force re-mounting would be run too frequently while under water to be an elegant solution to the issue.

 

Hopefully that makes sense, feel free to correct me if I am wrong.

Link to comment
Share on other sites

            if (!this.worldObj.isRemote && this.isRiding() && this.ridingEntity instanceof EntityLivingBase)
            {
                  this.mountEntity((Entity)null);
            }

To:

            if (!this.worldObj.isRemote && this.isRiding() && ridingEntity != null && ridingEntity.shouldDismountInWater(this))
            {
                this.mountEntity((Entity)null);
            }

 

and in entity you have:

public boolean shouldDismountInWater(Entity rider)
{
    return this instanceof EntityLivingBase;
}

...

How is that confusing? Give the mounted the choice of kicking the mountee off if it so chooses, Which mimics what this is doing.

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Link to comment
Share on other sites

That makes a lot of sense.

 

I was thinking you were suggesting only making the dismounting code inside the mount's code, and not doing the check at that line in the entityLivingBase code, since that wouldn't change things.

 

With the way you have it, it would give the ridden entity the control over kicking, which makes it more generally customizable, and it will make the check at the right place in code, so that is the best solution to the issue. Thanks for your help and insight.

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
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.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • They were already updated, and just to double check I even did a cleanup and fresh update from that same page. I'm quite sure drivers are not the problem here. 
    • i tried downloading the drivers but it says no AMD graphics hardware has been detected    
    • Update your AMD/ATI drivers - get the drivers from their website - do not update via system  
    • As the title says i keep on crashing on forge 1.20.1 even without any mods downloaded, i have the latest drivers (nvidia) and vanilla minecraft works perfectly fine for me logs: https://pastebin.com/5UR01yG9
    • Hello everyone, I'm making this post to seek help for my modded block, It's a special block called FrozenBlock supposed to take the place of an old block, then after a set amount of ticks, it's supposed to revert its Block State, Entity, data... to the old block like this :  The problem I have is that the system breaks when handling multi blocks (I tried some fix but none of them worked) :  The bug I have identified is that the function "setOldBlockFields" in the item's "setFrozenBlock" function gets called once for the 1st block of multiblock getting frozen (as it should), but gets called a second time BEFORE creating the first FrozenBlock with the data of the 1st block, hence giving the same data to the two FrozenBlock :   Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=head] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@73681674 BlockEntityData : id:"minecraft:bed",x:3,y:-60,z:-6} Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=3, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=2, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} here is the code inside my custom "freeze" item :    @Override     public @NotNull InteractionResult useOn(@NotNull UseOnContext pContext) {         if (!pContext.getLevel().isClientSide() && pContext.getHand() == InteractionHand.MAIN_HAND) {             BlockPos blockPos = pContext.getClickedPos();             BlockPos secondBlockPos = getMultiblockPos(blockPos, pContext.getLevel().getBlockState(blockPos));             if (secondBlockPos != null) {                 createFrozenBlock(pContext, secondBlockPos);             }             createFrozenBlock(pContext, blockPos);             return InteractionResult.SUCCESS;         }         return super.useOn(pContext);     }     public static void createFrozenBlock(UseOnContext pContext, BlockPos blockPos) {         BlockState oldState = pContext.getLevel().getBlockState(blockPos);         BlockEntity oldBlockEntity = oldState.hasBlockEntity() ? pContext.getLevel().getBlockEntity(blockPos) : null;         CompoundTag oldBlockEntityData = oldState.hasBlockEntity() ? oldBlockEntity.serializeNBT() : null;         if (oldBlockEntity != null) {             pContext.getLevel().removeBlockEntity(blockPos);         }         BlockState FrozenBlock = setFrozenBlock(oldState, oldBlockEntity, oldBlockEntityData);         pContext.getLevel().setBlockAndUpdate(blockPos, FrozenBlock);     }     public static BlockState setFrozenBlock(BlockState blockState, @Nullable BlockEntity blockEntity, @Nullable CompoundTag blockEntityData) {         BlockState FrozenBlock = BlockRegister.FROZEN_BLOCK.get().defaultBlockState();         ((FrozenBlock) FrozenBlock.getBlock()).setOldBlockFields(blockState, blockEntity, blockEntityData);         return FrozenBlock;     }  
  • Topics

×
×
  • Create New...

Important Information

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