Skip to content

fluid regulators now check that the correct amount of fluid was moved#4693

Merged
TarLaboratories merged 1 commit intoGregTechCEu:1.20.1from
Bumperdo09:1.20.1
Feb 24, 2026
Merged

fluid regulators now check that the correct amount of fluid was moved#4693
TarLaboratories merged 1 commit intoGregTechCEu:1.20.1from
Bumperdo09:1.20.1

Conversation

@Bumperdo09
Copy link
Contributor

What

Fixes #4692

Implementation Details

This new check also does the same as the old check (that insertableAmount > 0), because supplyAmount > 0 is checked earlier in the method

How Was This Tested

Tested the same way as to cause the bug
Also tested with multiple fluid types being moved (by using fluid buffers instead of drums with the same setup to cause the bug), the behavior is that each fluid type is treated independently, which is intended.

@Bumperdo09 Bumperdo09 requested a review from a team as a code owner February 24, 2026 02:27
@TarLaboratories TarLaboratories added type: bugfix General bug fixes Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. labels Feb 24, 2026
@TarLaboratories TarLaboratories merged commit 47847e7 into GregTechCEu:1.20.1 Feb 24, 2026
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. type: bugfix General bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fluid regulator supply exact doesnt consider if the target tank can fully accept the supplied amount

2 participants