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

Powered machines not updating state correctly #1006

Open
wants to merge 2 commits into
base: dev/1.21.1
Choose a base branch
from

Conversation

CrazyPants
Copy link
Contributor

@CrazyPants CrazyPants commented Feb 19, 2025

Description

State was only being updated when canUse returned true. This meant that if the state changed so that canUse went from true to false the state change was not applied.
An example of this occurring is turning on 'only active with redstone signal'. This results in canUse returning false so the the check fails and the state update is not made. This results, for example, the StirlingGen block still showing its active 'running' block on the client when it has been disabled.

Checklist

  • My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you).
  • I have made corresponding changes to the documentation.
  • My changes are ready for review from a contributor.

…d not present. As such, if you toggle a machine to 'requires redstone' the machine gets stuck showing that it is active. For example, if a StirlingGenerator is changed to required redstone the generator keeps showing its active animation despite no being active.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants