Slimefun / Slimefun4

Slimefun 4 - A unique Spigot/Paper plugin that looks and feels like a modpack. We've been giving you backpacks, jetpacks, reactors and much more since 2013.
GNU General Public License v3.0
973 stars 546 forks source link

Animal Growth Accelerator sometimes turning into a regular hay bale when breaking #3395

Closed marvjosh1 closed 11 months ago

marvjosh1 commented 2 years ago

ā— Checklist

šŸ“ Description

When placing down the Animal Growth Accelerator and then breaking it, it sometimes turns into a regular hay bale (does also not throw any errors in console)

šŸ“‘ Reproduction Steps

  1. Place the Animal Growth Accelerator
  2. Break it Repeat as this only occasionally happens

šŸ’” Expected Behavior

The Animal Growth Accelerator to be returned as the machine

šŸ“· Screenshots / Videos

https://youtu.be/I3ovQpz5Cmo https://youtu.be/ZEVpHCGHq9A

šŸ“œ Server Log

No response

šŸ“‚ /error-reports/ folder

No response

šŸ’» Server Software

Purpur

šŸŽ® Minecraft Version

1.17.x

ā­ Slimefun version

https://i.imgur.com/XPPsX4B.png https://i.imgur.com/b4ZWRqo.png

šŸ§­ Other plugins

No response

Boomer-1 commented 2 years ago

were there any errors in the console or the slimefun folder? I've tried over 100 times mining one and it never dropped as anything but the animal growth accelerator. I tested with every official addon on my server and couldn't reproduce this

marvjosh1 commented 2 years ago

No error in the console or anything - but as you can see it kinda happens sometimes from the youtube videos

Boomer-1 commented 2 years ago

I just tried it again letting the accelerator sit for varying amounts of time, from mining right away to sitting for 30 seconds, next to, away from and different lengths from the power regulator and cannot produce this. Now I've only tested it with all slimefun plugins on the server, no essentials, cmi, or any of the custom slimefun plugins void realms has. So far it would appear another plugin may be interfering with it. If you have a test server where you can load up the official slimefun plugins, then one at a time add the others until the culprit is found?

variananora commented 2 years ago

I can't seems to replicate this. If you have something feel free to reply on this or tag me on Discord.

https://youtu.be/yCIuUHxmjaQ

image

Skizzles commented 2 years ago

Was able to reproduce with just Slimefun installed: https://screenshots.skizzles.me/wAnA3/Kofimixi49.mp4

Simply by placing and removing blocks quickly causes the item to break.

Boomer-1 commented 2 years ago

Variananora, I gave it a second run and with some consistency I was able to reproduce it as well. the debug fish doesn't recognize it and the paper dumpitem command showed no nbt data. It truly is a haybale. This time I was on dev 994 in 1.18.1.

Skizzles commented 2 years ago

Mine is on 1.17.1 and dev 994

WalshyDev commented 2 years ago

I feel like making a debug thing for you guys. Something that dumps all the events being fired and which are being handled by Slimefun. Then it can also dump a bunch of other info out.

Could use the existing debug function for that (just need to make sure regular people don't run it... it'll make a huge fucking log)

variananora commented 2 years ago

Simply by placing and removing blocks quickly causes the item to break.

The easiest way to do this is just to hold right click when you are breaking the blocks. I seems to been able to repro this with other blocks as well, the one I tested and works are:

Video: https://youtu.be/osxTZkMTI2M

/sf versions ``` > sf versions [09:40:00 INFO]: This Server uses the following setup of Slimefun: Paper git-Paper-133 (MC: 1.18.1) Slimefun DEV - 992 (git 0538e345) Metrics-Module #28 Java 17 No Addons installed > pl [09:40:01 INFO]: Plugins (1): Slimefun > ver [09:40:07 INFO]: Checking version, please wait... [09:40:07 INFO]: This server is running Paper version git-Paper-133 (MC: 1.18.1) (Implementing API version 1.18.1-R0.1-SNAPSHOT) (Git: 5e30e19) You are running the latest version Previous version: git-Paper-121 (MC: 1.18.1) ```
Boomer-1 commented 2 years ago

I've been testing this and other blocks, and I believe I may have figured out why. While in the real world no one really places and item and mines it up right away, I think this is what's happening.

Minecraft ticks faster than slimefun, so follow this logic

as you mine the accelerator with your left hand, and are holding down right click to place another, once the placed one is mined, the next mc tick the one in your right hand is placed. at the same time, the next sf tick, slimefun goes to delete the stored data of that accelerator. Because a new one is already placed, sf deletes the data of that new block you just placed, and it reverts to a vanilla block. slimefun doesn't go to update the stored data to become the new accelerator, because the block you just placed becomes a vanilla hay bale as a result of deleting the data on the new block, and therefore it's no longer a slimefun item.

FN-FAL113 commented 2 years ago

Can be fixed in the blockstorage rewrite, had this problem on block breaker, and even tested it on android miner, fast place and break will turn the sf block to Normal vanilla, that's why I ditched sf block breaking for electric block breaker and opt to vanilla only for the meantime. Blockstorage checks is not synchronized with mc ticks iirc

Boomer-1 commented 11 months ago

this has been fixed with the release of dev version 16 on blob builds.