Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG: Mushrooms replace traders #226

Closed
SU-5-KsavvaZ opened this issue May 28, 2024 · 5 comments
Closed

BUG: Mushrooms replace traders #226

SU-5-KsavvaZ opened this issue May 28, 2024 · 5 comments
Assignees
Labels
Bug Something isn't working Fixed in dev Fixed in an unreleased version of the mod, or fix is not yet published to all supported versions. Other Mods Issue Issue is caused by another mod, and is not my responsibility to fix

Comments

@SU-5-KsavvaZ
Copy link

Minecraft Version

1.18.2

Forge Version

40.2.21

Lightman's Currency Version

2.1.2.5a

Environment Type

Minecraft Client (Bug/crash happened in a single-player world or while the game was booting)

Other relevant Mods

Create

Describe the bug

If you grow a large mushroom, it can replace the blocks of trader machines. Tree fetilizer from the create mod only aggravates the problem, since it allows you to grow trees anywhere, as well as replace blocks of trader machines

To Reproduce

  1. place trader block
  2. place micelium block close to it
  3. place mushroom on it
  4. grow it using bonemeal

Screenshots/Logs/Crash Reports

image
image

@SU-5-KsavvaZ SU-5-KsavvaZ added the Bug Something isn't working label May 28, 2024
@Lightman314
Copy link
Owner

Lightman314 commented May 29, 2024

Looked at the mushroom code, and apparently mushrooms can replace any block that isn't a full block. After testing this includes not only my blocks, but also many vanilla blocks such as signs, stonecutters, and even chests.

While I'm definitely going to try to make my traders immune to this bug, this is 100% a vanilla Minecraft issue, not something caused by any mistakes on my part.

If it helps, quark appears to have an option to fix this bug by forcing mushrooms to use tree growing logic (can only replace air or leaves), so that should help you in the meantime if this is somehow an actual major issue for you.

Edit:
Never thought I'd ever see the day when I used the "Other Mods Issue" for a Minecraft issue, but so I have.

For now though, I've used the forge SaplingGrowTreeEvent to block all mushrooms from growing with any LC block 3 or fewer blocks away horizontally and 10 or fewer blocks above the mushroom of origin.

@Lightman314 Lightman314 added Other Mods Issue Issue is caused by another mod, and is not my responsibility to fix Fixed in dev Fixed in an unreleased version of the mod, or fix is not yet published to all supported versions. labels May 29, 2024
@SU-5-KsavvaZ
Copy link
Author

Looked at the mushroom code, and apparently mushrooms can replace any block that isn't a full block. After testing this includes not only my blocks, but also many vanilla blocks such as signs, stonecutters, and even chests.

While I'm definitely going to try to make my traders immune to this bug, this is 100% a vanilla Minecraft issue, not something caused by any mistakes on my part.

If it helps, quark appears to have an option to fix this bug by forcing mushrooms to use tree growing logic (can only replace air or leaves), so that should help you in the meantime if this is somehow an actual major issue for you.

Edit:
Never thought I'd ever see the day when I used the "Other Mods Issue" for a Minecraft issue, but so I have.

For now though, I've used the forge SaplingGrowTreeEvent to block all mushrooms from growing with any LC block 3 or fewer blocks away horizontally and 10 or fewer blocks above the mushroom of origin.

I always thought that breaking blocks with mushrooms was a kind of minecraft feature, not a bug. Thanks for solving the problem, I'll check your solutions later.

@SU-5-KsavvaZ
Copy link
Author

In any case mushrooms have been able to destroy blocks for years and I don't think modjang will do anything about it, especially on older versions.

@Lightman314
Copy link
Owner

I always thought that breaking blocks with mushrooms was a kind of minecraft feature, not a bug. Thanks for solving the problem, I'll check your solutions later.

While I agree that for vanilla it could be considered a "feature" not a bug, the fact that there's no way to flag your block as something that shouldn't be destroyed by growing mushrooms (such as a block tag) is definitely an issue from a modding standpoint. Heck, even the ender dragon and wither have "X_immune" block tags that I can utilize to prevent my traders from being griefed by players spawning withers inside of malls, etc.

@Lightman314
Copy link
Owner

Fixed in 1.20.1 in v2.2.2.0. Fix will be backported to 1.19.2, but I have no plans to update LC 1.18.2 at this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Fixed in dev Fixed in an unreleased version of the mod, or fix is not yet published to all supported versions. Other Mods Issue Issue is caused by another mod, and is not my responsibility to fix
Projects
None yet
Development

No branches or pull requests

2 participants