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: Milk Bucket Fluid Capabilities #2569

Open
6 tasks done
VoidLeech opened this issue Feb 14, 2025 · 0 comments
Open
6 tasks done

Bug: Milk Bucket Fluid Capabilities #2569

VoidLeech opened this issue Feb 14, 2025 · 0 comments
Assignees
Labels
loader/neoforge This issue affects the NeoForge modloader. status/in-progress The issue is currently being worked on. type/compatibility The issue regards a compatibility or additional feature request relating to another mod. version/1.20.1 This issue affects Minecraft version 1.20.1. version/1.21.1 This issue affects Minecraft version 1.21.1.

Comments

@VoidLeech
Copy link

What Feature Types Apply to This Bug?

Item

Other Type

No response

What Type of Bug Is This?

Unexpected Behavior

Mod Loader

Neoforge

Mod Loader Version

47.1.70, 21.1.72

The Aether Version

1.5.2, 1.5.4

Is This Bug a Conflict With Another Mod?

No response

Client Log

No response

Crash Report (if applicable)

No response

Steps to Reproduce

  1. Query for ForgeCapabilities.FLUID_HANDLER_ITEM/Capabilities.FluidHandler.ITEM on a stack of the Skyroot Milk Bucket
  2. Call getContainer() on the handler.

What You Expect To Happen

The returned container is a skyroot bucket.

What Actually Happened

On 1.21.1, there is no capability.
On 1.20.1, this returned container is a normal bucket, not a skyroot bucket. This can seen in action with a mod like Create, which generates certain recipes from fluid capabilities: Creators-of-Create/Create#7084

Additional Details

Code analysis:
1.20.1 registered the capability via MilkBucketItem while 1.21.1 doesn't, explaining the missing capability.

1.20.1's MilkBucketItem's initCapabilities should be overridden: it currently returns a FluidBucketWrapper, whose default setFluid method will set the container to a normal bucket if the fluid is empty)

Please Read and Confirm The Following

  • I have confirmed this bug can be replicated without the use of Optifine.
  • I have confirmed this bug is on the most recently supported version of Minecraft.
  • I have confirmed the details provided in this report are concise as possible and does not contain vague information (ie. Versions are properly recorded, answers to questions are clear).
  • I have confirmed this bug is unique and has not been reported already.
  • If playing on a modpack, I have reported this bug to their issue tracker already.
  • I have confirmed that I'm reporting a bug in The Aether I, not The Aether II.
@github-project-automation github-project-automation bot moved this to Back Log in The Aether Feb 14, 2025
@bconlon1 bconlon1 added the status/in-progress The issue is currently being worked on. label Feb 15, 2025
@bconlon1 bconlon1 self-assigned this Feb 15, 2025
@bconlon1 bconlon1 moved this from Back Log to In Progress in The Aether Feb 17, 2025
@bconlon1 bconlon1 added version/1.20.1 This issue affects Minecraft version 1.20.1. version/1.21.1 This issue affects Minecraft version 1.21.1. loader/neoforge This issue affects the NeoForge modloader. type/compatibility The issue regards a compatibility or additional feature request relating to another mod. and removed type/bug labels Feb 17, 2025
@bconlon1 bconlon1 moved this from In Progress to Review Requested in The Aether Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
loader/neoforge This issue affects the NeoForge modloader. status/in-progress The issue is currently being worked on. type/compatibility The issue regards a compatibility or additional feature request relating to another mod. version/1.20.1 This issue affects Minecraft version 1.20.1. version/1.21.1 This issue affects Minecraft version 1.21.1.
Projects
Status: Review Requested
Development

No branches or pull requests

2 participants