Engineer's Decor

Engineer's Decor

19M Downloads

Raised catwalks defaultState are waterlogged?

hvdklauw opened this issue · 3 comments

commented

For building gadgets we got an issue report in: Direwolf20-MC/BuildingGadgets#553

I did some digging and it seems the default state for the raised catwalk is waterlogged.

Now I'm not sure if this is a bug or intentional for some reason, but because of placement and block updates we take the default state and then copy all the properties we think are safe to do, waterlogged is not one of them so we don't copy that, but the placed block is waterlogged, so I so seems to come from the default state.

As engineers decor seems to be closed source I have no way to verify this.

We could just always force the waterlogged property to false, but I thought i'd reach out to come to a workable solution.

commented

Hi hvdklauw, I took a quick code peek, you're correct, I have a bug in the constructor of the raised catwalk, in the line here:

setDefaultState(getStateContainer().getBaseState().with(VARIANT, 0));

... (this should be setDefaultState(super.getDefaultState().with(VARIANT, 0))), so that the WATERLOGGED property is initialized (the code is located in MC version branches, the master branch is only the documentation ;-) ).

Thanks for digging into this also accross mods - not many people do that. I gonna dig dive into the matter at the weekend (also schedule the fix for then), and write a quick note in the Building Gadgets parent thread so that it's confirmed that no action is needed there.

commented

Ahh awesome, good to know I can peek at the code, makes future debugging always easier :D

commented

Oky, the fix is in engineersdecor-1.16.4-1.1.4 (up on Curse now). I'll close this issue directly, as the parent issue thread is already closed - if something's coming up we'll reopen it ;-], Cheers,-