[1.12] Setting fire to a block [Solved]

[nil]

I'm trying to get a custom projectile to set fire to blocks in the method onhit(raytraceResult) like this:

            BlockPos blockpos = raytraceResultIn.getBlockPos();
            //blockpos = blockpos.add(0, 1, 0);
            IBlockState iblockstate = this.world.getBlockState(blockpos);
            
            
            if (raytraceResultIn.entityHit.world.isAirBlock(blockpos))
            {
            	 raytraceResultIn.entityHit.world.setBlockState(blockpos, Blocks.FIRE.getDefaultState(), 11);
            }
            this.setDead();

similar to ItemFlintAndSteel. However it always crashes the world when used, causing a TickingEntity error.

What is the correct way to have projctile set fire to a block?

Edited November 27, 2019 by nil

[SerpentDagger]

  On 11/26/2019 at 10:58 PM, nil said:

causing a TickingEntity error

Expand  

Post the actual crash log, please.

A couple problems, if this is all the relevant code:

-You assume that the result is always an Entity. This is not necessarily the case.

-You assume that the result always contains a BlockPos. This not necessarily the case, and, in fact, this and the previous assumption are mutually exclusive. No Entity result has a BlockPos, and no Block result has an Entity.

-Unless you know what you're doing, you should probably remove ", 11" from your call of setBlockState() to use the standard one.

 

I suggest you take a look at EntitySmallFireball#onImpact().

[nil]

  On 11/27/2019 at 5:04 AM, SerpentDagger said:

You assume that the result is always an Entity. This is not necessarily the case.

Expand  

It's never the case, i don't even know why i didn't just use this.world in the first place.

 

Everything works now. Thx.

 

[SerpentDagger]

  On 11/27/2019 at 9:58 PM, nil said:

It's never the case

Expand  

I wouldn't go that far. There's a reason RayTraceResult has an entityHit field.

[nil]

I meant in this specific area in my code, my bad.