Vanhal / ProgressiveAutomation

Progressive Automation
Other
49 stars 31 forks source link

PA Wrench not working - Discussing solutions #345

Closed Psycho-Ray closed 6 years ago

Psycho-Ray commented 6 years ago

So, you Shift+Right Click on a miner using the PA Wrench, but all items inside it are lost when you place the miner back on.

I've been trying to fix it for days now, and, as far as I can tell, the title of this Issue is a lie. The Wrench works fine, and the bug happens when you place the block. At this point, the NBT information should be copied from the item stack to the new block you're placing, but that is not happening.

The method that does it should be called by placeBlockAt(), from the ItemBlockMachine class, which will overwrite that method that places vanilla blocks. The strange thing is that no one ever seem to call this class. There are no references to it's constructor, and no classes that extends it.

Psycho-Ray commented 6 years ago

FINALLY. I fixed it... Gods be good, it took days to figure it out.