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

Upgrading Energy Conduit stops transmitting power #908

Open
Rover656 opened this issue Dec 9, 2024 · 2 comments
Open

Upgrading Energy Conduit stops transmitting power #908

Rover656 opened this issue Dec 9, 2024 · 2 comments

Comments

@Rover656
Copy link
Member

Rover656 commented Dec 9, 2024

Needs investigation, reported via Monifactory survey

@Y3RmKHgp
Copy link

i've never done minecraft modding so I don't really know how to test this, but from a quick glance of the code:

if (action instanceof RightClickAction.Upgrade upgrade && !upgrade.replacedConduit().value().canConnectTo(conduit)) {
removeNeighborConnections(upgrade.replacedConduit());
}

upgrade.replacedConduit() returns the old conduit, that was upgraded. So this check always returns true when an energy conduit is upgraded.

therefore removeNeighborConnections() is getting called every time an energy conduit is upgraded.

I think logically, an easier way to handle upgrades would just be to treat it the same as a break then a placement.

@TheMDVF
Copy link

TheMDVF commented Dec 23, 2024

I did some tests and as Y3RmKHgp said and seems like when upgrading any conduit it changes its connections. For energy conduits, the extract seems to remain after the upgrade, but the insert doesn't. And for fluids, both needed to be re-toggled.
https://youtu.be/hsEU6EoMbuA

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

No branches or pull requests

3 participants