stfwi / engineers-decor

Engineer's Decor
MIT License
35 stars 16 forks source link

[1.16.5] Block placer does not place blocks, they pop as items. #179

Closed ProsperCraft closed 3 years ago

ProsperCraft commented 3 years ago

https://youtu.be/ktBTaFXooJ4

engineersdecor-1.16.4-1.1.14-b1

ProsperCraft commented 3 years ago

Seems similar to https://github.com/stfwi/engineers-decor/issues/176

stfwi commented 3 years ago

Hi Goshen, gonna take a glimpse what's doing this.

There is a setting that may help analysing this in the common config:

[server.miscellaneous]
  with_debug_logging = true

This prints into the log why the placer does spit out an item. The messages start with Placer spit: ..... Would it be possible to forward the request to enable that and attach the logfile here? Currently the reason can vary between "no placement state retrieved from the block" to an exception when placing. Cheers man!

stfwi commented 3 years ago

Quickly checked, it seems something is blocking the placement. The setup in your video works in my test world and another situation.

2021-05-29-check-placer

I could try to circumvent the problem by forcing a default state placement instead of like-player-clicked-placement, but this may have side effects for other modded blocks, which have to be placed the correct way. Or maybe as a "problem-catch-alternative" placing the default state or so. Let's see, we'll get there ;)

stfwi commented 3 years ago

Sorry for the late notification, I could push a engineersdecor-1.16.4-1.1.14-b2, where I included circumventing a possible placement prevention (e.g. from external hooks which may affect the default Minecraft fake player instance). Would it be possible to check if that helped? Cheers!

stfwi commented 3 years ago

Hi guys, is the trouble gone with version b2? Cheers,-

stfwi commented 3 years ago

Oky, I presume it's fixed ;)

jinkhya commented 3 years ago

Hello, this is happening to me and I'm on 1.1.14. I'm also using them to automate industrial foregoing latex. I dunno how relevant that is.