-
Notifications
You must be signed in to change notification settings - Fork 408
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
Fix Fluid Router GUI crash #6173
base: 1.21.1
Are you sure you want to change the base?
Conversation
Address issue BluSunrize#6172 by adding check for Empty fluid in `setFluidInSlot`
With this fix, the Fluid Router GUI no longer crashes with an Illegal State Exception. I was finally able to configure it for my automated potion of Strength setup, feeding into my refinery with the goal of producing cetane-boosted diesel. https://imgur.com/gallery/immersive-engineering-fixed-fluid-router-gui-crash-KLUogXg |
Noticed that the Fluid Router settings are not working at all on multiplayer. Nothing happens. |
@@ -86,7 +90,8 @@ public void receiveMessageFromScreen(CompoundTag message) | |||
int side = message.getInt("filter_side"); | |||
int slot = message.getInt("filter_slot"); | |||
FluidStack newFilter = FluidStack.parseOptional( | |||
Minecraft.getInstance().level.registryAccess(), | |||
((MinecraftServer)LogicalSidedProvider.WORKQUEUE.get(LogicalSide.SERVER)).overworld() | |||
.registryAccess(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Surely there is a cleaner way to do this? Maybe via the player?
Fix #6172 by adding check for Empty fluid in
setFluidInSlot
Fix Fluid Router not working on remote world (Multiplayer)